@@ -74,7 +74,7 @@ discard block |
||
| 74 | 74 | protected function configure(): void |
| 75 | 75 | { |
| 76 | 76 | $plugins = array_map( |
| 77 | - function ($class) { |
|
| 77 | + function($class) { |
|
| 78 | 78 | return '"'.substr($class, strlen('Error')).'"'; |
| 79 | 79 | }, |
| 80 | 80 | $this->getAvailablePlugins() |
@@ -262,7 +262,7 @@ discard block |
||
| 262 | 262 | protected function disablePlugins(array $disabledPlugins, array $plugins): array |
| 263 | 263 | { |
| 264 | 264 | $disabledPlugins = array_map( |
| 265 | - function ($param) { |
|
| 265 | + function($param) { |
|
| 266 | 266 | return strtolower($param); |
| 267 | 267 | }, |
| 268 | 268 | $disabledPlugins |
@@ -203,7 +203,7 @@ |
||
| 203 | 203 | } |
| 204 | 204 | |
| 205 | 205 | // delete folder recursive |
| 206 | - if (! $iterator->isDot() && $iterator->isDir()) { |
|
| 206 | + if (!$iterator->isDot() && $iterator->isDir()) { |
|
| 207 | 207 | self::deleteDirectory($src); |
| 208 | 208 | } |
| 209 | 209 | |
@@ -228,7 +228,7 @@ |
||
| 228 | 228 | '/some/file/name.php', |
| 229 | 229 | [$this->issues[0], $this->issues[1]] |
| 230 | 230 | ); |
| 231 | - $otherFile = new File( |
|
| 231 | + $otherFile = new File( |
|
| 232 | 232 | '/some/file/name.php', |
| 233 | 233 | [$this->issues[2]] |
| 234 | 234 | ); |
@@ -117,12 +117,12 @@ discard block |
||
| 117 | 117 | $expectedFile = PHPCB_TEST_OUTPUT.DIRECTORY_SEPARATOR.basename(__FILE__).'.html'; |
| 118 | 118 | |
| 119 | 119 | $this->ioMock->expects($this->once()) |
| 120 | - ->method('loadFile') |
|
| 121 | - ->with($this->equalTo(__FILE__)) |
|
| 122 | - ->will($this->returnValue(file_get_contents(__FILE__))); |
|
| 120 | + ->method('loadFile') |
|
| 121 | + ->with($this->equalTo(__FILE__)) |
|
| 122 | + ->will($this->returnValue(file_get_contents(__FILE__))); |
|
| 123 | 123 | $this->ioMock->expects($this->once()) |
| 124 | - ->method('createFile') |
|
| 125 | - ->with($this->equalTo($expectedFile)); |
|
| 124 | + ->method('createFile') |
|
| 125 | + ->with($this->equalTo($expectedFile)); |
|
| 126 | 126 | |
| 127 | 127 | $this->viewReview->generate( |
| 128 | 128 | [], |
@@ -151,12 +151,12 @@ discard block |
||
| 151 | 151 | |
| 152 | 152 | $expectedFile = PHPCB_TEST_OUTPUT.DIRECTORY_SEPARATOR.basename(__FILE__).'.html'; |
| 153 | 153 | $this->ioMock->expects($this->once()) |
| 154 | - ->method('loadFile') |
|
| 155 | - ->with($this->equalTo(__FILE__)) |
|
| 156 | - ->will($this->returnValue(file_get_contents(__FILE__))); |
|
| 154 | + ->method('loadFile') |
|
| 155 | + ->with($this->equalTo(__FILE__)) |
|
| 156 | + ->will($this->returnValue(file_get_contents(__FILE__))); |
|
| 157 | 157 | $this->ioMock->expects($this->once()) |
| 158 | - ->method('createFile') |
|
| 159 | - ->with($this->equalTo($expectedFile)); |
|
| 158 | + ->method('createFile') |
|
| 159 | + ->with($this->equalTo($expectedFile)); |
|
| 160 | 160 | |
| 161 | 161 | $this->viewReview->generate( |
| 162 | 162 | $issueList, |
@@ -179,12 +179,12 @@ discard block |
||
| 179 | 179 | |
| 180 | 180 | $expectedFile = PHPCB_TEST_OUTPUT.DIRECTORY_SEPARATOR.basename(__FILE__).'.html'; |
| 181 | 181 | $this->ioMock->expects($this->once()) |
| 182 | - ->method('loadFile') |
|
| 183 | - ->with($this->equalTo(__FILE__)) |
|
| 184 | - ->will($this->returnValue(file_get_contents(__FILE__))); |
|
| 182 | + ->method('loadFile') |
|
| 183 | + ->with($this->equalTo(__FILE__)) |
|
| 184 | + ->will($this->returnValue(file_get_contents(__FILE__))); |
|
| 185 | 185 | $this->ioMock->expects($this->once()) |
| 186 | - ->method('createFile') |
|
| 187 | - ->with($this->equalTo($expectedFile)); |
|
| 186 | + ->method('createFile') |
|
| 187 | + ->with($this->equalTo($expectedFile)); |
|
| 188 | 188 | |
| 189 | 189 | $this->viewReview->generate( |
| 190 | 190 | $issueList, |
@@ -219,12 +219,12 @@ discard block |
||
| 219 | 219 | |
| 220 | 220 | $expectedFile = PHPCB_TEST_OUTPUT.'/file.html.html'; |
| 221 | 221 | $this->ioMock->expects($this->once()) |
| 222 | - ->method('loadFile') |
|
| 223 | - ->with($this->equalTo($fileName)) |
|
| 224 | - ->will($this->returnValue($html)); |
|
| 222 | + ->method('loadFile') |
|
| 223 | + ->with($this->equalTo($fileName)) |
|
| 224 | + ->will($this->returnValue($html)); |
|
| 225 | 225 | $this->ioMock->expects($this->once()) |
| 226 | - ->method('createFile') |
|
| 227 | - ->with($this->equalTo($expectedFile)); |
|
| 226 | + ->method('createFile') |
|
| 227 | + ->with($this->equalTo($expectedFile)); |
|
| 228 | 228 | |
| 229 | 229 | $issues = [ |
| 230 | 230 | new Issue($fileName, 5, 5, 'finder', 'description', 'severity'), |
@@ -243,8 +243,8 @@ discard block |
||
| 243 | 243 | $expectedFile = PHPCB_TEST_OUTPUT.DIRECTORY_SEPARATOR.basename(self::$xmlBasic).'.html'; |
| 244 | 244 | |
| 245 | 245 | $this->ioMock->expects($this->once()) |
| 246 | - ->method('createFile') |
|
| 247 | - ->with($this->equalTo($expectedFile)); |
|
| 246 | + ->method('createFile') |
|
| 247 | + ->with($this->equalTo($expectedFile)); |
|
| 248 | 248 | |
| 249 | 249 | $issueList = [ |
| 250 | 250 | new Issue(self::$xmlBasic, 5, 5, 'finder', 'description', 'severity'), |
@@ -265,12 +265,12 @@ discard block |
||
| 265 | 265 | public function testCopyResourceFolders(): void |
| 266 | 266 | { |
| 267 | 267 | $this->ioMock->expects($this->exactly(3)) |
| 268 | - ->method('copyDirectory') |
|
| 269 | - ->with( |
|
| 270 | - $this->matchesRegularExpression( |
|
| 271 | - '|^'.realpath(dirname(__FILE__).'/../../../templates/').'|' |
|
| 272 | - ) |
|
| 273 | - ); |
|
| 268 | + ->method('copyDirectory') |
|
| 269 | + ->with( |
|
| 270 | + $this->matchesRegularExpression( |
|
| 271 | + '|^'.realpath(dirname(__FILE__).'/../../../templates/').'|' |
|
| 272 | + ) |
|
| 273 | + ); |
|
| 274 | 274 | $this->viewReview->copyResourceFolders(); |
| 275 | 275 | } |
| 276 | 276 | |
@@ -288,12 +288,12 @@ discard block |
||
| 288 | 288 | ]; |
| 289 | 289 | |
| 290 | 290 | $this->ioMock->expects($this->once()) |
| 291 | - ->method('createFile') |
|
| 292 | - ->with( |
|
| 293 | - $this->logicalAnd( |
|
| 294 | - $this->stringEndsWith('index.html') |
|
| 295 | - ) |
|
| 296 | - ); |
|
| 291 | + ->method('createFile') |
|
| 292 | + ->with( |
|
| 293 | + $this->logicalAnd( |
|
| 294 | + $this->stringEndsWith('index.html') |
|
| 295 | + ) |
|
| 296 | + ); |
|
| 297 | 297 | $this->viewReview->generateIndex($files); |
| 298 | 298 | } |
| 299 | 299 | } |
@@ -204,7 +204,7 @@ |
||
| 204 | 204 | $this->markTestIncomplete(); |
| 205 | 205 | } |
| 206 | 206 | |
| 207 | - $html = <<< EOT |
|
| 207 | + $html = <<< EOT |
|
| 208 | 208 | <html> |
| 209 | 209 | <head> |
| 210 | 210 | <title>Title</title> |
@@ -196,7 +196,7 @@ |
||
| 196 | 196 | } |
| 197 | 197 | |
| 198 | 198 | // delete folder recursive |
| 199 | - if (! $iterator->isDot() && $iterator->isDir()) { |
|
| 199 | + if (!$iterator->isDot() && $iterator->isDir()) { |
|
| 200 | 200 | $this->cleanUp($dir.'/'.$iterator->current()); |
| 201 | 201 | rmdir($dir.'/'.$iterator->current()); |
| 202 | 202 | } |
@@ -151,7 +151,7 @@ |
||
| 151 | 151 | ] |
| 152 | 152 | ), |
| 153 | 153 | ]; |
| 154 | - $actual = $this->errorCPD->getFileList(); |
|
| 154 | + $actual = $this->errorCPD->getFileList(); |
|
| 155 | 155 | $this->assertEquals($expected, $actual); |
| 156 | 156 | } |
| 157 | 157 | } |
@@ -179,7 +179,7 @@ |
||
| 179 | 179 | [] |
| 180 | 180 | ), |
| 181 | 181 | ]; |
| 182 | - $actual = $this->errorPmd->getFileList(); |
|
| 182 | + $actual = $this->errorPmd->getFileList(); |
|
| 183 | 183 | $this->assertEquals($expected, $actual); |
| 184 | 184 | } |
| 185 | 185 | } |
@@ -178,7 +178,7 @@ |
||
| 178 | 178 | [] |
| 179 | 179 | ), |
| 180 | 180 | ]; |
| 181 | - $actual = $this->errorCheckstyle->getFileList(); |
|
| 181 | + $actual = $this->errorCheckstyle->getFileList(); |
|
| 182 | 182 | $this->assertEquals($expected, $actual); |
| 183 | 183 | } |
| 184 | 184 | } |
@@ -175,7 +175,7 @@ discard block |
||
| 175 | 175 | [] |
| 176 | 176 | ), |
| 177 | 177 | ]; |
| 178 | - $actual = $this->errorCrap->getFileList(); |
|
| 178 | + $actual = $this->errorCrap->getFileList(); |
|
| 179 | 179 | $this->assertEquals($expected, $actual); |
| 180 | 180 | } |
| 181 | 181 | |
@@ -214,7 +214,7 @@ discard block |
||
| 214 | 214 | [] |
| 215 | 215 | ), |
| 216 | 216 | ]; |
| 217 | - $actual = $this->errorCrap->getFileList(); |
|
| 217 | + $actual = $this->errorCrap->getFileList(); |
|
| 218 | 218 | $this->assertEquals($expected, $actual); |
| 219 | 219 | } |
| 220 | 220 | } |