@@ -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 | } |