@@ -14,13 +14,13 @@ discard block |
||
14 | 14 | class StraussConfigTest extends TestCase |
15 | 15 | { |
16 | 16 | |
17 | - /** |
|
18 | - * With a full (at time of writing) config, test the getters. |
|
19 | - */ |
|
20 | - public function testGetters() |
|
21 | - { |
|
17 | + /** |
|
18 | + * With a full (at time of writing) config, test the getters. |
|
19 | + */ |
|
20 | + public function testGetters() |
|
21 | + { |
|
22 | 22 | |
23 | - $composerExtraStraussJson = <<<'EOD' |
|
23 | + $composerExtraStraussJson = <<<'EOD' |
|
24 | 24 | { |
25 | 25 | "name": "brianhenryie/strauss-config-test", |
26 | 26 | "require": { |
@@ -50,38 +50,38 @@ discard block |
||
50 | 50 | } |
51 | 51 | EOD; |
52 | 52 | |
53 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
54 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
53 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
54 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
55 | 55 | |
56 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
56 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
57 | 57 | |
58 | - $sut = new StraussConfig($composer); |
|
58 | + $sut = new StraussConfig($composer); |
|
59 | 59 | |
60 | - $this->assertContains('pimple/pimple', $sut->getPackages()); |
|
60 | + $this->assertContains('pimple/pimple', $sut->getPackages()); |
|
61 | 61 | |
62 | - $this->assertEquals('target_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory()); |
|
62 | + $this->assertEquals('target_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory()); |
|
63 | 63 | |
64 | - $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix()); |
|
64 | + $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix()); |
|
65 | 65 | |
66 | - $this->assertEquals('BrianHenryIE_Strauss_', $sut->getClassmapPrefix()); |
|
66 | + $this->assertEquals('BrianHenryIE_Strauss_', $sut->getClassmapPrefix()); |
|
67 | 67 | |
68 | - // @see https://github.com/BrianHenryIE/strauss/issues/14 |
|
69 | - $this->assertContains('/^psr.*$/', $sut->getExcludeFilePatternsFromPrefixing()); |
|
68 | + // @see https://github.com/BrianHenryIE/strauss/issues/14 |
|
69 | + $this->assertContains('/^psr.*$/', $sut->getExcludeFilePatternsFromPrefixing()); |
|
70 | 70 | |
71 | - $this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload()); |
|
71 | + $this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload()); |
|
72 | 72 | |
73 | - $this->assertFalse($sut->isDeleteVendorFiles()); |
|
74 | - } |
|
73 | + $this->assertFalse($sut->isDeleteVendorFiles()); |
|
74 | + } |
|
75 | 75 | |
76 | - /** |
|
77 | - * Test how it handles an extra key. |
|
78 | - * |
|
79 | - * Turns out it just ignores it... good! |
|
80 | - */ |
|
81 | - public function testExtraKey() |
|
82 | - { |
|
76 | + /** |
|
77 | + * Test how it handles an extra key. |
|
78 | + * |
|
79 | + * Turns out it just ignores it... good! |
|
80 | + */ |
|
81 | + public function testExtraKey() |
|
82 | + { |
|
83 | 83 | |
84 | - $composerExtraStraussJson = <<<'EOD' |
|
84 | + $composerExtraStraussJson = <<<'EOD' |
|
85 | 85 | { |
86 | 86 | "name": "brianhenryie/strauss-config-test", |
87 | 87 | "require": { |
@@ -112,31 +112,31 @@ discard block |
||
112 | 112 | } |
113 | 113 | EOD; |
114 | 114 | |
115 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
116 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
115 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
116 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
117 | 117 | |
118 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
118 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
119 | 119 | |
120 | - $exception = null; |
|
120 | + $exception = null; |
|
121 | 121 | |
122 | - try { |
|
123 | - $sut = new StraussConfig($composer); |
|
124 | - } catch (\Exception $e) { |
|
125 | - $exception = $e; |
|
126 | - } |
|
122 | + try { |
|
123 | + $sut = new StraussConfig($composer); |
|
124 | + } catch (\Exception $e) { |
|
125 | + $exception = $e; |
|
126 | + } |
|
127 | 127 | |
128 | - $this->assertNull($exception); |
|
129 | - } |
|
128 | + $this->assertNull($exception); |
|
129 | + } |
|
130 | 130 | |
131 | - /** |
|
132 | - * straussconfig-test-3.json has no target_dir key. |
|
133 | - * |
|
134 | - * If no target_dir is specified, used "strauss/" |
|
135 | - */ |
|
136 | - public function testDefaultTargetDir() |
|
137 | - { |
|
131 | + /** |
|
132 | + * straussconfig-test-3.json has no target_dir key. |
|
133 | + * |
|
134 | + * If no target_dir is specified, used "strauss/" |
|
135 | + */ |
|
136 | + public function testDefaultTargetDir() |
|
137 | + { |
|
138 | 138 | |
139 | - $composerExtraStraussJson = <<<'EOD' |
|
139 | + $composerExtraStraussJson = <<<'EOD' |
|
140 | 140 | { |
141 | 141 | "name": "brianhenryie/strauss-config-test", |
142 | 142 | "require": { |
@@ -163,23 +163,23 @@ discard block |
||
163 | 163 | } |
164 | 164 | EOD; |
165 | 165 | |
166 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
167 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
166 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
167 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
168 | 168 | |
169 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
169 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
170 | 170 | |
171 | - $sut = new StraussConfig($composer); |
|
171 | + $sut = new StraussConfig($composer); |
|
172 | 172 | |
173 | - $this->assertEquals('strauss'. DIRECTORY_SEPARATOR, $sut->getTargetDirectory()); |
|
174 | - } |
|
173 | + $this->assertEquals('strauss'. DIRECTORY_SEPARATOR, $sut->getTargetDirectory()); |
|
174 | + } |
|
175 | 175 | |
176 | - /** |
|
177 | - * When the namespace prefix isn't provided, use the PSR-4 autoload key name. |
|
178 | - */ |
|
179 | - public function testDefaultNamespacePrefixFromAutoloaderPsr4() |
|
180 | - { |
|
176 | + /** |
|
177 | + * When the namespace prefix isn't provided, use the PSR-4 autoload key name. |
|
178 | + */ |
|
179 | + public function testDefaultNamespacePrefixFromAutoloaderPsr4() |
|
180 | + { |
|
181 | 181 | |
182 | - $composerExtraStraussJson = <<<'EOD' |
|
182 | + $composerExtraStraussJson = <<<'EOD' |
|
183 | 183 | { |
184 | 184 | "name": "brianhenryie/strauss-config-test", |
185 | 185 | "require": { |
@@ -194,23 +194,23 @@ discard block |
||
194 | 194 | } |
195 | 195 | EOD; |
196 | 196 | |
197 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
198 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
197 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
198 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
199 | 199 | |
200 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
200 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
201 | 201 | |
202 | - $sut = new StraussConfig($composer); |
|
202 | + $sut = new StraussConfig($composer); |
|
203 | 203 | |
204 | - $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix()); |
|
205 | - } |
|
204 | + $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix()); |
|
205 | + } |
|
206 | 206 | |
207 | - /** |
|
208 | - * When the namespace prefix isn't provided, use the PSR-0 autoload key name. |
|
209 | - */ |
|
210 | - public function testDefaultNamespacePrefixFromAutoloaderPsr0() |
|
211 | - { |
|
207 | + /** |
|
208 | + * When the namespace prefix isn't provided, use the PSR-0 autoload key name. |
|
209 | + */ |
|
210 | + public function testDefaultNamespacePrefixFromAutoloaderPsr0() |
|
211 | + { |
|
212 | 212 | |
213 | - $composerExtraStraussJson = <<<'EOD' |
|
213 | + $composerExtraStraussJson = <<<'EOD' |
|
214 | 214 | { |
215 | 215 | "name": "brianhenryie/strauss-config-test", |
216 | 216 | "require": { |
@@ -224,25 +224,25 @@ discard block |
||
224 | 224 | } |
225 | 225 | } |
226 | 226 | EOD; |
227 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
228 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
227 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
228 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
229 | 229 | |
230 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
230 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
231 | 231 | |
232 | - $sut = new StraussConfig($composer); |
|
232 | + $sut = new StraussConfig($composer); |
|
233 | 233 | |
234 | - $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix()); |
|
235 | - } |
|
234 | + $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix()); |
|
235 | + } |
|
236 | 236 | |
237 | - /** |
|
238 | - * When the namespace prefix isn't provided, and there's no PSR-0 or PSR-4 autoloader to figure it from... |
|
239 | - * |
|
240 | - * brianhenryie/strauss-config-test |
|
241 | - */ |
|
242 | - public function testDefaultNamespacePrefixWithNoAutoloader() |
|
243 | - { |
|
237 | + /** |
|
238 | + * When the namespace prefix isn't provided, and there's no PSR-0 or PSR-4 autoloader to figure it from... |
|
239 | + * |
|
240 | + * brianhenryie/strauss-config-test |
|
241 | + */ |
|
242 | + public function testDefaultNamespacePrefixWithNoAutoloader() |
|
243 | + { |
|
244 | 244 | |
245 | - $composerExtraStraussJson = <<<'EOD' |
|
245 | + $composerExtraStraussJson = <<<'EOD' |
|
246 | 246 | { |
247 | 247 | "name": "brianhenryie/strauss-config-test", |
248 | 248 | "require": { |
@@ -251,23 +251,23 @@ discard block |
||
251 | 251 | } |
252 | 252 | EOD; |
253 | 253 | |
254 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
255 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
254 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
255 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
256 | 256 | |
257 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
257 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
258 | 258 | |
259 | - $sut = new StraussConfig($composer); |
|
259 | + $sut = new StraussConfig($composer); |
|
260 | 260 | |
261 | - $this->assertEquals("Brianhenryie\\Strauss_Config_Test", $sut->getNamespacePrefix()); |
|
262 | - } |
|
261 | + $this->assertEquals("Brianhenryie\\Strauss_Config_Test", $sut->getNamespacePrefix()); |
|
262 | + } |
|
263 | 263 | |
264 | - /** |
|
265 | - * When the classmap prefix isn't provided, use the PSR-4 autoload key name. |
|
266 | - */ |
|
267 | - public function testDefaultClassmapPrefixFromAutoloaderPsr4() |
|
268 | - { |
|
264 | + /** |
|
265 | + * When the classmap prefix isn't provided, use the PSR-4 autoload key name. |
|
266 | + */ |
|
267 | + public function testDefaultClassmapPrefixFromAutoloaderPsr4() |
|
268 | + { |
|
269 | 269 | |
270 | - $composerExtraStraussJson = <<<'EOD' |
|
270 | + $composerExtraStraussJson = <<<'EOD' |
|
271 | 271 | { |
272 | 272 | "name": "brianhenryie/strauss-config-test", |
273 | 273 | "require": { |
@@ -282,23 +282,23 @@ discard block |
||
282 | 282 | } |
283 | 283 | EOD; |
284 | 284 | |
285 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
286 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
285 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
286 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
287 | 287 | |
288 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
288 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
289 | 289 | |
290 | - $sut = new StraussConfig($composer); |
|
290 | + $sut = new StraussConfig($composer); |
|
291 | 291 | |
292 | - $this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix()); |
|
293 | - } |
|
292 | + $this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix()); |
|
293 | + } |
|
294 | 294 | |
295 | - /** |
|
296 | - * When the classmap prefix isn't provided, use the PSR-0 autoload key name. |
|
297 | - */ |
|
298 | - public function testDefaultClassmapPrefixFromAutoloaderPsr0() |
|
299 | - { |
|
295 | + /** |
|
296 | + * When the classmap prefix isn't provided, use the PSR-0 autoload key name. |
|
297 | + */ |
|
298 | + public function testDefaultClassmapPrefixFromAutoloaderPsr0() |
|
299 | + { |
|
300 | 300 | |
301 | - $composerExtraStraussJson = <<<'EOD' |
|
301 | + $composerExtraStraussJson = <<<'EOD' |
|
302 | 302 | { |
303 | 303 | "name": "brianhenryie/strauss-config-test", |
304 | 304 | "require": { |
@@ -312,26 +312,26 @@ discard block |
||
312 | 312 | } |
313 | 313 | } |
314 | 314 | EOD; |
315 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
316 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
315 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
316 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
317 | 317 | |
318 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
318 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
319 | 319 | |
320 | 320 | |
321 | - $sut = new StraussConfig($composer); |
|
321 | + $sut = new StraussConfig($composer); |
|
322 | 322 | |
323 | - $this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix()); |
|
324 | - } |
|
323 | + $this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix()); |
|
324 | + } |
|
325 | 325 | |
326 | - /** |
|
327 | - * When the classmap prefix isn't provided, and there's no PSR-0 or PSR-4 autoloader to figure it from... |
|
328 | - * |
|
329 | - * brianhenryie/strauss-config-test |
|
330 | - */ |
|
331 | - public function testDefaultClassmapPrefixWithNoAutoloader() |
|
332 | - { |
|
326 | + /** |
|
327 | + * When the classmap prefix isn't provided, and there's no PSR-0 or PSR-4 autoloader to figure it from... |
|
328 | + * |
|
329 | + * brianhenryie/strauss-config-test |
|
330 | + */ |
|
331 | + public function testDefaultClassmapPrefixWithNoAutoloader() |
|
332 | + { |
|
333 | 333 | |
334 | - $composerExtraStraussJson = <<<'EOD' |
|
334 | + $composerExtraStraussJson = <<<'EOD' |
|
335 | 335 | { |
336 | 336 | "name": "brianhenryie/strauss-config-test", |
337 | 337 | "require": { |
@@ -340,23 +340,23 @@ discard block |
||
340 | 340 | |
341 | 341 | } |
342 | 342 | EOD; |
343 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
344 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
343 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
344 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
345 | 345 | |
346 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
346 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
347 | 347 | |
348 | - $sut = new StraussConfig($composer); |
|
348 | + $sut = new StraussConfig($composer); |
|
349 | 349 | |
350 | - $this->assertEquals("Brianhenryie_Strauss_Config_Test", $sut->getClassmapPrefix()); |
|
351 | - } |
|
350 | + $this->assertEquals("Brianhenryie_Strauss_Config_Test", $sut->getClassmapPrefix()); |
|
351 | + } |
|
352 | 352 | |
353 | - /** |
|
354 | - * When Strauss config has packages specified, obviously use them. |
|
355 | - */ |
|
356 | - public function testGetPackagesFromConfig() |
|
357 | - { |
|
353 | + /** |
|
354 | + * When Strauss config has packages specified, obviously use them. |
|
355 | + */ |
|
356 | + public function testGetPackagesFromConfig() |
|
357 | + { |
|
358 | 358 | |
359 | - $composerExtraStraussJson = <<<'EOD' |
|
359 | + $composerExtraStraussJson = <<<'EOD' |
|
360 | 360 | { |
361 | 361 | "name": "brianhenryie/strauss-config-test", |
362 | 362 | "require": { |
@@ -386,23 +386,23 @@ discard block |
||
386 | 386 | } |
387 | 387 | |
388 | 388 | EOD; |
389 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
390 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
389 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
390 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
391 | 391 | |
392 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
392 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
393 | 393 | |
394 | - $sut = new StraussConfig($composer); |
|
394 | + $sut = new StraussConfig($composer); |
|
395 | 395 | |
396 | - $this->assertContains('pimple/pimple', $sut->getPackages()); |
|
397 | - } |
|
396 | + $this->assertContains('pimple/pimple', $sut->getPackages()); |
|
397 | + } |
|
398 | 398 | |
399 | - /** |
|
400 | - * When Strauss config has no packages specified, use composer.json's require list. |
|
401 | - */ |
|
402 | - public function testGetPackagesNoConfig() |
|
403 | - { |
|
399 | + /** |
|
400 | + * When Strauss config has no packages specified, use composer.json's require list. |
|
401 | + */ |
|
402 | + public function testGetPackagesNoConfig() |
|
403 | + { |
|
404 | 404 | |
405 | - $composerExtraStraussJson = <<<'EOD' |
|
405 | + $composerExtraStraussJson = <<<'EOD' |
|
406 | 406 | { |
407 | 407 | "name": "brianhenryie/strauss-config-test", |
408 | 408 | "require": { |
@@ -428,23 +428,23 @@ discard block |
||
428 | 428 | } |
429 | 429 | } |
430 | 430 | EOD; |
431 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
432 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
431 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
432 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
433 | 433 | |
434 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
434 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
435 | 435 | |
436 | - $sut = new StraussConfig($composer); |
|
436 | + $sut = new StraussConfig($composer); |
|
437 | 437 | |
438 | - $this->assertContains('league/container', $sut->getPackages()); |
|
439 | - } |
|
438 | + $this->assertContains('league/container', $sut->getPackages()); |
|
439 | + } |
|
440 | 440 | |
441 | - /** |
|
442 | - * For backwards compatibility, if a Mozart config is present, use it. |
|
443 | - */ |
|
444 | - public function testMapMozartConfig() |
|
445 | - { |
|
441 | + /** |
|
442 | + * For backwards compatibility, if a Mozart config is present, use it. |
|
443 | + */ |
|
444 | + public function testMapMozartConfig() |
|
445 | + { |
|
446 | 446 | |
447 | - $composerExtraStraussJson = <<<'EOD' |
|
447 | + $composerExtraStraussJson = <<<'EOD' |
|
448 | 448 | { |
449 | 449 | "extra": { |
450 | 450 | "mozart": { |
@@ -469,41 +469,41 @@ discard block |
||
469 | 469 | } |
470 | 470 | } |
471 | 471 | EOD; |
472 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
473 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
472 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
473 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
474 | 474 | |
475 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
475 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
476 | 476 | |
477 | - $sut = new StraussConfig($composer); |
|
477 | + $sut = new StraussConfig($composer); |
|
478 | 478 | |
479 | - $this->assertContains('pimple/pimple', $sut->getPackages()); |
|
479 | + $this->assertContains('pimple/pimple', $sut->getPackages()); |
|
480 | 480 | |
481 | - $this->assertEquals('dep_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory()); |
|
481 | + $this->assertEquals('dep_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory()); |
|
482 | 482 | |
483 | - $this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix()); |
|
483 | + $this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix()); |
|
484 | 484 | |
485 | - $this->assertEquals('My_Mozart_Config_', $sut->getClassmapPrefix()); |
|
485 | + $this->assertEquals('My_Mozart_Config_', $sut->getClassmapPrefix()); |
|
486 | 486 | |
487 | - $this->assertContains('psr/container', $sut->getExcludePackagesFromPrefixing()); |
|
487 | + $this->assertContains('psr/container', $sut->getExcludePackagesFromPrefixing()); |
|
488 | 488 | |
489 | - $this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload()); |
|
489 | + $this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload()); |
|
490 | 490 | |
491 | - // Mozart default was true. |
|
492 | - $this->assertTrue($sut->isDeleteVendorFiles()); |
|
493 | - } |
|
491 | + // Mozart default was true. |
|
492 | + $this->assertTrue($sut->isDeleteVendorFiles()); |
|
493 | + } |
|
494 | + |
|
495 | + /** |
|
496 | + * Since sometimes the namespace we're prefixing will already have a leading backslash, sometimes |
|
497 | + * the namespace_prefix will want its slash at the beginning, sometimes at the end. |
|
498 | + * |
|
499 | + * @see Prefixer::replaceNamespace() |
|
500 | + * |
|
501 | + * @covers \BrianHenryIE\Strauss\Composer\Extra\StraussConfig::getNamespacePrefix |
|
502 | + */ |
|
503 | + public function testNamespacePrefixHasNoSlash() |
|
504 | + { |
|
494 | 505 | |
495 | - /** |
|
496 | - * Since sometimes the namespace we're prefixing will already have a leading backslash, sometimes |
|
497 | - * the namespace_prefix will want its slash at the beginning, sometimes at the end. |
|
498 | - * |
|
499 | - * @see Prefixer::replaceNamespace() |
|
500 | - * |
|
501 | - * @covers \BrianHenryIE\Strauss\Composer\Extra\StraussConfig::getNamespacePrefix |
|
502 | - */ |
|
503 | - public function testNamespacePrefixHasNoSlash() |
|
504 | - { |
|
505 | - |
|
506 | - $composerExtraStraussJson = <<<'EOD' |
|
506 | + $composerExtraStraussJson = <<<'EOD' |
|
507 | 507 | { |
508 | 508 | "extra": { |
509 | 509 | "mozart": { |
@@ -512,13 +512,13 @@ discard block |
||
512 | 512 | } |
513 | 513 | } |
514 | 514 | EOD; |
515 | - $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
516 | - file_put_contents($tmpfname, $composerExtraStraussJson); |
|
515 | + $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-'); |
|
516 | + file_put_contents($tmpfname, $composerExtraStraussJson); |
|
517 | 517 | |
518 | - $composer = Factory::create(new NullIO(), $tmpfname); |
|
518 | + $composer = Factory::create(new NullIO(), $tmpfname); |
|
519 | 519 | |
520 | - $sut = new StraussConfig($composer); |
|
520 | + $sut = new StraussConfig($composer); |
|
521 | 521 | |
522 | - $this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix()); |
|
523 | - } |
|
522 | + $this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix()); |
|
523 | + } |
|
524 | 524 | } |
@@ -8,134 +8,134 @@ |
||
8 | 8 | class ComposerPackageTest extends TestCase |
9 | 9 | { |
10 | 10 | |
11 | - /** |
|
12 | - * A simple test to check the getters all work. |
|
13 | - */ |
|
14 | - public function testParseJson() |
|
15 | - { |
|
11 | + /** |
|
12 | + * A simple test to check the getters all work. |
|
13 | + */ |
|
14 | + public function testParseJson() |
|
15 | + { |
|
16 | 16 | |
17 | - $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
|
17 | + $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
|
18 | 18 | |
19 | - $composer = new ComposerPackage($testFile); |
|
19 | + $composer = new ComposerPackage($testFile); |
|
20 | 20 | |
21 | - $this->assertEquals('iio/libmergepdf', $composer->getName()); |
|
21 | + $this->assertEquals('iio/libmergepdf', $composer->getName()); |
|
22 | 22 | |
23 | - $this->assertIsArray($composer->getAutoload()); |
|
23 | + $this->assertIsArray($composer->getAutoload()); |
|
24 | 24 | |
25 | - $this->assertIsArray($composer->getRequiresNames()); |
|
26 | - } |
|
25 | + $this->assertIsArray($composer->getRequiresNames()); |
|
26 | + } |
|
27 | 27 | |
28 | - /** |
|
29 | - * Test the dependencies' names are returned. |
|
30 | - */ |
|
31 | - public function testGetRequiresNames() |
|
32 | - { |
|
28 | + /** |
|
29 | + * Test the dependencies' names are returned. |
|
30 | + */ |
|
31 | + public function testGetRequiresNames() |
|
32 | + { |
|
33 | 33 | |
34 | - $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
|
34 | + $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
|
35 | 35 | |
36 | - $composer = new ComposerPackage($testFile); |
|
36 | + $composer = new ComposerPackage($testFile); |
|
37 | 37 | |
38 | - $requiresNames = $composer->getRequiresNames(); |
|
38 | + $requiresNames = $composer->getRequiresNames(); |
|
39 | 39 | |
40 | - $this->assertContains('tecnickcom/tcpdf', $requiresNames); |
|
41 | - $this->assertContains('setasign/fpdi', $requiresNames); |
|
42 | - } |
|
40 | + $this->assertContains('tecnickcom/tcpdf', $requiresNames); |
|
41 | + $this->assertContains('setasign/fpdi', $requiresNames); |
|
42 | + } |
|
43 | 43 | |
44 | - /** |
|
45 | - * Test PHP and ext- are not returned, since we won't be dealing with them. |
|
46 | - */ |
|
47 | - public function testGetRequiresNamesDoesNotContain() |
|
48 | - { |
|
44 | + /** |
|
45 | + * Test PHP and ext- are not returned, since we won't be dealing with them. |
|
46 | + */ |
|
47 | + public function testGetRequiresNamesDoesNotContain() |
|
48 | + { |
|
49 | 49 | |
50 | - $testFile = __DIR__ . '/composerpackage-test-easypost-php.json'; |
|
50 | + $testFile = __DIR__ . '/composerpackage-test-easypost-php.json'; |
|
51 | 51 | |
52 | - $composer = new ComposerPackage($testFile); |
|
52 | + $composer = new ComposerPackage($testFile); |
|
53 | 53 | |
54 | - $requiresNames = $composer->getRequiresNames(); |
|
54 | + $requiresNames = $composer->getRequiresNames(); |
|
55 | 55 | |
56 | - $this->assertNotContains('ext-curl', $requiresNames); |
|
57 | - $this->assertNotContains('php', $requiresNames); |
|
58 | - } |
|
56 | + $this->assertNotContains('ext-curl', $requiresNames); |
|
57 | + $this->assertNotContains('php', $requiresNames); |
|
58 | + } |
|
59 | 59 | |
60 | 60 | |
61 | - /** |
|
62 | - * |
|
63 | - */ |
|
64 | - public function testAutoloadPsr0() |
|
65 | - { |
|
61 | + /** |
|
62 | + * |
|
63 | + */ |
|
64 | + public function testAutoloadPsr0() |
|
65 | + { |
|
66 | 66 | |
67 | - $testFile = __DIR__ . '/composerpackage-test-easypost-php.json'; |
|
67 | + $testFile = __DIR__ . '/composerpackage-test-easypost-php.json'; |
|
68 | 68 | |
69 | - $composer = new ComposerPackage($testFile); |
|
69 | + $composer = new ComposerPackage($testFile); |
|
70 | 70 | |
71 | - $autoload = $composer->getAutoload(); |
|
71 | + $autoload = $composer->getAutoload(); |
|
72 | 72 | |
73 | - $this->assertArrayHasKey('psr-0', $autoload); |
|
73 | + $this->assertArrayHasKey('psr-0', $autoload); |
|
74 | 74 | |
75 | - $this->assertIsArray($autoload['psr-0']); |
|
76 | - } |
|
75 | + $this->assertIsArray($autoload['psr-0']); |
|
76 | + } |
|
77 | 77 | |
78 | - /** |
|
79 | - * |
|
80 | - */ |
|
81 | - public function testAutoloadPsr4() |
|
82 | - { |
|
78 | + /** |
|
79 | + * |
|
80 | + */ |
|
81 | + public function testAutoloadPsr4() |
|
82 | + { |
|
83 | 83 | |
84 | - $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
|
84 | + $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
|
85 | 85 | |
86 | - $composer = new ComposerPackage($testFile); |
|
86 | + $composer = new ComposerPackage($testFile); |
|
87 | 87 | |
88 | - $autoload = $composer->getAutoload(); |
|
88 | + $autoload = $composer->getAutoload(); |
|
89 | 89 | |
90 | - $this->assertArrayHasKey('psr-4', $autoload); |
|
90 | + $this->assertArrayHasKey('psr-4', $autoload); |
|
91 | 91 | |
92 | - $this->assertIsArray($autoload['psr-4']); |
|
93 | - } |
|
92 | + $this->assertIsArray($autoload['psr-4']); |
|
93 | + } |
|
94 | 94 | |
95 | - /** |
|
96 | - * |
|
97 | - */ |
|
98 | - public function testAutoloadClassmap() |
|
99 | - { |
|
95 | + /** |
|
96 | + * |
|
97 | + */ |
|
98 | + public function testAutoloadClassmap() |
|
99 | + { |
|
100 | 100 | |
101 | - $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
|
101 | + $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
|
102 | 102 | |
103 | - $composer = new ComposerPackage($testFile); |
|
103 | + $composer = new ComposerPackage($testFile); |
|
104 | 104 | |
105 | - $autoload = $composer->getAutoload(); |
|
105 | + $autoload = $composer->getAutoload(); |
|
106 | 106 | |
107 | - $this->assertArrayHasKey('classmap', $autoload); |
|
107 | + $this->assertArrayHasKey('classmap', $autoload); |
|
108 | 108 | |
109 | - $this->assertIsArray($autoload['classmap']); |
|
110 | - } |
|
109 | + $this->assertIsArray($autoload['classmap']); |
|
110 | + } |
|
111 | 111 | |
112 | - /** |
|
113 | - * |
|
114 | - */ |
|
115 | - public function testAutoloadFiles() |
|
116 | - { |
|
112 | + /** |
|
113 | + * |
|
114 | + */ |
|
115 | + public function testAutoloadFiles() |
|
116 | + { |
|
117 | 117 | |
118 | - $testFile = __DIR__ . '/composerpackage-test-php-di.json'; |
|
118 | + $testFile = __DIR__ . '/composerpackage-test-php-di.json'; |
|
119 | 119 | |
120 | - $composer = new ComposerPackage($testFile); |
|
120 | + $composer = new ComposerPackage($testFile); |
|
121 | 121 | |
122 | - $autoload = $composer->getAutoload(); |
|
122 | + $autoload = $composer->getAutoload(); |
|
123 | 123 | |
124 | - $this->assertArrayHasKey('files', $autoload); |
|
124 | + $this->assertArrayHasKey('files', $autoload); |
|
125 | 125 | |
126 | - $this->assertIsArray($autoload['files']); |
|
127 | - } |
|
126 | + $this->assertIsArray($autoload['files']); |
|
127 | + } |
|
128 | 128 | |
129 | - public function testOverrideAutoload() |
|
130 | - { |
|
131 | - $this->markTestIncomplete(); |
|
132 | - } |
|
129 | + public function testOverrideAutoload() |
|
130 | + { |
|
131 | + $this->markTestIncomplete(); |
|
132 | + } |
|
133 | 133 | |
134 | - /** |
|
135 | - * When composer.json is not where it was specified, what error message (via Exception) should be returned? |
|
136 | - */ |
|
137 | - public function testMissingComposer() |
|
138 | - { |
|
139 | - $this->markTestIncomplete(); |
|
140 | - } |
|
134 | + /** |
|
135 | + * When composer.json is not where it was specified, what error message (via Exception) should be returned? |
|
136 | + */ |
|
137 | + public function testMissingComposer() |
|
138 | + { |
|
139 | + $this->markTestIncomplete(); |
|
140 | + } |
|
141 | 141 | } |
@@ -9,18 +9,18 @@ |
||
9 | 9 | class ProjectComposerPackageTest extends TestCase |
10 | 10 | { |
11 | 11 | |
12 | - /** |
|
13 | - * A simple test to check the getters all work. |
|
14 | - */ |
|
15 | - public function testParseJson() |
|
16 | - { |
|
12 | + /** |
|
13 | + * A simple test to check the getters all work. |
|
14 | + */ |
|
15 | + public function testParseJson() |
|
16 | + { |
|
17 | 17 | |
18 | - $testFile = __DIR__ . '/projectcomposerpackage-test-1.json'; |
|
18 | + $testFile = __DIR__ . '/projectcomposerpackage-test-1.json'; |
|
19 | 19 | |
20 | - $composer = new ProjectComposerPackage($testFile); |
|
20 | + $composer = new ProjectComposerPackage($testFile); |
|
21 | 21 | |
22 | - $config = $composer->getStraussConfig(); |
|
22 | + $config = $composer->getStraussConfig(); |
|
23 | 23 | |
24 | - $this->assertInstanceOf(StraussConfig::class, $config); |
|
25 | - } |
|
24 | + $this->assertInstanceOf(StraussConfig::class, $config); |
|
25 | + } |
|
26 | 26 | } |
@@ -12,19 +12,19 @@ discard block |
||
12 | 12 | class ChangeEnumeratorTest extends TestCase |
13 | 13 | { |
14 | 14 | |
15 | - // PREG_BACKTRACK_LIMIT_ERROR |
|
15 | + // PREG_BACKTRACK_LIMIT_ERROR |
|
16 | 16 | |
17 | - // Single implied global namespace. |
|
18 | - // Single named namespace. |
|
19 | - // Single explicit global namespace. |
|
20 | - // Multiple namespaces. |
|
17 | + // Single implied global namespace. |
|
18 | + // Single named namespace. |
|
19 | + // Single explicit global namespace. |
|
20 | + // Multiple namespaces. |
|
21 | 21 | |
22 | 22 | |
23 | 23 | |
24 | - public function testSingleNamespace() |
|
25 | - { |
|
24 | + public function testSingleNamespace() |
|
25 | + { |
|
26 | 26 | |
27 | - $validPhp = <<<'EOD' |
|
27 | + $validPhp = <<<'EOD' |
|
28 | 28 | <?php |
29 | 29 | namespace MyNamespace; |
30 | 30 | |
@@ -32,22 +32,22 @@ discard block |
||
32 | 32 | } |
33 | 33 | EOD; |
34 | 34 | |
35 | - $config = $this->createMock(StraussConfig::class); |
|
36 | - $config->method('getNamespacePrefix')->willReturn('Prefix'); |
|
37 | - $sut = new ChangeEnumerator($config); |
|
35 | + $config = $this->createMock(StraussConfig::class); |
|
36 | + $config->method('getNamespacePrefix')->willReturn('Prefix'); |
|
37 | + $sut = new ChangeEnumerator($config); |
|
38 | 38 | |
39 | - $sut->find($validPhp); |
|
39 | + $sut->find($validPhp); |
|
40 | 40 | |
41 | - $this->assertArrayHasKey('MyNamespace', $sut->getDiscoveredNamespaceReplacements()); |
|
42 | - $this->assertContains('Prefix\MyNamespace', $sut->getDiscoveredNamespaceReplacements()); |
|
41 | + $this->assertArrayHasKey('MyNamespace', $sut->getDiscoveredNamespaceReplacements()); |
|
42 | + $this->assertContains('Prefix\MyNamespace', $sut->getDiscoveredNamespaceReplacements()); |
|
43 | 43 | |
44 | - $this->assertNotContains('MyClass', $sut->getDiscoveredClasses()); |
|
45 | - } |
|
44 | + $this->assertNotContains('MyClass', $sut->getDiscoveredClasses()); |
|
45 | + } |
|
46 | 46 | |
47 | - public function testGlobalNamespace() |
|
48 | - { |
|
47 | + public function testGlobalNamespace() |
|
48 | + { |
|
49 | 49 | |
50 | - $validPhp = <<<'EOD' |
|
50 | + $validPhp = <<<'EOD' |
|
51 | 51 | <?php |
52 | 52 | namespace { |
53 | 53 | class MyClass { |
@@ -55,21 +55,21 @@ discard block |
||
55 | 55 | } |
56 | 56 | EOD; |
57 | 57 | |
58 | - $config = $this->createMock(StraussConfig::class); |
|
59 | - $sut = new ChangeEnumerator($config); |
|
58 | + $config = $this->createMock(StraussConfig::class); |
|
59 | + $sut = new ChangeEnumerator($config); |
|
60 | 60 | |
61 | - $sut->find($validPhp); |
|
61 | + $sut->find($validPhp); |
|
62 | 62 | |
63 | - $this->assertContains('MyClass', $sut->getDiscoveredClasses()); |
|
64 | - } |
|
63 | + $this->assertContains('MyClass', $sut->getDiscoveredClasses()); |
|
64 | + } |
|
65 | 65 | |
66 | - /** |
|
67 | - * |
|
68 | - */ |
|
69 | - public function testMultipleNamespace() |
|
70 | - { |
|
66 | + /** |
|
67 | + * |
|
68 | + */ |
|
69 | + public function testMultipleNamespace() |
|
70 | + { |
|
71 | 71 | |
72 | - $validPhp = <<<'EOD' |
|
72 | + $validPhp = <<<'EOD' |
|
73 | 73 | <?php |
74 | 74 | namespace MyNamespace { |
75 | 75 | } |
@@ -79,24 +79,24 @@ discard block |
||
79 | 79 | } |
80 | 80 | EOD; |
81 | 81 | |
82 | - $config = $this->createMock(StraussConfig::class); |
|
83 | - $sut = new ChangeEnumerator($config); |
|
82 | + $config = $this->createMock(StraussConfig::class); |
|
83 | + $sut = new ChangeEnumerator($config); |
|
84 | 84 | |
85 | - $sut->find($validPhp); |
|
85 | + $sut->find($validPhp); |
|
86 | 86 | |
87 | - $this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements()); |
|
87 | + $this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements()); |
|
88 | 88 | |
89 | - $this->assertContains('MyClass', $sut->getDiscoveredClasses()); |
|
90 | - } |
|
89 | + $this->assertContains('MyClass', $sut->getDiscoveredClasses()); |
|
90 | + } |
|
91 | 91 | |
92 | 92 | |
93 | - /** |
|
94 | - * |
|
95 | - */ |
|
96 | - public function testMultipleNamespaceGlobalFirst() |
|
97 | - { |
|
93 | + /** |
|
94 | + * |
|
95 | + */ |
|
96 | + public function testMultipleNamespaceGlobalFirst() |
|
97 | + { |
|
98 | 98 | |
99 | - $validPhp = <<<'EOD' |
|
99 | + $validPhp = <<<'EOD' |
|
100 | 100 | <?php |
101 | 101 | |
102 | 102 | namespace { |
@@ -109,25 +109,25 @@ discard block |
||
109 | 109 | } |
110 | 110 | EOD; |
111 | 111 | |
112 | - $config = $this->createMock(StraussConfig::class); |
|
113 | - $sut = new ChangeEnumerator($config); |
|
112 | + $config = $this->createMock(StraussConfig::class); |
|
113 | + $sut = new ChangeEnumerator($config); |
|
114 | 114 | |
115 | - $sut->find($validPhp); |
|
115 | + $sut->find($validPhp); |
|
116 | 116 | |
117 | - $this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements()); |
|
117 | + $this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements()); |
|
118 | 118 | |
119 | - $this->assertContains('MyClass', $sut->getDiscoveredClasses()); |
|
120 | - $this->assertNotContains('MyOtherClass', $sut->getDiscoveredClasses()); |
|
121 | - } |
|
119 | + $this->assertContains('MyClass', $sut->getDiscoveredClasses()); |
|
120 | + $this->assertNotContains('MyOtherClass', $sut->getDiscoveredClasses()); |
|
121 | + } |
|
122 | 122 | |
123 | 123 | |
124 | - /** |
|
125 | - * |
|
126 | - */ |
|
127 | - public function testMultipleClasses() |
|
128 | - { |
|
124 | + /** |
|
125 | + * |
|
126 | + */ |
|
127 | + public function testMultipleClasses() |
|
128 | + { |
|
129 | 129 | |
130 | - $validPhp = <<<'EOD' |
|
130 | + $validPhp = <<<'EOD' |
|
131 | 131 | <?php |
132 | 132 | class MyClass { |
133 | 133 | } |
@@ -136,43 +136,43 @@ discard block |
||
136 | 136 | } |
137 | 137 | EOD; |
138 | 138 | |
139 | - $config = $this->createMock(StraussConfig::class); |
|
140 | - $sut = new ChangeEnumerator($config); |
|
139 | + $config = $this->createMock(StraussConfig::class); |
|
140 | + $sut = new ChangeEnumerator($config); |
|
141 | 141 | |
142 | - $sut->find($validPhp); |
|
142 | + $sut->find($validPhp); |
|
143 | 143 | |
144 | - $this->assertContains('MyClass', $sut->getDiscoveredClasses()); |
|
145 | - $this->assertContains('MyOtherClass', $sut->getDiscoveredClasses()); |
|
146 | - } |
|
144 | + $this->assertContains('MyClass', $sut->getDiscoveredClasses()); |
|
145 | + $this->assertContains('MyOtherClass', $sut->getDiscoveredClasses()); |
|
146 | + } |
|
147 | 147 | |
148 | - /** |
|
149 | - * |
|
150 | - * @author BrianHenryIE |
|
151 | - */ |
|
152 | - public function test_it_does_not_treat_comments_as_classes() |
|
153 | - { |
|
154 | - $contents = " |
|
148 | + /** |
|
149 | + * |
|
150 | + * @author BrianHenryIE |
|
151 | + */ |
|
152 | + public function test_it_does_not_treat_comments_as_classes() |
|
153 | + { |
|
154 | + $contents = " |
|
155 | 155 | // A class as good as any. |
156 | 156 | class Whatever { |
157 | 157 | |
158 | 158 | } |
159 | 159 | "; |
160 | 160 | |
161 | - $config = $this->createMock(StraussConfig::class); |
|
162 | - $changeEnumerator = new ChangeEnumerator($config); |
|
163 | - $changeEnumerator->find($contents); |
|
164 | - |
|
165 | - $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
166 | - $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses()); |
|
167 | - } |
|
168 | - |
|
169 | - /** |
|
170 | - * |
|
171 | - * @author BrianHenryIE |
|
172 | - */ |
|
173 | - public function test_it_does_not_treat_multiline_comments_as_classes() |
|
174 | - { |
|
175 | - $contents = " |
|
161 | + $config = $this->createMock(StraussConfig::class); |
|
162 | + $changeEnumerator = new ChangeEnumerator($config); |
|
163 | + $changeEnumerator->find($contents); |
|
164 | + |
|
165 | + $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
166 | + $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses()); |
|
167 | + } |
|
168 | + |
|
169 | + /** |
|
170 | + * |
|
171 | + * @author BrianHenryIE |
|
172 | + */ |
|
173 | + public function test_it_does_not_treat_multiline_comments_as_classes() |
|
174 | + { |
|
175 | + $contents = " |
|
176 | 176 | /** |
177 | 177 | * A class as good as any; class as. |
178 | 178 | */ |
@@ -180,24 +180,24 @@ discard block |
||
180 | 180 | } |
181 | 181 | "; |
182 | 182 | |
183 | - $config = $this->createMock(StraussConfig::class); |
|
184 | - $changeEnumerator = new ChangeEnumerator($config); |
|
185 | - $changeEnumerator->find($contents); |
|
186 | - |
|
187 | - $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
188 | - $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses()); |
|
189 | - } |
|
190 | - |
|
191 | - /** |
|
192 | - * This worked without adding the expected regex: |
|
193 | - * |
|
194 | - * // \s*\\/?\\*{2,}[^\n]* | # Skip multiline comment bodies |
|
195 | - * |
|
196 | - * @author BrianHenryIE |
|
197 | - */ |
|
198 | - public function test_it_does_not_treat_multiline_comments_opening_line_as_classes() |
|
199 | - { |
|
200 | - $contents = " |
|
183 | + $config = $this->createMock(StraussConfig::class); |
|
184 | + $changeEnumerator = new ChangeEnumerator($config); |
|
185 | + $changeEnumerator->find($contents); |
|
186 | + |
|
187 | + $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
188 | + $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses()); |
|
189 | + } |
|
190 | + |
|
191 | + /** |
|
192 | + * This worked without adding the expected regex: |
|
193 | + * |
|
194 | + * // \s*\\/?\\*{2,}[^\n]* | # Skip multiline comment bodies |
|
195 | + * |
|
196 | + * @author BrianHenryIE |
|
197 | + */ |
|
198 | + public function test_it_does_not_treat_multiline_comments_opening_line_as_classes() |
|
199 | + { |
|
200 | + $contents = " |
|
201 | 201 | /** A class as good as any; class as. |
202 | 202 | * |
203 | 203 | */ |
@@ -205,110 +205,110 @@ discard block |
||
205 | 205 | } |
206 | 206 | "; |
207 | 207 | |
208 | - $config = $this->createMock(StraussConfig::class); |
|
209 | - $changeEnumerator = new ChangeEnumerator($config); |
|
210 | - $changeEnumerator->find($contents); |
|
208 | + $config = $this->createMock(StraussConfig::class); |
|
209 | + $changeEnumerator = new ChangeEnumerator($config); |
|
210 | + $changeEnumerator->find($contents); |
|
211 | 211 | |
212 | - $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
213 | - $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses()); |
|
214 | - } |
|
212 | + $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
213 | + $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses()); |
|
214 | + } |
|
215 | 215 | |
216 | 216 | |
217 | - /** |
|
218 | - * |
|
219 | - * @author BrianHenryIE |
|
220 | - */ |
|
221 | - public function test_it_does_not_treat_multiline_comments_on_one_line_as_classes() |
|
222 | - { |
|
223 | - $contents = " |
|
217 | + /** |
|
218 | + * |
|
219 | + * @author BrianHenryIE |
|
220 | + */ |
|
221 | + public function test_it_does_not_treat_multiline_comments_on_one_line_as_classes() |
|
222 | + { |
|
223 | + $contents = " |
|
224 | 224 | /** A class as good as any; class as. */ class Whatever_Trevor { |
225 | 225 | } |
226 | 226 | "; |
227 | 227 | |
228 | - $config = $this->createMock(StraussConfig::class); |
|
229 | - $changeEnumerator = new ChangeEnumerator($config); |
|
230 | - $changeEnumerator->find($contents); |
|
231 | - |
|
232 | - $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
233 | - $this->assertContains('Whatever_Trevor', $changeEnumerator->getDiscoveredClasses()); |
|
234 | - } |
|
235 | - |
|
236 | - /** |
|
237 | - * If someone were to put a semicolon in the comment it would mess with the previous fix. |
|
238 | - * |
|
239 | - * @author BrianHenryIE |
|
240 | - * |
|
241 | - * @test |
|
242 | - */ |
|
243 | - public function test_it_does_not_treat_comments_with_semicolons_as_classes() |
|
244 | - { |
|
245 | - $contents = " |
|
228 | + $config = $this->createMock(StraussConfig::class); |
|
229 | + $changeEnumerator = new ChangeEnumerator($config); |
|
230 | + $changeEnumerator->find($contents); |
|
231 | + |
|
232 | + $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
233 | + $this->assertContains('Whatever_Trevor', $changeEnumerator->getDiscoveredClasses()); |
|
234 | + } |
|
235 | + |
|
236 | + /** |
|
237 | + * If someone were to put a semicolon in the comment it would mess with the previous fix. |
|
238 | + * |
|
239 | + * @author BrianHenryIE |
|
240 | + * |
|
241 | + * @test |
|
242 | + */ |
|
243 | + public function test_it_does_not_treat_comments_with_semicolons_as_classes() |
|
244 | + { |
|
245 | + $contents = " |
|
246 | 246 | // A class as good as any; class as versatile as any. |
247 | 247 | class Whatever_Ever { |
248 | 248 | |
249 | 249 | } |
250 | 250 | "; |
251 | 251 | |
252 | - $config = $this->createMock(StraussConfig::class); |
|
253 | - $changeEnumerator = new ChangeEnumerator($config); |
|
254 | - $changeEnumerator->find($contents); |
|
252 | + $config = $this->createMock(StraussConfig::class); |
|
253 | + $changeEnumerator = new ChangeEnumerator($config); |
|
254 | + $changeEnumerator->find($contents); |
|
255 | 255 | |
256 | - $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
257 | - $this->assertContains('Whatever_Ever', $changeEnumerator->getDiscoveredClasses()); |
|
258 | - } |
|
256 | + $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses()); |
|
257 | + $this->assertContains('Whatever_Ever', $changeEnumerator->getDiscoveredClasses()); |
|
258 | + } |
|
259 | 259 | |
260 | - /** |
|
261 | - * @author BrianHenryIE |
|
262 | - */ |
|
263 | - public function test_it_parses_classes_after_semicolon() |
|
264 | - { |
|
260 | + /** |
|
261 | + * @author BrianHenryIE |
|
262 | + */ |
|
263 | + public function test_it_parses_classes_after_semicolon() |
|
264 | + { |
|
265 | 265 | |
266 | - $contents = " |
|
266 | + $contents = " |
|
267 | 267 | myvar = 123; class Pear { }; |
268 | 268 | "; |
269 | 269 | |
270 | - $config = $this->createMock(StraussConfig::class); |
|
271 | - $changeEnumerator = new ChangeEnumerator($config); |
|
272 | - $changeEnumerator->find($contents); |
|
270 | + $config = $this->createMock(StraussConfig::class); |
|
271 | + $changeEnumerator = new ChangeEnumerator($config); |
|
272 | + $changeEnumerator->find($contents); |
|
273 | 273 | |
274 | - $this->assertContains('Pear', $changeEnumerator->getDiscoveredClasses()); |
|
275 | - } |
|
274 | + $this->assertContains('Pear', $changeEnumerator->getDiscoveredClasses()); |
|
275 | + } |
|
276 | 276 | |
277 | 277 | |
278 | - /** |
|
279 | - * @author BrianHenryIE |
|
280 | - */ |
|
281 | - public function test_it_parses_classes_followed_by_comment() |
|
282 | - { |
|
278 | + /** |
|
279 | + * @author BrianHenryIE |
|
280 | + */ |
|
281 | + public function test_it_parses_classes_followed_by_comment() |
|
282 | + { |
|
283 | 283 | |
284 | - $contents = <<<'EOD' |
|
284 | + $contents = <<<'EOD' |
|
285 | 285 | class WP_Dependency_Installer { |
286 | 286 | /** |
287 | 287 | * |
288 | 288 | */ |
289 | 289 | EOD; |
290 | 290 | |
291 | - $config = $this->createMock(StraussConfig::class); |
|
292 | - $changeEnumerator = new ChangeEnumerator($config); |
|
293 | - $changeEnumerator->find($contents); |
|
291 | + $config = $this->createMock(StraussConfig::class); |
|
292 | + $changeEnumerator = new ChangeEnumerator($config); |
|
293 | + $changeEnumerator->find($contents); |
|
294 | 294 | |
295 | - $this->assertContains('WP_Dependency_Installer', $changeEnumerator->getDiscoveredClasses()); |
|
296 | - } |
|
295 | + $this->assertContains('WP_Dependency_Installer', $changeEnumerator->getDiscoveredClasses()); |
|
296 | + } |
|
297 | 297 | |
298 | 298 | |
299 | - /** |
|
300 | - * It's possible to have multiple namespaces inside one file. |
|
301 | - * |
|
302 | - * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit. |
|
303 | - * |
|
304 | - * @author BrianHenryIE |
|
305 | - * |
|
306 | - * @test |
|
307 | - */ |
|
308 | - public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_a(): void |
|
309 | - { |
|
299 | + /** |
|
300 | + * It's possible to have multiple namespaces inside one file. |
|
301 | + * |
|
302 | + * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit. |
|
303 | + * |
|
304 | + * @author BrianHenryIE |
|
305 | + * |
|
306 | + * @test |
|
307 | + */ |
|
308 | + public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_a(): void |
|
309 | + { |
|
310 | 310 | |
311 | - $contents = " |
|
311 | + $contents = " |
|
312 | 312 | namespace My_Project { |
313 | 313 | class A_Class { } |
314 | 314 | } |
@@ -317,85 +317,85 @@ discard block |
||
317 | 317 | } |
318 | 318 | "; |
319 | 319 | |
320 | - $config = $this->createMock(StraussConfig::class); |
|
321 | - $changeEnumerator = new ChangeEnumerator($config); |
|
322 | - $changeEnumerator->find($contents); |
|
320 | + $config = $this->createMock(StraussConfig::class); |
|
321 | + $changeEnumerator = new ChangeEnumerator($config); |
|
322 | + $changeEnumerator->find($contents); |
|
323 | 323 | |
324 | - $this->assertNotContains('A_Class', $changeEnumerator->getDiscoveredClasses()); |
|
325 | - $this->assertContains('B_Class', $changeEnumerator->getDiscoveredClasses()); |
|
326 | - } |
|
324 | + $this->assertNotContains('A_Class', $changeEnumerator->getDiscoveredClasses()); |
|
325 | + $this->assertContains('B_Class', $changeEnumerator->getDiscoveredClasses()); |
|
326 | + } |
|
327 | 327 | |
328 | - public function testExcludePackagesFromPrefix() |
|
329 | - { |
|
328 | + public function testExcludePackagesFromPrefix() |
|
329 | + { |
|
330 | 330 | |
331 | - $config = $this->createMock(StraussConfig::class); |
|
332 | - $config->method('getExcludePackagesFromPrefixing')->willReturn( |
|
333 | - array('brianhenryie/pdfhelpers') |
|
334 | - ); |
|
331 | + $config = $this->createMock(StraussConfig::class); |
|
332 | + $config->method('getExcludePackagesFromPrefixing')->willReturn( |
|
333 | + array('brianhenryie/pdfhelpers') |
|
334 | + ); |
|
335 | 335 | |
336 | - $dir = ''; |
|
337 | - $composerPackage = $this->createMock(ComposerPackage::class); |
|
338 | - $composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers'); |
|
339 | - $relativeFilepaths = array( 'irrelevent' => $composerPackage); |
|
336 | + $dir = ''; |
|
337 | + $composerPackage = $this->createMock(ComposerPackage::class); |
|
338 | + $composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers'); |
|
339 | + $relativeFilepaths = array( 'irrelevent' => $composerPackage); |
|
340 | 340 | |
341 | - $changeEnumerator = new ChangeEnumerator($config); |
|
342 | - $changeEnumerator->findInFiles($dir, $relativeFilepaths); |
|
341 | + $changeEnumerator = new ChangeEnumerator($config); |
|
342 | + $changeEnumerator->findInFiles($dir, $relativeFilepaths); |
|
343 | 343 | |
344 | - $this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
345 | - } |
|
344 | + $this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
345 | + } |
|
346 | 346 | |
347 | 347 | |
348 | - public function testExcludeFilePatternsFromPrefix() |
|
349 | - { |
|
350 | - $config = $this->createMock(StraussConfig::class); |
|
351 | - $config->method('getExcludeFilePatternsFromPrefixing')->willReturn( |
|
352 | - array('/to/') |
|
353 | - ); |
|
348 | + public function testExcludeFilePatternsFromPrefix() |
|
349 | + { |
|
350 | + $config = $this->createMock(StraussConfig::class); |
|
351 | + $config->method('getExcludeFilePatternsFromPrefixing')->willReturn( |
|
352 | + array('/to/') |
|
353 | + ); |
|
354 | 354 | |
355 | - $dir = ''; |
|
356 | - $composerPackage = $this->createMock(ComposerPackage::class); |
|
357 | - $composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers'); |
|
358 | - $relativeFilepaths = array( 'path/to/file' => $composerPackage); |
|
355 | + $dir = ''; |
|
356 | + $composerPackage = $this->createMock(ComposerPackage::class); |
|
357 | + $composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers'); |
|
358 | + $relativeFilepaths = array( 'path/to/file' => $composerPackage); |
|
359 | 359 | |
360 | - $changeEnumerator = new ChangeEnumerator($config); |
|
361 | - $changeEnumerator->findInFiles($dir, $relativeFilepaths); |
|
360 | + $changeEnumerator = new ChangeEnumerator($config); |
|
361 | + $changeEnumerator->findInFiles($dir, $relativeFilepaths); |
|
362 | 362 | |
363 | - $this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
364 | - } |
|
363 | + $this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
364 | + } |
|
365 | 365 | |
366 | - /** |
|
367 | - * Test custom replacements |
|
368 | - */ |
|
369 | - public function testNamespaceReplacementPatterns() |
|
370 | - { |
|
366 | + /** |
|
367 | + * Test custom replacements |
|
368 | + */ |
|
369 | + public function testNamespaceReplacementPatterns() |
|
370 | + { |
|
371 | 371 | |
372 | - $contents = " |
|
372 | + $contents = " |
|
373 | 373 | namespace BrianHenryIE\PdfHelpers { |
374 | 374 | class A_Class { } |
375 | 375 | } |
376 | 376 | "; |
377 | 377 | |
378 | - $config = $this->createMock(StraussConfig::class); |
|
379 | - $config->method('getNamespacePrefix')->willReturn('BrianHenryIE\Prefix'); |
|
380 | - $config->method('getNamespaceReplacementPatterns')->willReturn( |
|
381 | - array('/BrianHenryIE\\\\(PdfHelpers)/'=>'BrianHenryIE\\Prefix\\\\$1') |
|
382 | - ); |
|
378 | + $config = $this->createMock(StraussConfig::class); |
|
379 | + $config->method('getNamespacePrefix')->willReturn('BrianHenryIE\Prefix'); |
|
380 | + $config->method('getNamespaceReplacementPatterns')->willReturn( |
|
381 | + array('/BrianHenryIE\\\\(PdfHelpers)/'=>'BrianHenryIE\\Prefix\\\\$1') |
|
382 | + ); |
|
383 | 383 | |
384 | - $changeEnumerator = new ChangeEnumerator($config); |
|
385 | - $changeEnumerator->find($contents); |
|
384 | + $changeEnumerator = new ChangeEnumerator($config); |
|
385 | + $changeEnumerator->find($contents); |
|
386 | 386 | |
387 | - $this->assertArrayHasKey('BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
388 | - $this->assertContains('BrianHenryIE\Prefix\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
389 | - $this->assertNotContains('BrianHenryIE\Prefix\BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
390 | - } |
|
387 | + $this->assertArrayHasKey('BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
388 | + $this->assertContains('BrianHenryIE\Prefix\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
389 | + $this->assertNotContains('BrianHenryIE\Prefix\BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements()); |
|
390 | + } |
|
391 | 391 | |
392 | - /** |
|
393 | - * @see https://github.com/BrianHenryIE/strauss/issues/19 |
|
394 | - */ |
|
395 | - public function testPhraseClassObjectIsNotMistaken() |
|
396 | - { |
|
392 | + /** |
|
393 | + * @see https://github.com/BrianHenryIE/strauss/issues/19 |
|
394 | + */ |
|
395 | + public function testPhraseClassObjectIsNotMistaken() |
|
396 | + { |
|
397 | 397 | |
398 | - $contents = <<<'EOD' |
|
398 | + $contents = <<<'EOD' |
|
399 | 399 | <?php |
400 | 400 | |
401 | 401 | class TCPDF_STATIC |
@@ -419,10 +419,10 @@ discard block |
||
419 | 419 | } |
420 | 420 | EOD; |
421 | 421 | |
422 | - $config = $this->createMock(StraussConfig::class); |
|
423 | - $changeEnumerator = new ChangeEnumerator($config); |
|
424 | - $changeEnumerator->find($contents); |
|
422 | + $config = $this->createMock(StraussConfig::class); |
|
423 | + $changeEnumerator = new ChangeEnumerator($config); |
|
424 | + $changeEnumerator->find($contents); |
|
425 | 425 | |
426 | - $this->assertNotContains('object', $changeEnumerator->getDiscoveredClasses()); |
|
427 | - } |
|
426 | + $this->assertNotContains('object', $changeEnumerator->getDiscoveredClasses()); |
|
427 | + } |
|
428 | 428 | } |
@@ -23,13 +23,13 @@ discard block |
||
23 | 23 | class PrefixerTest extends TestCase |
24 | 24 | { |
25 | 25 | |
26 | - protected StraussConfig $config; |
|
26 | + protected StraussConfig $config; |
|
27 | 27 | |
28 | - protected function aaasetUp(): void |
|
29 | - { |
|
30 | - parent::setUp(); |
|
28 | + protected function aaasetUp(): void |
|
29 | + { |
|
30 | + parent::setUp(); |
|
31 | 31 | |
32 | - $composerJson = <<<'EOD' |
|
32 | + $composerJson = <<<'EOD' |
|
33 | 33 | { |
34 | 34 | "name": "brianhenryie/strauss-replacer-test", |
35 | 35 | "extra": { |
@@ -38,18 +38,18 @@ discard block |
||
38 | 38 | } |
39 | 39 | EOD; |
40 | 40 | |
41 | - $composerConfig = new Config(false); |
|
42 | - $composerConfig->merge(json_decode($composerJson, true)); |
|
43 | - $composer = new Composer(); |
|
44 | - $composer->setConfig($composerConfig); |
|
41 | + $composerConfig = new Config(false); |
|
42 | + $composerConfig->merge(json_decode($composerJson, true)); |
|
43 | + $composer = new Composer(); |
|
44 | + $composer->setConfig($composerConfig); |
|
45 | 45 | |
46 | - $this->config = new StraussConfig($composer); |
|
47 | - } |
|
46 | + $this->config = new StraussConfig($composer); |
|
47 | + } |
|
48 | 48 | |
49 | - public function testNamespaceReplacer() |
|
50 | - { |
|
49 | + public function testNamespaceReplacer() |
|
50 | + { |
|
51 | 51 | |
52 | - $contents = <<<'EOD' |
|
52 | + $contents = <<<'EOD' |
|
53 | 53 | <?php |
54 | 54 | /* |
55 | 55 | * Copyright 2010 Google Inc. |
@@ -122,25 +122,25 @@ discard block |
||
122 | 122 | } |
123 | 123 | } |
124 | 124 | EOD; |
125 | - $config = $this->createMock(StraussConfig::class); |
|
125 | + $config = $this->createMock(StraussConfig::class); |
|
126 | 126 | |
127 | - $replacer = new Prefixer($config, __DIR__); |
|
127 | + $replacer = new Prefixer($config, __DIR__); |
|
128 | 128 | |
129 | - $original = "Google\\Http\\Batch"; |
|
130 | - $replacement = "BrianHenryIE\\Strauss\\Google\\Http\\Batch"; |
|
129 | + $original = "Google\\Http\\Batch"; |
|
130 | + $replacement = "BrianHenryIE\\Strauss\\Google\\Http\\Batch"; |
|
131 | 131 | |
132 | - $result = $replacer->replaceNamespace($contents, $original, $replacement); |
|
132 | + $result = $replacer->replaceNamespace($contents, $original, $replacement); |
|
133 | 133 | |
134 | - $expected = "use BrianHenryIE\\Strauss\\Google\\Http\\Batch;"; |
|
134 | + $expected = "use BrianHenryIE\\Strauss\\Google\\Http\\Batch;"; |
|
135 | 135 | |
136 | - $this->assertStringContainsString($expected, $result); |
|
137 | - } |
|
136 | + $this->assertStringContainsString($expected, $result); |
|
137 | + } |
|
138 | 138 | |
139 | 139 | |
140 | - public function testClassnameReplacer() |
|
141 | - { |
|
140 | + public function testClassnameReplacer() |
|
141 | + { |
|
142 | 142 | |
143 | - $contents = <<<'EOD' |
|
143 | + $contents = <<<'EOD' |
|
144 | 144 | <?php |
145 | 145 | /******************************************************************************* |
146 | 146 | * FPDF * |
@@ -161,263 +161,263 @@ discard block |
||
161 | 161 | } |
162 | 162 | EOD; |
163 | 163 | |
164 | - $config = $this->createMock(StraussConfig::class); |
|
164 | + $config = $this->createMock(StraussConfig::class); |
|
165 | 165 | |
166 | - $replacer = new Prefixer($config, __DIR__); |
|
166 | + $replacer = new Prefixer($config, __DIR__); |
|
167 | 167 | |
168 | - $original = "FPDF"; |
|
169 | - $classnamePrefix = "BrianHenryIE_Strauss_"; |
|
168 | + $original = "FPDF"; |
|
169 | + $classnamePrefix = "BrianHenryIE_Strauss_"; |
|
170 | 170 | |
171 | - $result = $replacer->replaceClassname($contents, $original, $classnamePrefix); |
|
171 | + $result = $replacer->replaceClassname($contents, $original, $classnamePrefix); |
|
172 | 172 | |
173 | - $expected = "class BrianHenryIE_Strauss_FPDF"; |
|
173 | + $expected = "class BrianHenryIE_Strauss_FPDF"; |
|
174 | 174 | |
175 | - $this->assertStringContainsString($expected, $result); |
|
176 | - } |
|
175 | + $this->assertStringContainsString($expected, $result); |
|
176 | + } |
|
177 | 177 | |
178 | - /** |
|
179 | - * PHP 7.4 typed parameters were being prefixed. |
|
180 | - */ |
|
181 | - public function testTypeFunctionParameter() |
|
182 | - { |
|
183 | - $this->markTestIncomplete(); |
|
184 | - } |
|
178 | + /** |
|
179 | + * PHP 7.4 typed parameters were being prefixed. |
|
180 | + */ |
|
181 | + public function testTypeFunctionParameter() |
|
182 | + { |
|
183 | + $this->markTestIncomplete(); |
|
184 | + } |
|
185 | 185 | |
186 | - /** |
|
187 | - * @author CoenJacobs |
|
188 | - */ |
|
189 | - public function test_it_replaces_class_declarations(): void |
|
190 | - { |
|
191 | - $contents = 'class Hello_World {'; |
|
192 | - $originalClassname = 'Hello_World'; |
|
193 | - $classnamePrefix = 'Mozart_'; |
|
186 | + /** |
|
187 | + * @author CoenJacobs |
|
188 | + */ |
|
189 | + public function test_it_replaces_class_declarations(): void |
|
190 | + { |
|
191 | + $contents = 'class Hello_World {'; |
|
192 | + $originalClassname = 'Hello_World'; |
|
193 | + $classnamePrefix = 'Mozart_'; |
|
194 | 194 | |
195 | - $config = $this->createMock(StraussConfig::class); |
|
195 | + $config = $this->createMock(StraussConfig::class); |
|
196 | 196 | |
197 | - $replacer = new Prefixer($config, __DIR__); |
|
197 | + $replacer = new Prefixer($config, __DIR__); |
|
198 | 198 | |
199 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
199 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
200 | 200 | |
201 | - $this->assertEquals('class Mozart_Hello_World {', $result); |
|
202 | - } |
|
201 | + $this->assertEquals('class Mozart_Hello_World {', $result); |
|
202 | + } |
|
203 | 203 | |
204 | - /** |
|
205 | - * @author CoenJacobs |
|
206 | - */ |
|
207 | - public function test_it_replaces_abstract_class_declarations(): void |
|
208 | - { |
|
209 | - $contents = 'abstract class Hello_World {'; |
|
204 | + /** |
|
205 | + * @author CoenJacobs |
|
206 | + */ |
|
207 | + public function test_it_replaces_abstract_class_declarations(): void |
|
208 | + { |
|
209 | + $contents = 'abstract class Hello_World {'; |
|
210 | 210 | |
211 | - $originalClassname = 'Hello_World'; |
|
212 | - $classnamePrefix = 'Mozart_'; |
|
211 | + $originalClassname = 'Hello_World'; |
|
212 | + $classnamePrefix = 'Mozart_'; |
|
213 | 213 | |
214 | - $config = $this->createMock(StraussConfig::class); |
|
214 | + $config = $this->createMock(StraussConfig::class); |
|
215 | 215 | |
216 | - $replacer = new Prefixer($config, __DIR__); |
|
216 | + $replacer = new Prefixer($config, __DIR__); |
|
217 | 217 | |
218 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
218 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
219 | 219 | |
220 | - $this->assertEquals('abstract class Mozart_Hello_World {', $result); |
|
221 | - } |
|
220 | + $this->assertEquals('abstract class Mozart_Hello_World {', $result); |
|
221 | + } |
|
222 | 222 | |
223 | - /** |
|
224 | - * @author CoenJacobs |
|
225 | - */ |
|
226 | - public function test_it_replaces_interface_class_declarations(): void |
|
227 | - { |
|
228 | - $contents = 'interface Hello_World {'; |
|
223 | + /** |
|
224 | + * @author CoenJacobs |
|
225 | + */ |
|
226 | + public function test_it_replaces_interface_class_declarations(): void |
|
227 | + { |
|
228 | + $contents = 'interface Hello_World {'; |
|
229 | 229 | |
230 | - $originalClassname = 'Hello_World'; |
|
231 | - $classnamePrefix = 'Mozart_'; |
|
230 | + $originalClassname = 'Hello_World'; |
|
231 | + $classnamePrefix = 'Mozart_'; |
|
232 | 232 | |
233 | - $config = $this->createMock(StraussConfig::class); |
|
233 | + $config = $this->createMock(StraussConfig::class); |
|
234 | 234 | |
235 | - $replacer = new Prefixer($config, __DIR__); |
|
235 | + $replacer = new Prefixer($config, __DIR__); |
|
236 | 236 | |
237 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
237 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
238 | 238 | |
239 | - $this->assertEquals('interface Mozart_Hello_World {', $result); |
|
240 | - } |
|
239 | + $this->assertEquals('interface Mozart_Hello_World {', $result); |
|
240 | + } |
|
241 | 241 | |
242 | - /** |
|
243 | - * @author CoenJacobs |
|
244 | - */ |
|
245 | - public function test_it_replaces_class_declarations_that_extend_other_classes(): void |
|
246 | - { |
|
247 | - $contents = 'class Hello_World extends Bye_World {'; |
|
242 | + /** |
|
243 | + * @author CoenJacobs |
|
244 | + */ |
|
245 | + public function test_it_replaces_class_declarations_that_extend_other_classes(): void |
|
246 | + { |
|
247 | + $contents = 'class Hello_World extends Bye_World {'; |
|
248 | 248 | |
249 | - $originalClassname = 'Hello_World'; |
|
250 | - $classnamePrefix = 'Mozart_'; |
|
249 | + $originalClassname = 'Hello_World'; |
|
250 | + $classnamePrefix = 'Mozart_'; |
|
251 | 251 | |
252 | - $config = $this->createMock(StraussConfig::class); |
|
252 | + $config = $this->createMock(StraussConfig::class); |
|
253 | 253 | |
254 | - $replacer = new Prefixer($config, __DIR__); |
|
254 | + $replacer = new Prefixer($config, __DIR__); |
|
255 | 255 | |
256 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
256 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
257 | 257 | |
258 | - $this->assertEquals('class Mozart_Hello_World extends Bye_World {', $result); |
|
259 | - } |
|
258 | + $this->assertEquals('class Mozart_Hello_World extends Bye_World {', $result); |
|
259 | + } |
|
260 | 260 | |
261 | - /** |
|
262 | - * @author CoenJacobs |
|
263 | - */ |
|
264 | - public function test_it_replaces_class_declarations_that_implement_interfaces(): void |
|
265 | - { |
|
266 | - $contents = 'class Hello_World implements Bye_World {'; |
|
261 | + /** |
|
262 | + * @author CoenJacobs |
|
263 | + */ |
|
264 | + public function test_it_replaces_class_declarations_that_implement_interfaces(): void |
|
265 | + { |
|
266 | + $contents = 'class Hello_World implements Bye_World {'; |
|
267 | 267 | |
268 | - $originalClassname = 'Hello_World'; |
|
269 | - $classnamePrefix = 'Mozart_'; |
|
268 | + $originalClassname = 'Hello_World'; |
|
269 | + $classnamePrefix = 'Mozart_'; |
|
270 | 270 | |
271 | - $config = $this->createMock(StraussConfig::class); |
|
271 | + $config = $this->createMock(StraussConfig::class); |
|
272 | 272 | |
273 | - $replacer = new Prefixer($config, __DIR__); |
|
273 | + $replacer = new Prefixer($config, __DIR__); |
|
274 | 274 | |
275 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
275 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
276 | 276 | |
277 | - $this->assertEquals('class Mozart_Hello_World implements Bye_World {', $result); |
|
278 | - } |
|
277 | + $this->assertEquals('class Mozart_Hello_World implements Bye_World {', $result); |
|
278 | + } |
|
279 | 279 | |
280 | 280 | |
281 | - /** |
|
282 | - * @author BrianHenryIE |
|
283 | - */ |
|
284 | - public function testItReplacesNamespacesInInterface(): void |
|
285 | - { |
|
286 | - $contents = 'class Hello_World implements \Strauss\Bye_World {'; |
|
281 | + /** |
|
282 | + * @author BrianHenryIE |
|
283 | + */ |
|
284 | + public function testItReplacesNamespacesInInterface(): void |
|
285 | + { |
|
286 | + $contents = 'class Hello_World implements \Strauss\Bye_World {'; |
|
287 | 287 | |
288 | - $originalNamespace = 'Strauss'; |
|
289 | - $replacement = 'Prefix\Strauss'; |
|
288 | + $originalNamespace = 'Strauss'; |
|
289 | + $replacement = 'Prefix\Strauss'; |
|
290 | 290 | |
291 | - $config = $this->createMock(StraussConfig::class); |
|
291 | + $config = $this->createMock(StraussConfig::class); |
|
292 | 292 | |
293 | - $replacer = new Prefixer($config, __DIR__); |
|
293 | + $replacer = new Prefixer($config, __DIR__); |
|
294 | 294 | |
295 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
295 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
296 | 296 | |
297 | - $this->assertEquals('class Hello_World implements \Prefix\Strauss\Bye_World {', $result); |
|
298 | - } |
|
297 | + $this->assertEquals('class Hello_World implements \Prefix\Strauss\Bye_World {', $result); |
|
298 | + } |
|
299 | 299 | |
300 | - /** |
|
301 | - * @author CoenJacobs |
|
302 | - */ |
|
303 | - public function test_it_stores_replaced_class_names(): void |
|
304 | - { |
|
305 | - $this->markTestIncomplete('TODO Delete/move'); |
|
306 | - |
|
307 | - $contents = 'class Hello_World {'; |
|
308 | - $replacer = new Prefixer($config, __DIR__); |
|
309 | - $replacer->setClassmapPrefix('Mozart_'); |
|
310 | - $replacer->replace($contents); |
|
311 | - $this->assertArrayHasKey('Hello_World', $replacer->getReplacedClasses()); |
|
312 | - } |
|
300 | + /** |
|
301 | + * @author CoenJacobs |
|
302 | + */ |
|
303 | + public function test_it_stores_replaced_class_names(): void |
|
304 | + { |
|
305 | + $this->markTestIncomplete('TODO Delete/move'); |
|
313 | 306 | |
314 | - /** |
|
315 | - * @author https://github.com/stephenharris |
|
316 | - * @see https://github.com/coenjacobs/mozart/commit/fd7906943396c9a17110d1bfaf9d778f3b1f322a#diff-87828794e62b55ce8d7263e3ab1a918d1370e283ac750cd44e3ac61db5daee54 |
|
317 | - */ |
|
318 | - public function test_it_replaces_class_declarations_psr2(): void |
|
319 | - { |
|
320 | - $contents = "class Hello_World\n{"; |
|
307 | + $contents = 'class Hello_World {'; |
|
308 | + $replacer = new Prefixer($config, __DIR__); |
|
309 | + $replacer->setClassmapPrefix('Mozart_'); |
|
310 | + $replacer->replace($contents); |
|
311 | + $this->assertArrayHasKey('Hello_World', $replacer->getReplacedClasses()); |
|
312 | + } |
|
321 | 313 | |
322 | - $originalClassname = 'Hello_World'; |
|
323 | - $classnamePrefix = 'Mozart_'; |
|
314 | + /** |
|
315 | + * @author https://github.com/stephenharris |
|
316 | + * @see https://github.com/coenjacobs/mozart/commit/fd7906943396c9a17110d1bfaf9d778f3b1f322a#diff-87828794e62b55ce8d7263e3ab1a918d1370e283ac750cd44e3ac61db5daee54 |
|
317 | + */ |
|
318 | + public function test_it_replaces_class_declarations_psr2(): void |
|
319 | + { |
|
320 | + $contents = "class Hello_World\n{"; |
|
324 | 321 | |
325 | - $config = $this->createMock(StraussConfig::class); |
|
322 | + $originalClassname = 'Hello_World'; |
|
323 | + $classnamePrefix = 'Mozart_'; |
|
326 | 324 | |
327 | - $replacer = new Prefixer($config, __DIR__); |
|
325 | + $config = $this->createMock(StraussConfig::class); |
|
328 | 326 | |
329 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
327 | + $replacer = new Prefixer($config, __DIR__); |
|
330 | 328 | |
331 | - $this->assertEquals("class Mozart_Hello_World\n{", $result); |
|
332 | - } |
|
329 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
333 | 330 | |
334 | - /** |
|
335 | - * @see https://github.com/coenjacobs/mozart/issues/81 |
|
336 | - * @author BrianHenryIE |
|
337 | - * |
|
338 | - */ |
|
339 | - public function test_it_replaces_class(): void |
|
340 | - { |
|
341 | - $contents = "class Hello_World {"; |
|
331 | + $this->assertEquals("class Mozart_Hello_World\n{", $result); |
|
332 | + } |
|
342 | 333 | |
343 | - $originalClassname = 'Hello_World'; |
|
344 | - $classnamePrefix = 'Mozart_'; |
|
334 | + /** |
|
335 | + * @see https://github.com/coenjacobs/mozart/issues/81 |
|
336 | + * @author BrianHenryIE |
|
337 | + * |
|
338 | + */ |
|
339 | + public function test_it_replaces_class(): void |
|
340 | + { |
|
341 | + $contents = "class Hello_World {"; |
|
345 | 342 | |
346 | - $config = $this->createMock(StraussConfig::class); |
|
343 | + $originalClassname = 'Hello_World'; |
|
344 | + $classnamePrefix = 'Mozart_'; |
|
347 | 345 | |
348 | - $replacer = new Prefixer($config, __DIR__); |
|
346 | + $config = $this->createMock(StraussConfig::class); |
|
349 | 347 | |
350 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
348 | + $replacer = new Prefixer($config, __DIR__); |
|
351 | 349 | |
352 | - $this->assertEquals("class Mozart_Hello_World {", $result); |
|
353 | - } |
|
350 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
354 | 351 | |
352 | + $this->assertEquals("class Mozart_Hello_World {", $result); |
|
353 | + } |
|
355 | 354 | |
356 | - /** |
|
357 | - * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
358 | - * @see https://github.com/coenjacobs/mozart/issues/93 |
|
359 | - * |
|
360 | - * @author BrianHenryIE |
|
361 | - * |
|
362 | - * @test |
|
363 | - */ |
|
364 | - public function it_does_not_replace_inside_namespace_multiline(): void |
|
365 | - { |
|
366 | - $contents = " |
|
355 | + |
|
356 | + /** |
|
357 | + * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
358 | + * @see https://github.com/coenjacobs/mozart/issues/93 |
|
359 | + * |
|
360 | + * @author BrianHenryIE |
|
361 | + * |
|
362 | + * @test |
|
363 | + */ |
|
364 | + public function it_does_not_replace_inside_namespace_multiline(): void |
|
365 | + { |
|
366 | + $contents = " |
|
367 | 367 | namespace Mozart; |
368 | 368 | class Hello_World |
369 | 369 | "; |
370 | 370 | |
371 | - $originalClassname = 'Hello_World'; |
|
372 | - $classnamePrefix = 'Mozart_'; |
|
371 | + $originalClassname = 'Hello_World'; |
|
372 | + $classnamePrefix = 'Mozart_'; |
|
373 | 373 | |
374 | - $config = $this->createMock(StraussConfig::class); |
|
375 | - $config->method("getClassmapPrefix")->willReturn($classnamePrefix); |
|
374 | + $config = $this->createMock(StraussConfig::class); |
|
375 | + $config->method("getClassmapPrefix")->willReturn($classnamePrefix); |
|
376 | 376 | |
377 | - $replacer = new Prefixer($config, __DIR__); |
|
377 | + $replacer = new Prefixer($config, __DIR__); |
|
378 | 378 | |
379 | - $result = $replacer->replaceInString([$originalClassname], [], $contents); |
|
379 | + $result = $replacer->replaceInString([$originalClassname], [], $contents); |
|
380 | 380 | |
381 | - $this->assertEquals($contents, $result); |
|
382 | - } |
|
381 | + $this->assertEquals($contents, $result); |
|
382 | + } |
|
383 | 383 | |
384 | - /** |
|
385 | - * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
386 | - * @see https://github.com/coenjacobs/mozart/issues/93 |
|
387 | - * |
|
388 | - * @author BrianHenryIE |
|
389 | - */ |
|
390 | - public function test_it_does_not_replace_inside_namespace_singleline(): void |
|
391 | - { |
|
392 | - $contents = "namespace Mozart; class Hello_World"; |
|
384 | + /** |
|
385 | + * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
386 | + * @see https://github.com/coenjacobs/mozart/issues/93 |
|
387 | + * |
|
388 | + * @author BrianHenryIE |
|
389 | + */ |
|
390 | + public function test_it_does_not_replace_inside_namespace_singleline(): void |
|
391 | + { |
|
392 | + $contents = "namespace Mozart; class Hello_World"; |
|
393 | 393 | |
394 | - $originalClassname = 'Hello_World'; |
|
395 | - $classnamePrefix = 'Mozart_'; |
|
394 | + $originalClassname = 'Hello_World'; |
|
395 | + $classnamePrefix = 'Mozart_'; |
|
396 | 396 | |
397 | - $config = $this->createMock(StraussConfig::class); |
|
397 | + $config = $this->createMock(StraussConfig::class); |
|
398 | 398 | |
399 | - $replacer = new Prefixer($config, __DIR__); |
|
399 | + $replacer = new Prefixer($config, __DIR__); |
|
400 | 400 | |
401 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
401 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
402 | 402 | |
403 | - $this->assertEquals($contents, $result); |
|
404 | - } |
|
403 | + $this->assertEquals($contents, $result); |
|
404 | + } |
|
405 | 405 | |
406 | 406 | |
407 | 407 | |
408 | - /** |
|
409 | - * It's possible to have multiple namespaces inside one file. |
|
410 | - * |
|
411 | - * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit. |
|
412 | - * |
|
413 | - * @author BrianHenryIE |
|
414 | - * |
|
415 | - * @test |
|
416 | - */ |
|
417 | - public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_b(): void |
|
418 | - { |
|
408 | + /** |
|
409 | + * It's possible to have multiple namespaces inside one file. |
|
410 | + * |
|
411 | + * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit. |
|
412 | + * |
|
413 | + * @author BrianHenryIE |
|
414 | + * |
|
415 | + * @test |
|
416 | + */ |
|
417 | + public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_b(): void |
|
418 | + { |
|
419 | 419 | |
420 | - $contents = " |
|
420 | + $contents = " |
|
421 | 421 | namespace My_Project { |
422 | 422 | class A_Class { } |
423 | 423 | } |
@@ -426,389 +426,389 @@ discard block |
||
426 | 426 | } |
427 | 427 | "; |
428 | 428 | |
429 | - $classnamePrefix = 'Mozart_'; |
|
429 | + $classnamePrefix = 'Mozart_'; |
|
430 | 430 | |
431 | - $config = $this->createMock(StraussConfig::class); |
|
431 | + $config = $this->createMock(StraussConfig::class); |
|
432 | 432 | |
433 | - $replacer = new Prefixer($config, __DIR__); |
|
433 | + $replacer = new Prefixer($config, __DIR__); |
|
434 | 434 | |
435 | - $result = $replacer->replaceClassname($contents, 'B_Class', $classnamePrefix); |
|
435 | + $result = $replacer->replaceClassname($contents, 'B_Class', $classnamePrefix); |
|
436 | 436 | |
437 | - $this->assertStringContainsString('Mozart_B_Class', $result); |
|
438 | - } |
|
437 | + $this->assertStringContainsString('Mozart_B_Class', $result); |
|
438 | + } |
|
439 | 439 | |
440 | - /** @test */ |
|
441 | - public function it_replaces_namespace_declarations(): void |
|
442 | - { |
|
443 | - $contents = 'namespace Test\\Test;'; |
|
440 | + /** @test */ |
|
441 | + public function it_replaces_namespace_declarations(): void |
|
442 | + { |
|
443 | + $contents = 'namespace Test\\Test;'; |
|
444 | 444 | |
445 | - $namespace = "Test\\Test"; |
|
446 | - $replacement = "My\\Mozart\\Prefix\\Test\\Test"; |
|
445 | + $namespace = "Test\\Test"; |
|
446 | + $replacement = "My\\Mozart\\Prefix\\Test\\Test"; |
|
447 | 447 | |
448 | - $config = $this->createMock(StraussConfig::class); |
|
448 | + $config = $this->createMock(StraussConfig::class); |
|
449 | 449 | |
450 | - $replacer = new Prefixer($config, __DIR__); |
|
450 | + $replacer = new Prefixer($config, __DIR__); |
|
451 | 451 | |
452 | - $result = $replacer->replaceNamespace($contents, $namespace, $replacement); |
|
452 | + $result = $replacer->replaceNamespace($contents, $namespace, $replacement); |
|
453 | 453 | |
454 | - $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Test;', $result); |
|
455 | - } |
|
454 | + $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Test;', $result); |
|
455 | + } |
|
456 | 456 | |
457 | 457 | |
458 | - /** |
|
459 | - * This test doesn't seem to match its name. |
|
460 | - */ |
|
461 | - public function test_it_doesnt_replaces_namespace_inside_namespace(): void |
|
462 | - { |
|
463 | - $contents = "namespace Test\\Something;\n\nuse Test\\Test;"; |
|
458 | + /** |
|
459 | + * This test doesn't seem to match its name. |
|
460 | + */ |
|
461 | + public function test_it_doesnt_replaces_namespace_inside_namespace(): void |
|
462 | + { |
|
463 | + $contents = "namespace Test\\Something;\n\nuse Test\\Test;"; |
|
464 | 464 | |
465 | - $config = $this->createMock(StraussConfig::class); |
|
465 | + $config = $this->createMock(StraussConfig::class); |
|
466 | 466 | |
467 | - $replacer = new Prefixer($config, __DIR__); |
|
468 | - $result = $replacer->replaceNamespace($contents, "Test\\Something", "My\\Mozart\\Prefix\\Test\\Something"); |
|
469 | - $result = $replacer->replaceNamespace($result, "Test\\Test", "My\\Mozart\\Prefix\\Test\\Test"); |
|
467 | + $replacer = new Prefixer($config, __DIR__); |
|
468 | + $result = $replacer->replaceNamespace($contents, "Test\\Something", "My\\Mozart\\Prefix\\Test\\Something"); |
|
469 | + $result = $replacer->replaceNamespace($result, "Test\\Test", "My\\Mozart\\Prefix\\Test\\Test"); |
|
470 | 470 | |
471 | - $this->assertEquals("namespace My\\Mozart\\Prefix\\Test\\Something;\n\nuse My\\Mozart\\Prefix\\Test\\Test;", $result); |
|
472 | - } |
|
471 | + $this->assertEquals("namespace My\\Mozart\\Prefix\\Test\\Something;\n\nuse My\\Mozart\\Prefix\\Test\\Test;", $result); |
|
472 | + } |
|
473 | 473 | |
474 | - /** |
|
475 | - * |
|
476 | - */ |
|
477 | - public function test_it_does_notreplaces_partial_namespace_declarations(): void |
|
478 | - { |
|
479 | - $contents = 'namespace Test\\Test\\Another;'; |
|
474 | + /** |
|
475 | + * |
|
476 | + */ |
|
477 | + public function test_it_does_notreplaces_partial_namespace_declarations(): void |
|
478 | + { |
|
479 | + $contents = 'namespace Test\\Test\\Another;'; |
|
480 | 480 | |
481 | - $namespace = "Test\\Another"; |
|
482 | - $prefix = "My\\Mozart\\Prefix"; |
|
481 | + $namespace = "Test\\Another"; |
|
482 | + $prefix = "My\\Mozart\\Prefix"; |
|
483 | 483 | |
484 | - $config = $this->createMock(StraussConfig::class); |
|
484 | + $config = $this->createMock(StraussConfig::class); |
|
485 | 485 | |
486 | - $replacer = new Prefixer($config, __DIR__); |
|
487 | - $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
486 | + $replacer = new Prefixer($config, __DIR__); |
|
487 | + $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
488 | 488 | |
489 | - $this->assertEquals('namespace Test\\Test\\Another;', $result); |
|
490 | - } |
|
489 | + $this->assertEquals('namespace Test\\Test\\Another;', $result); |
|
490 | + } |
|
491 | 491 | |
492 | 492 | |
493 | - public function test_it_doesnt_prefix_already_prefixed_namespace(): void |
|
494 | - { |
|
493 | + public function test_it_doesnt_prefix_already_prefixed_namespace(): void |
|
494 | + { |
|
495 | 495 | |
496 | - $contents = 'namespace My\\Mozart\\Prefix\\Test\\Another;'; |
|
496 | + $contents = 'namespace My\\Mozart\\Prefix\\Test\\Another;'; |
|
497 | 497 | |
498 | - $namespace = "Test\\Another"; |
|
499 | - $prefix = "My\\Mozart\\Prefix"; |
|
498 | + $namespace = "Test\\Another"; |
|
499 | + $prefix = "My\\Mozart\\Prefix"; |
|
500 | 500 | |
501 | - $config = $this->createMock(StraussConfig::class); |
|
501 | + $config = $this->createMock(StraussConfig::class); |
|
502 | 502 | |
503 | - $replacer = new Prefixer($config, __DIR__); |
|
504 | - $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
503 | + $replacer = new Prefixer($config, __DIR__); |
|
504 | + $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
505 | 505 | |
506 | - $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Another;', $result); |
|
507 | - } |
|
506 | + $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Another;', $result); |
|
507 | + } |
|
508 | 508 | |
509 | - /** |
|
510 | - * @author markjaquith |
|
511 | - */ |
|
512 | - public function test_it_doesnt_double_replace_namespaces_that_also_exist_inside_another_namespace(): void |
|
513 | - { |
|
509 | + /** |
|
510 | + * @author markjaquith |
|
511 | + */ |
|
512 | + public function test_it_doesnt_double_replace_namespaces_that_also_exist_inside_another_namespace(): void |
|
513 | + { |
|
514 | 514 | |
515 | - // This is a tricky situation. We are referencing Chicken\Egg, |
|
516 | - // but Egg *also* exists as a separate top level class. |
|
517 | - $contents = 'use Chicken\\Egg;'; |
|
518 | - $expected = 'use My\\Mozart\\Prefix\\Chicken\\Egg;'; |
|
515 | + // This is a tricky situation. We are referencing Chicken\Egg, |
|
516 | + // but Egg *also* exists as a separate top level class. |
|
517 | + $contents = 'use Chicken\\Egg;'; |
|
518 | + $expected = 'use My\\Mozart\\Prefix\\Chicken\\Egg;'; |
|
519 | 519 | |
520 | - $config = $this->createMock(StraussConfig::class); |
|
520 | + $config = $this->createMock(StraussConfig::class); |
|
521 | 521 | |
522 | - $replacer = new Prefixer($config, __DIR__); |
|
522 | + $replacer = new Prefixer($config, __DIR__); |
|
523 | 523 | |
524 | - $result = $replacer->replaceNamespace($contents, 'Chicken', 'My\\Mozart\\Prefix\\Chicken'); |
|
525 | - $result = $replacer->replaceNamespace($result, 'Egg', 'My\\Mozart\\Prefix\\Egg'); |
|
524 | + $result = $replacer->replaceNamespace($contents, 'Chicken', 'My\\Mozart\\Prefix\\Chicken'); |
|
525 | + $result = $replacer->replaceNamespace($result, 'Egg', 'My\\Mozart\\Prefix\\Egg'); |
|
526 | 526 | |
527 | - $this->assertEquals($expected, $result); |
|
528 | - } |
|
527 | + $this->assertEquals($expected, $result); |
|
528 | + } |
|
529 | 529 | |
530 | - /** |
|
531 | - * @see https://github.com/coenjacobs/mozart/issues/75 |
|
532 | - * |
|
533 | - * @test |
|
534 | - */ |
|
535 | - public function it_replaces_namespace_use_as_declarations(): void |
|
536 | - { |
|
537 | - $namespace = 'Symfony\Polyfill\Mbstring'; |
|
538 | - $replacement = "MBViews\Dependencies\Symfony\Polyfill\Mbstring"; |
|
530 | + /** |
|
531 | + * @see https://github.com/coenjacobs/mozart/issues/75 |
|
532 | + * |
|
533 | + * @test |
|
534 | + */ |
|
535 | + public function it_replaces_namespace_use_as_declarations(): void |
|
536 | + { |
|
537 | + $namespace = 'Symfony\Polyfill\Mbstring'; |
|
538 | + $replacement = "MBViews\Dependencies\Symfony\Polyfill\Mbstring"; |
|
539 | 539 | |
540 | - $contents = "use Symfony\Polyfill\Mbstring as p;"; |
|
540 | + $contents = "use Symfony\Polyfill\Mbstring as p;"; |
|
541 | 541 | |
542 | - $config = $this->createMock(StraussConfig::class); |
|
542 | + $config = $this->createMock(StraussConfig::class); |
|
543 | 543 | |
544 | - $replacer = new Prefixer($config, __DIR__); |
|
545 | - $result = $replacer->replaceNamespace($contents, $namespace, $replacement); |
|
544 | + $replacer = new Prefixer($config, __DIR__); |
|
545 | + $result = $replacer->replaceNamespace($contents, $namespace, $replacement); |
|
546 | 546 | |
547 | - $expected = "use MBViews\Dependencies\Symfony\Polyfill\Mbstring as p;"; |
|
547 | + $expected = "use MBViews\Dependencies\Symfony\Polyfill\Mbstring as p;"; |
|
548 | 548 | |
549 | - $this->assertEquals($expected, $result); |
|
550 | - } |
|
549 | + $this->assertEquals($expected, $result); |
|
550 | + } |
|
551 | 551 | |
552 | - /** |
|
553 | - * @author BrianHenryIE |
|
554 | - */ |
|
555 | - public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace() |
|
556 | - { |
|
557 | - $namespace = 'Mpdf'; |
|
558 | - $prefix = "Mozart"; |
|
559 | - $contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
|
552 | + /** |
|
553 | + * @author BrianHenryIE |
|
554 | + */ |
|
555 | + public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace() |
|
556 | + { |
|
557 | + $namespace = 'Mpdf'; |
|
558 | + $prefix = "Mozart"; |
|
559 | + $contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
|
560 | 560 | |
561 | - $config = $this->createMock(StraussConfig::class); |
|
561 | + $config = $this->createMock(StraussConfig::class); |
|
562 | 562 | |
563 | - $replacer = new Prefixer($config, __DIR__); |
|
564 | - $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
563 | + $replacer = new Prefixer($config, __DIR__); |
|
564 | + $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
565 | 565 | |
566 | - $expected = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
|
566 | + $expected = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
|
567 | 567 | |
568 | - $this->assertEquals($expected, $result); |
|
569 | - } |
|
568 | + $this->assertEquals($expected, $result); |
|
569 | + } |
|
570 | 570 | |
571 | - public function testLeadingSlashInString() |
|
572 | - { |
|
573 | - $originalNamespace = "Strauss\\Test"; |
|
574 | - $replacement = "Prefix\\Strauss\\Test"; |
|
575 | - $contents = '$mentionedClass = "\\Strauss\\Test\\Classname";'; |
|
571 | + public function testLeadingSlashInString() |
|
572 | + { |
|
573 | + $originalNamespace = "Strauss\\Test"; |
|
574 | + $replacement = "Prefix\\Strauss\\Test"; |
|
575 | + $contents = '$mentionedClass = "\\Strauss\\Test\\Classname";'; |
|
576 | 576 | |
577 | - $config = $this->createMock(StraussConfig::class); |
|
577 | + $config = $this->createMock(StraussConfig::class); |
|
578 | 578 | |
579 | - $replacer = new Prefixer($config, __DIR__); |
|
580 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
579 | + $replacer = new Prefixer($config, __DIR__); |
|
580 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
581 | 581 | |
582 | - $expected = '$mentionedClass = "\\Prefix\\Strauss\\Test\\Classname";'; |
|
582 | + $expected = '$mentionedClass = "\\Prefix\\Strauss\\Test\\Classname";'; |
|
583 | 583 | |
584 | - $this->assertEquals($expected, $result); |
|
585 | - } |
|
584 | + $this->assertEquals($expected, $result); |
|
585 | + } |
|
586 | 586 | |
587 | - public function testDoubleLeadingSlashInString() |
|
588 | - { |
|
589 | - $originalNamespace = "Strauss\\Test"; |
|
590 | - $replacement = "Prefix\\Strauss\\Test"; |
|
591 | - $contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";'; |
|
587 | + public function testDoubleLeadingSlashInString() |
|
588 | + { |
|
589 | + $originalNamespace = "Strauss\\Test"; |
|
590 | + $replacement = "Prefix\\Strauss\\Test"; |
|
591 | + $contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";'; |
|
592 | 592 | |
593 | - $config = $this->createMock(StraussConfig::class); |
|
593 | + $config = $this->createMock(StraussConfig::class); |
|
594 | 594 | |
595 | - $replacer = new Prefixer($config, __DIR__); |
|
596 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
595 | + $replacer = new Prefixer($config, __DIR__); |
|
596 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
597 | 597 | |
598 | - $expected = '$mentionedClass = "\\\\Prefix\\\\Strauss\\\\Test\\\\Classname";'; |
|
598 | + $expected = '$mentionedClass = "\\\\Prefix\\\\Strauss\\\\Test\\\\Classname";'; |
|
599 | 599 | |
600 | - $this->assertEquals($expected, $result); |
|
601 | - } |
|
600 | + $this->assertEquals($expected, $result); |
|
601 | + } |
|
602 | 602 | |
603 | - public function testItReplacesSlashedNamespaceInFunctionParameter() |
|
604 | - { |
|
603 | + public function testItReplacesSlashedNamespaceInFunctionParameter() |
|
604 | + { |
|
605 | 605 | |
606 | - $originalNamespace = "net\\authorize\\api\\contract\\v1"; |
|
607 | - $replacement = "Prefix\\net\\authorize\\api\\contract\\v1"; |
|
608 | - $contents = "public function __construct(\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)"; |
|
606 | + $originalNamespace = "net\\authorize\\api\\contract\\v1"; |
|
607 | + $replacement = "Prefix\\net\\authorize\\api\\contract\\v1"; |
|
608 | + $contents = "public function __construct(\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)"; |
|
609 | 609 | |
610 | - $config = $this->createMock(StraussConfig::class); |
|
610 | + $config = $this->createMock(StraussConfig::class); |
|
611 | 611 | |
612 | - $replacer = new Prefixer($config, __DIR__); |
|
613 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
612 | + $replacer = new Prefixer($config, __DIR__); |
|
613 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
614 | 614 | |
615 | - $expected = "public function __construct(\\Prefix\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)"; |
|
615 | + $expected = "public function __construct(\\Prefix\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)"; |
|
616 | 616 | |
617 | - $this->assertEquals($expected, $result); |
|
618 | - } |
|
617 | + $this->assertEquals($expected, $result); |
|
618 | + } |
|
619 | 619 | |
620 | 620 | |
621 | - public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue() |
|
622 | - { |
|
621 | + public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue() |
|
622 | + { |
|
623 | 623 | |
624 | - $originalNamespace = "net\\authorize\\api\constants"; |
|
625 | - $replacement = "Prefix\\net\\authorize\\api\constants"; |
|
626 | - $contents = "public function executeWithApiResponse(\$endPoint = \\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)"; |
|
624 | + $originalNamespace = "net\\authorize\\api\constants"; |
|
625 | + $replacement = "Prefix\\net\\authorize\\api\constants"; |
|
626 | + $contents = "public function executeWithApiResponse(\$endPoint = \\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)"; |
|
627 | 627 | |
628 | - $config = $this->createMock(StraussConfig::class); |
|
628 | + $config = $this->createMock(StraussConfig::class); |
|
629 | 629 | |
630 | - $replacer = new Prefixer($config, __DIR__); |
|
631 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
630 | + $replacer = new Prefixer($config, __DIR__); |
|
631 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
632 | 632 | |
633 | - $expected = "public function executeWithApiResponse(\$endPoint = \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)"; |
|
633 | + $expected = "public function executeWithApiResponse(\$endPoint = \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)"; |
|
634 | 634 | |
635 | - $this->assertEquals($expected, $result); |
|
636 | - } |
|
635 | + $this->assertEquals($expected, $result); |
|
636 | + } |
|
637 | 637 | |
638 | 638 | |
639 | - public function testItReplacesNamespaceConcatenatedStringConst() |
|
640 | - { |
|
639 | + public function testItReplacesNamespaceConcatenatedStringConst() |
|
640 | + { |
|
641 | 641 | |
642 | - $originalNamespace = "net\\authorize\\api\\constants"; |
|
643 | - $replacement = "Prefix\\net\\authorize\\api\\constants"; |
|
644 | - $contents = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);"; |
|
642 | + $originalNamespace = "net\\authorize\\api\\constants"; |
|
643 | + $replacement = "Prefix\\net\\authorize\\api\\constants"; |
|
644 | + $contents = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);"; |
|
645 | 645 | |
646 | - $config = $this->createMock(StraussConfig::class); |
|
646 | + $config = $this->createMock(StraussConfig::class); |
|
647 | 647 | |
648 | - $replacer = new Prefixer($config, __DIR__); |
|
649 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
648 | + $replacer = new Prefixer($config, __DIR__); |
|
649 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
650 | 650 | |
651 | - $expected = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);"; |
|
651 | + $expected = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);"; |
|
652 | 652 | |
653 | 653 | |
654 | - $this->assertEquals($expected, $result); |
|
655 | - } |
|
654 | + $this->assertEquals($expected, $result); |
|
655 | + } |
|
656 | 656 | |
657 | - /** |
|
658 | - * Another mpdf issue where the class "Mpdf" is in the namespace "Mpdf" and incorrect replacements are being made. |
|
659 | - */ |
|
660 | - public function testClassnameNotConfusedWithNamespace() |
|
661 | - { |
|
657 | + /** |
|
658 | + * Another mpdf issue where the class "Mpdf" is in the namespace "Mpdf" and incorrect replacements are being made. |
|
659 | + */ |
|
660 | + public function testClassnameNotConfusedWithNamespace() |
|
661 | + { |
|
662 | 662 | |
663 | - $contents = '$default_font_size = $mmsize * (Mpdf::SCALE);'; |
|
664 | - $expected = $contents; |
|
663 | + $contents = '$default_font_size = $mmsize * (Mpdf::SCALE);'; |
|
664 | + $expected = $contents; |
|
665 | 665 | |
666 | - $config = $this->createMock(StraussConfig::class); |
|
666 | + $config = $this->createMock(StraussConfig::class); |
|
667 | 667 | |
668 | - $replacer = new Prefixer($config, __DIR__); |
|
669 | - $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf'); |
|
668 | + $replacer = new Prefixer($config, __DIR__); |
|
669 | + $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf'); |
|
670 | 670 | |
671 | - $this->assertEquals($expected, $result); |
|
672 | - } |
|
671 | + $this->assertEquals($expected, $result); |
|
672 | + } |
|
673 | 673 | |
674 | - public function testClassExtendsNamspacedClassIsPrefixed() |
|
675 | - { |
|
674 | + public function testClassExtendsNamspacedClassIsPrefixed() |
|
675 | + { |
|
676 | 676 | |
677 | - $contents = 'class BarcodeException extends \Mpdf\MpdfException'; |
|
678 | - $expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException'; |
|
677 | + $contents = 'class BarcodeException extends \Mpdf\MpdfException'; |
|
678 | + $expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException'; |
|
679 | 679 | |
680 | - $config = $this->createMock(StraussConfig::class); |
|
680 | + $config = $this->createMock(StraussConfig::class); |
|
681 | 681 | |
682 | - $replacer = new Prefixer($config, __DIR__); |
|
683 | - $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf'); |
|
682 | + $replacer = new Prefixer($config, __DIR__); |
|
683 | + $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf'); |
|
684 | 684 | |
685 | - $this->assertEquals($expected, $result); |
|
686 | - } |
|
685 | + $this->assertEquals($expected, $result); |
|
686 | + } |
|
687 | 687 | |
688 | - /** |
|
689 | - * Prefix namespaced classnames after `new` keyword. |
|
690 | - * |
|
691 | - * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
692 | - */ |
|
693 | - public function testNewNamespacedClassIsPrefixed() |
|
694 | - { |
|
688 | + /** |
|
689 | + * Prefix namespaced classnames after `new` keyword. |
|
690 | + * |
|
691 | + * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
692 | + */ |
|
693 | + public function testNewNamespacedClassIsPrefixed() |
|
694 | + { |
|
695 | 695 | |
696 | - $contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );'; |
|
697 | - $expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );'; |
|
696 | + $contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );'; |
|
697 | + $expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );'; |
|
698 | 698 | |
699 | - $config = $this->createMock(StraussConfig::class); |
|
699 | + $config = $this->createMock(StraussConfig::class); |
|
700 | 700 | |
701 | - $replacer = new Prefixer($config, __DIR__); |
|
702 | - $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Provider', 'BrianHenryIE\Strauss\Carbon_Fields\Provider'); |
|
701 | + $replacer = new Prefixer($config, __DIR__); |
|
702 | + $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Provider', 'BrianHenryIE\Strauss\Carbon_Fields\Provider'); |
|
703 | 703 | |
704 | - $this->assertEquals($expected, $result); |
|
705 | - } |
|
704 | + $this->assertEquals($expected, $result); |
|
705 | + } |
|
706 | 706 | |
707 | 707 | |
708 | 708 | |
709 | - /** |
|
710 | - * Prefix namespaced classnames after `static` keyword. |
|
711 | - * |
|
712 | - * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
713 | - */ |
|
714 | - public function testStaticNamespacedClassIsPrefixed() |
|
715 | - { |
|
709 | + /** |
|
710 | + * Prefix namespaced classnames after `static` keyword. |
|
711 | + * |
|
712 | + * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
713 | + */ |
|
714 | + public function testStaticNamespacedClassIsPrefixed() |
|
715 | + { |
|
716 | 716 | |
717 | - $contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container'; |
|
718 | - $expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container'; |
|
717 | + $contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container'; |
|
718 | + $expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container'; |
|
719 | 719 | |
720 | - $config = $this->createMock(StraussConfig::class); |
|
720 | + $config = $this->createMock(StraussConfig::class); |
|
721 | 721 | |
722 | - $replacer = new Prefixer($config, __DIR__); |
|
723 | - $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Container', 'BrianHenryIE\Strauss\Carbon_Fields\Container'); |
|
722 | + $replacer = new Prefixer($config, __DIR__); |
|
723 | + $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Container', 'BrianHenryIE\Strauss\Carbon_Fields\Container'); |
|
724 | 724 | |
725 | - $this->assertEquals($expected, $result); |
|
726 | - } |
|
725 | + $this->assertEquals($expected, $result); |
|
726 | + } |
|
727 | 727 | |
728 | - /** |
|
729 | - * Prefix namespaced classnames after return statement. |
|
730 | - * |
|
731 | - * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
732 | - */ |
|
733 | - public function testReturnedNamespacedClassIsPrefixed() |
|
734 | - { |
|
728 | + /** |
|
729 | + * Prefix namespaced classnames after return statement. |
|
730 | + * |
|
731 | + * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
732 | + */ |
|
733 | + public function testReturnedNamespacedClassIsPrefixed() |
|
734 | + { |
|
735 | 735 | |
736 | - $contents = 'return \Carbon_Fields\Carbon_Fields::resolve'; |
|
737 | - $expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve'; |
|
736 | + $contents = 'return \Carbon_Fields\Carbon_Fields::resolve'; |
|
737 | + $expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve'; |
|
738 | 738 | |
739 | - $config = $this->createMock(StraussConfig::class); |
|
739 | + $config = $this->createMock(StraussConfig::class); |
|
740 | 740 | |
741 | - $replacer = new Prefixer($config, __DIR__); |
|
742 | - $result = $replacer->replaceNamespace($contents, 'Carbon_Fields', 'BrianHenryIE\Strauss\Carbon_Fields'); |
|
741 | + $replacer = new Prefixer($config, __DIR__); |
|
742 | + $result = $replacer->replaceNamespace($contents, 'Carbon_Fields', 'BrianHenryIE\Strauss\Carbon_Fields'); |
|
743 | 743 | |
744 | - $this->assertEquals($expected, $result); |
|
745 | - } |
|
744 | + $this->assertEquals($expected, $result); |
|
745 | + } |
|
746 | 746 | |
747 | - /** |
|
748 | - * Prefix namespaced classnames between two tabs and colon. |
|
749 | - * |
|
750 | - * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
751 | - */ |
|
752 | - public function testNamespacedStaticIsPrefixed() |
|
753 | - { |
|
747 | + /** |
|
748 | + * Prefix namespaced classnames between two tabs and colon. |
|
749 | + * |
|
750 | + * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
751 | + */ |
|
752 | + public function testNamespacedStaticIsPrefixed() |
|
753 | + { |
|
754 | 754 | |
755 | - $contents = " \\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
|
756 | - $expected = " \\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
|
755 | + $contents = " \\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
|
756 | + $expected = " \\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
|
757 | 757 | |
758 | - $config = $this->createMock(StraussConfig::class); |
|
758 | + $config = $this->createMock(StraussConfig::class); |
|
759 | 759 | |
760 | - $replacer = new Prefixer($config, __DIR__); |
|
761 | - $result = $replacer->replaceNamespace( |
|
762 | - $contents, |
|
763 | - 'Carbon_Fields', |
|
764 | - 'BrianHenryIE\Strauss\Carbon_Fields' |
|
765 | - ); |
|
760 | + $replacer = new Prefixer($config, __DIR__); |
|
761 | + $result = $replacer->replaceNamespace( |
|
762 | + $contents, |
|
763 | + 'Carbon_Fields', |
|
764 | + 'BrianHenryIE\Strauss\Carbon_Fields' |
|
765 | + ); |
|
766 | 766 | |
767 | - $this->assertEquals($expected, $result); |
|
768 | - } |
|
767 | + $this->assertEquals($expected, $result); |
|
768 | + } |
|
769 | 769 | |
770 | - /** |
|
771 | - * Sometimes the namespace in a string should be replaced, but sometimes not. |
|
772 | - * |
|
773 | - * @see https://github.com/BrianHenryIE/strauss/issues/15 |
|
774 | - */ |
|
775 | - public function testDoNotReplaceInStringThatIsNotCode() |
|
776 | - { |
|
777 | - $originalNamespace = "TrustedLogin"; |
|
778 | - $replacement = "Prefix\\TrustedLogin"; |
|
779 | - $contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
|
770 | + /** |
|
771 | + * Sometimes the namespace in a string should be replaced, but sometimes not. |
|
772 | + * |
|
773 | + * @see https://github.com/BrianHenryIE/strauss/issues/15 |
|
774 | + */ |
|
775 | + public function testDoNotReplaceInStringThatIsNotCode() |
|
776 | + { |
|
777 | + $originalNamespace = "TrustedLogin"; |
|
778 | + $replacement = "Prefix\\TrustedLogin"; |
|
779 | + $contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
|
780 | 780 | |
781 | - $config = $this->createMock(StraussConfig::class); |
|
781 | + $config = $this->createMock(StraussConfig::class); |
|
782 | 782 | |
783 | - $replacer = new Prefixer($config, __DIR__); |
|
784 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
783 | + $replacer = new Prefixer($config, __DIR__); |
|
784 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
785 | 785 | |
786 | - $expected = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
|
786 | + $expected = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
|
787 | 787 | |
788 | - $this->assertEquals($expected, $result); |
|
789 | - } |
|
788 | + $this->assertEquals($expected, $result); |
|
789 | + } |
|
790 | 790 | |
791 | 791 | |
792 | - /** |
|
793 | - * |
|
794 | - * |
|
795 | - * @see https://github.com/BrianHenryIE/strauss/issues/19 |
|
796 | - * |
|
797 | - */ |
|
798 | - public function testDoNotReplaceInVariableNames() |
|
799 | - { |
|
800 | - $originalClassname = 'object'; |
|
801 | - $classnamePrefix = 'Strauss_Issue19_'; |
|
802 | - $contents = "public static function objclone(\$object) {"; |
|
792 | + /** |
|
793 | + * |
|
794 | + * |
|
795 | + * @see https://github.com/BrianHenryIE/strauss/issues/19 |
|
796 | + * |
|
797 | + */ |
|
798 | + public function testDoNotReplaceInVariableNames() |
|
799 | + { |
|
800 | + $originalClassname = 'object'; |
|
801 | + $classnamePrefix = 'Strauss_Issue19_'; |
|
802 | + $contents = "public static function objclone(\$object) {"; |
|
803 | 803 | |
804 | - $config = $this->createMock(StraussConfig::class); |
|
804 | + $config = $this->createMock(StraussConfig::class); |
|
805 | 805 | |
806 | - $replacer = new Prefixer($config, __DIR__); |
|
807 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
806 | + $replacer = new Prefixer($config, __DIR__); |
|
807 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
808 | 808 | |
809 | - // NOT public static function objclone($Strauss_Issue19_object) { |
|
810 | - $expected = "public static function objclone(\$object) {"; |
|
809 | + // NOT public static function objclone($Strauss_Issue19_object) { |
|
810 | + $expected = "public static function objclone(\$object) {"; |
|
811 | 811 | |
812 | - $this->assertEquals($expected, $result); |
|
813 | - } |
|
812 | + $this->assertEquals($expected, $result); |
|
813 | + } |
|
814 | 814 | } |
@@ -22,80 +22,80 @@ discard block |
||
22 | 22 | { |
23 | 23 | |
24 | 24 | |
25 | - /** |
|
26 | - * @covers ::findLicenseFiles |
|
27 | - * |
|
28 | - */ |
|
29 | - public function testFindLicenceFilesPathsAreRelative() |
|
30 | - { |
|
31 | - $config = $this->createStub(StraussConfig::class); |
|
32 | - $workingDir = __DIR__ . DIRECTORY_SEPARATOR; |
|
25 | + /** |
|
26 | + * @covers ::findLicenseFiles |
|
27 | + * |
|
28 | + */ |
|
29 | + public function testFindLicenceFilesPathsAreRelative() |
|
30 | + { |
|
31 | + $config = $this->createStub(StraussConfig::class); |
|
32 | + $workingDir = __DIR__ . DIRECTORY_SEPARATOR; |
|
33 | 33 | |
34 | - $dependencies = array(); |
|
34 | + $dependencies = array(); |
|
35 | 35 | |
36 | - $dependency = $this->createStub(ComposerPackage::class); |
|
37 | - $dependency->method('getPath')->willReturn('developer-name/project-name/'); |
|
38 | - $dependencies[] = $dependency; |
|
36 | + $dependency = $this->createStub(ComposerPackage::class); |
|
37 | + $dependency->method('getPath')->willReturn('developer-name/project-name/'); |
|
38 | + $dependencies[] = $dependency; |
|
39 | 39 | |
40 | - $sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE'); |
|
40 | + $sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE'); |
|
41 | 41 | |
42 | - $finder = $this->createStub(Finder::class); |
|
42 | + $finder = $this->createStub(Finder::class); |
|
43 | 43 | |
44 | - $file = $this->createStub(\SplFileInfo::class); |
|
45 | - $file->method('getPathname') |
|
46 | - ->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md'); |
|
44 | + $file = $this->createStub(\SplFileInfo::class); |
|
45 | + $file->method('getPathname') |
|
46 | + ->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md'); |
|
47 | 47 | |
48 | - $finderArrayIterator = new ArrayIterator(array( |
|
49 | - $file |
|
50 | - )); |
|
48 | + $finderArrayIterator = new ArrayIterator(array( |
|
49 | + $file |
|
50 | + )); |
|
51 | 51 | |
52 | - $finder->method('getIterator')->willReturn($finderArrayIterator); |
|
52 | + $finder->method('getIterator')->willReturn($finderArrayIterator); |
|
53 | 53 | |
54 | - // Make the rest fluent. |
|
55 | - $callableConstraintNotGetIterator = function ($methodName) { |
|
56 | - return 'getIterator' !== $methodName; |
|
57 | - }; |
|
58 | - $finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder); |
|
54 | + // Make the rest fluent. |
|
55 | + $callableConstraintNotGetIterator = function ($methodName) { |
|
56 | + return 'getIterator' !== $methodName; |
|
57 | + }; |
|
58 | + $finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder); |
|
59 | 59 | |
60 | - $sut->findLicenseFiles($finder); |
|
60 | + $sut->findLicenseFiles($finder); |
|
61 | 61 | |
62 | - $result = $sut->getDiscoveredLicenseFiles(); |
|
62 | + $result = $sut->getDiscoveredLicenseFiles(); |
|
63 | 63 | |
64 | - // Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md |
|
65 | - $this->assertStringContainsString('developer-name/project-name/license.md', $result[0]); |
|
66 | - } |
|
64 | + // Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md |
|
65 | + $this->assertStringContainsString('developer-name/project-name/license.md', $result[0]); |
|
66 | + } |
|
67 | 67 | |
68 | - /** |
|
69 | - * Licence files should be found regardless of case and regardless of British/US-English spelling. |
|
70 | - * |
|
71 | - * @see https://www.phpliveregex.com/p/A5y |
|
72 | - */ |
|
68 | + /** |
|
69 | + * Licence files should be found regardless of case and regardless of British/US-English spelling. |
|
70 | + * |
|
71 | + * @see https://www.phpliveregex.com/p/A5y |
|
72 | + */ |
|
73 | 73 | |
74 | - // https://schibsted.com/blog/mocking-the-file-system-using-phpunit-and-vfsstream/ |
|
74 | + // https://schibsted.com/blog/mocking-the-file-system-using-phpunit-and-vfsstream/ |
|
75 | 75 | |
76 | 76 | |
77 | - /** |
|
78 | - * @see https://github.com/AuthorizeNet/sdk-php/blob/a3e76f96f674d16e892f87c58bedb99dada4b067/lib/net/authorize/api/contract/v1/ANetApiRequestType.php |
|
79 | - * |
|
80 | - * @covers ::addChangeDeclarationToPhpString |
|
81 | - */ |
|
82 | - public function testAppendHeaderCommentInformationNoHeader() |
|
83 | - { |
|
77 | + /** |
|
78 | + * @see https://github.com/AuthorizeNet/sdk-php/blob/a3e76f96f674d16e892f87c58bedb99dada4b067/lib/net/authorize/api/contract/v1/ANetApiRequestType.php |
|
79 | + * |
|
80 | + * @covers ::addChangeDeclarationToPhpString |
|
81 | + */ |
|
82 | + public function testAppendHeaderCommentInformationNoHeader() |
|
83 | + { |
|
84 | 84 | |
85 | - $author = 'BrianHenryIE'; |
|
85 | + $author = 'BrianHenryIE'; |
|
86 | 86 | |
87 | - $config = $this->createStub(StraussConfig::class); |
|
88 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
87 | + $config = $this->createStub(StraussConfig::class); |
|
88 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
89 | 89 | |
90 | - $given = <<<'EOD' |
|
90 | + $given = <<<'EOD' |
|
91 | 91 | <?php |
92 | 92 | |
93 | 93 | namespace net\authorize\api\contract\v1; |
94 | 94 | EOD; |
95 | 95 | |
96 | - // "license": "proprietary", |
|
96 | + // "license": "proprietary", |
|
97 | 97 | |
98 | - $expected = <<<'EOD' |
|
98 | + $expected = <<<'EOD' |
|
99 | 99 | <?php |
100 | 100 | /** |
101 | 101 | * @license proprietary |
@@ -107,22 +107,22 @@ discard block |
||
107 | 107 | namespace net\authorize\api\contract\v1; |
108 | 108 | EOD; |
109 | 109 | |
110 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary'); |
|
110 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary'); |
|
111 | 111 | |
112 | - $this->assertEquals($expected, $actual); |
|
113 | - } |
|
112 | + $this->assertEquals($expected, $actual); |
|
113 | + } |
|
114 | 114 | |
115 | - /** |
|
116 | - * @covers ::addChangeDeclarationToPhpString |
|
117 | - */ |
|
118 | - public function testWithLicenceAlreadyInHeader(): void |
|
119 | - { |
|
115 | + /** |
|
116 | + * @covers ::addChangeDeclarationToPhpString |
|
117 | + */ |
|
118 | + public function testWithLicenceAlreadyInHeader(): void |
|
119 | + { |
|
120 | 120 | |
121 | - $config = $this->createStub(StraussConfig::class); |
|
122 | - $author = 'BrianHenryIE'; |
|
123 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
121 | + $config = $this->createStub(StraussConfig::class); |
|
122 | + $author = 'BrianHenryIE'; |
|
123 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
124 | 124 | |
125 | - $given = <<<'EOD' |
|
125 | + $given = <<<'EOD' |
|
126 | 126 | <?php // phpcs:ignore WordPress.Files.FileName |
127 | 127 | /** |
128 | 128 | * Handles dismissing admin notices. |
@@ -137,7 +137,7 @@ discard block |
||
137 | 137 | namespace Yeah; |
138 | 138 | EOD; |
139 | 139 | |
140 | - $expected = <<<'EOD' |
|
140 | + $expected = <<<'EOD' |
|
141 | 141 | <?php // phpcs:ignore WordPress.Files.FileName |
142 | 142 | /** |
143 | 143 | * Handles dismissing admin notices. |
@@ -155,25 +155,25 @@ discard block |
||
155 | 155 | namespace Yeah; |
156 | 156 | EOD; |
157 | 157 | |
158 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later'); |
|
158 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later'); |
|
159 | 159 | |
160 | - $this->assertEquals($expected, $actual); |
|
161 | - } |
|
160 | + $this->assertEquals($expected, $actual); |
|
161 | + } |
|
162 | 162 | |
163 | 163 | |
164 | - /** |
|
165 | - * Shouldn't matter too much but y'know regexes. |
|
166 | - * |
|
167 | - * @covers ::addChangeDeclarationToPhpString |
|
168 | - */ |
|
169 | - public function testWithTwoCommentsBeforeFirstCode() |
|
170 | - { |
|
164 | + /** |
|
165 | + * Shouldn't matter too much but y'know regexes. |
|
166 | + * |
|
167 | + * @covers ::addChangeDeclarationToPhpString |
|
168 | + */ |
|
169 | + public function testWithTwoCommentsBeforeFirstCode() |
|
170 | + { |
|
171 | 171 | |
172 | - $config = $this->createStub(StraussConfig::class); |
|
173 | - $author = 'BrianHenryIE'; |
|
174 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
172 | + $config = $this->createStub(StraussConfig::class); |
|
173 | + $author = 'BrianHenryIE'; |
|
174 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
175 | 175 | |
176 | - $given = <<<'EOD' |
|
176 | + $given = <<<'EOD' |
|
177 | 177 | <?php |
178 | 178 | /** |
179 | 179 | * WP Dependency Installer |
@@ -194,7 +194,7 @@ discard block |
||
194 | 194 | if ( ! defined( 'WPINC' ) ) { |
195 | 195 | EOD; |
196 | 196 | |
197 | - $expected = <<<'EOD' |
|
197 | + $expected = <<<'EOD' |
|
198 | 198 | <?php |
199 | 199 | /** |
200 | 200 | * WP Dependency Installer |
@@ -219,22 +219,22 @@ discard block |
||
219 | 219 | EOD; |
220 | 220 | |
221 | 221 | |
222 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
222 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
223 | 223 | |
224 | - $this->assertEquals($expected, $actual); |
|
225 | - } |
|
224 | + $this->assertEquals($expected, $actual); |
|
225 | + } |
|
226 | 226 | |
227 | - /** |
|
228 | - * @covers ::addChangeDeclarationToPhpString |
|
229 | - */ |
|
230 | - public function testUnusualHeaderCommentStyle() |
|
231 | - { |
|
227 | + /** |
|
228 | + * @covers ::addChangeDeclarationToPhpString |
|
229 | + */ |
|
230 | + public function testUnusualHeaderCommentStyle() |
|
231 | + { |
|
232 | 232 | |
233 | - $config = $this->createStub(StraussConfig::class); |
|
234 | - $author = 'BrianHenryIE'; |
|
235 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
233 | + $config = $this->createStub(StraussConfig::class); |
|
234 | + $author = 'BrianHenryIE'; |
|
235 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
236 | 236 | |
237 | - $given = <<<'EOD' |
|
237 | + $given = <<<'EOD' |
|
238 | 238 | <?php |
239 | 239 | /******************************************************************************* |
240 | 240 | * FPDF * |
@@ -247,7 +247,7 @@ discard block |
||
247 | 247 | define('FPDF_VERSION','1.82'); |
248 | 248 | EOD; |
249 | 249 | |
250 | - $expected = <<<'EOD' |
|
250 | + $expected = <<<'EOD' |
|
251 | 251 | <?php |
252 | 252 | /******************************************************************************* |
253 | 253 | * FPDF * |
@@ -264,22 +264,22 @@ discard block |
||
264 | 264 | define('FPDF_VERSION','1.82'); |
265 | 265 | EOD; |
266 | 266 | |
267 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary'); |
|
267 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary'); |
|
268 | 268 | |
269 | - $this->assertEquals($expected, $actual); |
|
270 | - } |
|
269 | + $this->assertEquals($expected, $actual); |
|
270 | + } |
|
271 | 271 | |
272 | - /** |
|
273 | - * @covers ::addChangeDeclarationToPhpString |
|
274 | - */ |
|
275 | - public function testCommentWithLicenseWord() |
|
276 | - { |
|
272 | + /** |
|
273 | + * @covers ::addChangeDeclarationToPhpString |
|
274 | + */ |
|
275 | + public function testCommentWithLicenseWord() |
|
276 | + { |
|
277 | 277 | |
278 | - $config = $this->createStub(StraussConfig::class); |
|
279 | - $author = 'BrianHenryIE'; |
|
280 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
278 | + $config = $this->createStub(StraussConfig::class); |
|
279 | + $author = 'BrianHenryIE'; |
|
280 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
281 | 281 | |
282 | - $given = <<<'EOD' |
|
282 | + $given = <<<'EOD' |
|
283 | 283 | <?php |
284 | 284 | |
285 | 285 | /** |
@@ -297,7 +297,7 @@ discard block |
||
297 | 297 | namespace Your_Domain\Assert; |
298 | 298 | EOD; |
299 | 299 | |
300 | - $expected = <<<'EOD' |
|
300 | + $expected = <<<'EOD' |
|
301 | 301 | <?php |
302 | 302 | |
303 | 303 | /** |
@@ -318,27 +318,27 @@ discard block |
||
318 | 318 | namespace Your_Domain\Assert; |
319 | 319 | EOD; |
320 | 320 | |
321 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
321 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
322 | 322 | |
323 | - $this->assertEquals($expected, $actual); |
|
324 | - } |
|
323 | + $this->assertEquals($expected, $actual); |
|
324 | + } |
|
325 | 325 | |
326 | - /** |
|
327 | - * This was matching the "no header comment" regex. |
|
328 | - * |
|
329 | - * FOCK: The test passed. How do I debug when the test passes?! The test is passing but actual output is incorrect. |
|
330 | - * @see https://www.youtube.com/watch?v=QnxpHIl5Ynw |
|
331 | - * |
|
332 | - * Seems files loaded are treated different to strings passed. |
|
333 | - */ |
|
334 | - public function testIncorrectlyMatching() |
|
335 | - { |
|
326 | + /** |
|
327 | + * This was matching the "no header comment" regex. |
|
328 | + * |
|
329 | + * FOCK: The test passed. How do I debug when the test passes?! The test is passing but actual output is incorrect. |
|
330 | + * @see https://www.youtube.com/watch?v=QnxpHIl5Ynw |
|
331 | + * |
|
332 | + * Seems files loaded are treated different to strings passed. |
|
333 | + */ |
|
334 | + public function testIncorrectlyMatching() |
|
335 | + { |
|
336 | 336 | |
337 | - $config = $this->createStub(StraussConfig::class); |
|
338 | - $author = 'BrianHenryIE'; |
|
339 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
337 | + $config = $this->createStub(StraussConfig::class); |
|
338 | + $author = 'BrianHenryIE'; |
|
339 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
340 | 340 | |
341 | - $fileContents = <<<'EOD' |
|
341 | + $fileContents = <<<'EOD' |
|
342 | 342 | <?php |
343 | 343 | /** |
344 | 344 | * WP Dependency Installer |
@@ -361,12 +361,12 @@ discard block |
||
361 | 361 | } |
362 | 362 | EOD; |
363 | 363 | |
364 | - // Attempt to replicate the failing test, since the contents seem the same but the input manner is different. |
|
365 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
366 | - file_put_contents($tmpfname, $fileContents); |
|
367 | - $given = file_get_contents($tmpfname); |
|
364 | + // Attempt to replicate the failing test, since the contents seem the same but the input manner is different. |
|
365 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
366 | + file_put_contents($tmpfname, $fileContents); |
|
367 | + $given = file_get_contents($tmpfname); |
|
368 | 368 | |
369 | - $expected = <<<'EOD' |
|
369 | + $expected = <<<'EOD' |
|
370 | 370 | <?php |
371 | 371 | /** |
372 | 372 | * WP Dependency Installer |
@@ -393,22 +393,22 @@ discard block |
||
393 | 393 | EOD; |
394 | 394 | |
395 | 395 | |
396 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
396 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
397 | 397 | |
398 | - $this->assertEquals($expected, $actual); |
|
399 | - } |
|
398 | + $this->assertEquals($expected, $actual); |
|
399 | + } |
|
400 | 400 | |
401 | - /** |
|
402 | - * The licence was being inserted after every `<?php` in the file. |
|
403 | - */ |
|
404 | - public function testLicenseDetailsOnlyInsertedOncePerFile() |
|
405 | - { |
|
401 | + /** |
|
402 | + * The licence was being inserted after every `<?php` in the file. |
|
403 | + */ |
|
404 | + public function testLicenseDetailsOnlyInsertedOncePerFile() |
|
405 | + { |
|
406 | 406 | |
407 | - $config = $this->createStub(StraussConfig::class); |
|
408 | - $author = 'BrianHenryIE'; |
|
409 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
407 | + $config = $this->createStub(StraussConfig::class); |
|
408 | + $author = 'BrianHenryIE'; |
|
409 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
410 | 410 | |
411 | - $fileContents = <<<'EOD' |
|
411 | + $fileContents = <<<'EOD' |
|
412 | 412 | <?php |
413 | 413 | |
414 | 414 | ?> |
@@ -418,7 +418,7 @@ discard block |
||
418 | 418 | ?> |
419 | 419 | EOD; |
420 | 420 | |
421 | - $expected = <<<'EOD' |
|
421 | + $expected = <<<'EOD' |
|
422 | 422 | <?php |
423 | 423 | /** |
424 | 424 | * @license MIT |
@@ -435,8 +435,8 @@ discard block |
||
435 | 435 | EOD; |
436 | 436 | |
437 | 437 | |
438 | - $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT'); |
|
438 | + $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT'); |
|
439 | 439 | |
440 | - $this->assertEquals($expected, $actual); |
|
441 | - } |
|
440 | + $this->assertEquals($expected, $actual); |
|
441 | + } |
|
442 | 442 | } |
@@ -11,194 +11,194 @@ |
||
11 | 11 | class ComposeTest extends TestCase |
12 | 12 | { |
13 | 13 | |
14 | - /** |
|
15 | - * When composer.json is absent, instead of failing with: |
|
16 | - * "failed to open stream: No such file or directory" |
|
17 | - * a better message should be written to the OutputInterface. |
|
18 | - * |
|
19 | - * @test |
|
20 | - */ |
|
21 | - public function it_fails_gracefully_when_composer_json_absent(): void |
|
22 | - { |
|
23 | - chdir(sys_get_temp_dir()); |
|
24 | - |
|
25 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
26 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
27 | - |
|
28 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
29 | - ->method('write'); |
|
30 | - |
|
31 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
32 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
33 | - { |
|
34 | - parent::__construct(); |
|
35 | - |
|
36 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
37 | - } |
|
38 | - }; |
|
39 | - } |
|
40 | - |
|
41 | - /** |
|
42 | - * When json_decode fails, instead of |
|
43 | - * "Trying to get property 'extra' of non-object" |
|
44 | - * a better message should be written to the OutputInterface. |
|
45 | - * |
|
46 | - * @test |
|
47 | - */ |
|
48 | - public function it_handles_malformed_json_with_grace(): void |
|
49 | - { |
|
50 | - |
|
51 | - $badComposerJson = '{ "name": "coenjacobs/mozart", }'; |
|
52 | - |
|
53 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
54 | - file_put_contents($tmpfname, $badComposerJson); |
|
55 | - chdir(dirname($tmpfname)); |
|
56 | - |
|
57 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
58 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
59 | - |
|
60 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
61 | - ->method('write'); |
|
62 | - |
|
63 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
64 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
65 | - { |
|
66 | - parent::__construct(); |
|
67 | - |
|
68 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
69 | - } |
|
70 | - }; |
|
71 | - } |
|
72 | - |
|
73 | - /** |
|
74 | - * When composer.json->extra is absent, instead of |
|
75 | - * "Undefined property: stdClass::$extra" |
|
76 | - * a better message should be written to the OutputInterface. |
|
77 | - * |
|
78 | - * When package name is not set, `\Composer\Composer::getPackage()->getName()` returns '__root__'. |
|
79 | - * |
|
80 | - */ |
|
81 | - public function test_it_handles_absent_extra_config_with_grace(): void |
|
82 | - { |
|
83 | - |
|
84 | - $badComposerJson = '{ }'; |
|
85 | - |
|
86 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
87 | - file_put_contents($tmpfname, $badComposerJson); |
|
88 | - chdir(dirname($tmpfname)); |
|
89 | - |
|
90 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
91 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
92 | - |
|
93 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
94 | - ->method('write'); |
|
95 | - |
|
96 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
97 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
98 | - { |
|
99 | - parent::__construct(); |
|
100 | - |
|
101 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
102 | - } |
|
103 | - }; |
|
104 | - } |
|
105 | - |
|
106 | - |
|
107 | - /** |
|
108 | - * When composer.json->extra is not an object, instead of |
|
109 | - * "Trying to get property 'mozart' of non-object" |
|
110 | - * a better message should be written to the OutputInterface. |
|
111 | - * |
|
112 | - * @test |
|
113 | - */ |
|
114 | - public function it_handles_malformed_extra_config_with_grace(): void |
|
115 | - { |
|
116 | - |
|
117 | - $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }'; |
|
118 | - |
|
119 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
120 | - file_put_contents($tmpfname, $badComposerJson); |
|
121 | - chdir(dirname($tmpfname)); |
|
122 | - |
|
123 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
124 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
125 | - |
|
126 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
127 | - ->method('write'); |
|
128 | - |
|
129 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
130 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
131 | - { |
|
132 | - parent::__construct(); |
|
133 | - |
|
134 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
135 | - } |
|
136 | - }; |
|
137 | - } |
|
138 | - |
|
139 | - /** |
|
140 | - * When composer.json->extra->mozart is absent, instead of |
|
141 | - * "Undefined property: stdClass::$mozart" |
|
142 | - * a better message should be written to the OutputInterface. |
|
143 | - * |
|
144 | - * @test |
|
145 | - */ |
|
146 | - public function it_handles_absent_mozart_config_with_grace(): void |
|
147 | - { |
|
148 | - |
|
149 | - $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }'; |
|
150 | - |
|
151 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
152 | - file_put_contents($tmpfname, $badComposerJson); |
|
153 | - chdir(dirname($tmpfname)); |
|
154 | - |
|
155 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
156 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
157 | - |
|
158 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
159 | - ->method('write'); |
|
160 | - |
|
161 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
162 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
163 | - { |
|
164 | - parent::__construct(); |
|
165 | - |
|
166 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
167 | - } |
|
168 | - }; |
|
169 | - } |
|
170 | - |
|
171 | - /** |
|
172 | - * When composer.json->extra->mozart is malformed, instead of |
|
173 | - * "Undefined property: stdClass::$mozart" |
|
174 | - * a better message should be written to the OutputInterface. |
|
175 | - * |
|
176 | - * is_object() added. |
|
177 | - * |
|
178 | - * @test |
|
179 | - */ |
|
180 | - public function it_handles_malformed_mozart_config__with_grace(): void |
|
181 | - { |
|
182 | - |
|
183 | - $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": [] }'; |
|
184 | - |
|
185 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
186 | - file_put_contents($tmpfname, $badComposerJson); |
|
187 | - chdir(dirname($tmpfname)); |
|
188 | - |
|
189 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
190 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
191 | - |
|
192 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
193 | - ->method('write'); |
|
194 | - |
|
195 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
196 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
197 | - { |
|
198 | - parent::__construct(); |
|
199 | - |
|
200 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
201 | - } |
|
202 | - }; |
|
203 | - } |
|
14 | + /** |
|
15 | + * When composer.json is absent, instead of failing with: |
|
16 | + * "failed to open stream: No such file or directory" |
|
17 | + * a better message should be written to the OutputInterface. |
|
18 | + * |
|
19 | + * @test |
|
20 | + */ |
|
21 | + public function it_fails_gracefully_when_composer_json_absent(): void |
|
22 | + { |
|
23 | + chdir(sys_get_temp_dir()); |
|
24 | + |
|
25 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
26 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
27 | + |
|
28 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
29 | + ->method('write'); |
|
30 | + |
|
31 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
32 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
33 | + { |
|
34 | + parent::__construct(); |
|
35 | + |
|
36 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
37 | + } |
|
38 | + }; |
|
39 | + } |
|
40 | + |
|
41 | + /** |
|
42 | + * When json_decode fails, instead of |
|
43 | + * "Trying to get property 'extra' of non-object" |
|
44 | + * a better message should be written to the OutputInterface. |
|
45 | + * |
|
46 | + * @test |
|
47 | + */ |
|
48 | + public function it_handles_malformed_json_with_grace(): void |
|
49 | + { |
|
50 | + |
|
51 | + $badComposerJson = '{ "name": "coenjacobs/mozart", }'; |
|
52 | + |
|
53 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
54 | + file_put_contents($tmpfname, $badComposerJson); |
|
55 | + chdir(dirname($tmpfname)); |
|
56 | + |
|
57 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
58 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
59 | + |
|
60 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
61 | + ->method('write'); |
|
62 | + |
|
63 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
64 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
65 | + { |
|
66 | + parent::__construct(); |
|
67 | + |
|
68 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
69 | + } |
|
70 | + }; |
|
71 | + } |
|
72 | + |
|
73 | + /** |
|
74 | + * When composer.json->extra is absent, instead of |
|
75 | + * "Undefined property: stdClass::$extra" |
|
76 | + * a better message should be written to the OutputInterface. |
|
77 | + * |
|
78 | + * When package name is not set, `\Composer\Composer::getPackage()->getName()` returns '__root__'. |
|
79 | + * |
|
80 | + */ |
|
81 | + public function test_it_handles_absent_extra_config_with_grace(): void |
|
82 | + { |
|
83 | + |
|
84 | + $badComposerJson = '{ }'; |
|
85 | + |
|
86 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
87 | + file_put_contents($tmpfname, $badComposerJson); |
|
88 | + chdir(dirname($tmpfname)); |
|
89 | + |
|
90 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
91 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
92 | + |
|
93 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
94 | + ->method('write'); |
|
95 | + |
|
96 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
97 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
98 | + { |
|
99 | + parent::__construct(); |
|
100 | + |
|
101 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
102 | + } |
|
103 | + }; |
|
104 | + } |
|
105 | + |
|
106 | + |
|
107 | + /** |
|
108 | + * When composer.json->extra is not an object, instead of |
|
109 | + * "Trying to get property 'mozart' of non-object" |
|
110 | + * a better message should be written to the OutputInterface. |
|
111 | + * |
|
112 | + * @test |
|
113 | + */ |
|
114 | + public function it_handles_malformed_extra_config_with_grace(): void |
|
115 | + { |
|
116 | + |
|
117 | + $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }'; |
|
118 | + |
|
119 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
120 | + file_put_contents($tmpfname, $badComposerJson); |
|
121 | + chdir(dirname($tmpfname)); |
|
122 | + |
|
123 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
124 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
125 | + |
|
126 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
127 | + ->method('write'); |
|
128 | + |
|
129 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
130 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
131 | + { |
|
132 | + parent::__construct(); |
|
133 | + |
|
134 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
135 | + } |
|
136 | + }; |
|
137 | + } |
|
138 | + |
|
139 | + /** |
|
140 | + * When composer.json->extra->mozart is absent, instead of |
|
141 | + * "Undefined property: stdClass::$mozart" |
|
142 | + * a better message should be written to the OutputInterface. |
|
143 | + * |
|
144 | + * @test |
|
145 | + */ |
|
146 | + public function it_handles_absent_mozart_config_with_grace(): void |
|
147 | + { |
|
148 | + |
|
149 | + $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }'; |
|
150 | + |
|
151 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
152 | + file_put_contents($tmpfname, $badComposerJson); |
|
153 | + chdir(dirname($tmpfname)); |
|
154 | + |
|
155 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
156 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
157 | + |
|
158 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
159 | + ->method('write'); |
|
160 | + |
|
161 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
162 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
163 | + { |
|
164 | + parent::__construct(); |
|
165 | + |
|
166 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
167 | + } |
|
168 | + }; |
|
169 | + } |
|
170 | + |
|
171 | + /** |
|
172 | + * When composer.json->extra->mozart is malformed, instead of |
|
173 | + * "Undefined property: stdClass::$mozart" |
|
174 | + * a better message should be written to the OutputInterface. |
|
175 | + * |
|
176 | + * is_object() added. |
|
177 | + * |
|
178 | + * @test |
|
179 | + */ |
|
180 | + public function it_handles_malformed_mozart_config__with_grace(): void |
|
181 | + { |
|
182 | + |
|
183 | + $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": [] }'; |
|
184 | + |
|
185 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
186 | + file_put_contents($tmpfname, $badComposerJson); |
|
187 | + chdir(dirname($tmpfname)); |
|
188 | + |
|
189 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
190 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
191 | + |
|
192 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
193 | + ->method('write'); |
|
194 | + |
|
195 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
196 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
197 | + { |
|
198 | + parent::__construct(); |
|
199 | + |
|
200 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
201 | + } |
|
202 | + }; |
|
203 | + } |
|
204 | 204 | } |
@@ -9,22 +9,22 @@ |
||
9 | 9 | class ApplicationTest extends TestCase |
10 | 10 | { |
11 | 11 | |
12 | - /** |
|
13 | - * Test the Symfony\Component\Console\Application instance contains the Compose command. |
|
14 | - */ |
|
15 | - public function testInstantiation() |
|
16 | - { |
|
12 | + /** |
|
13 | + * Test the Symfony\Component\Console\Application instance contains the Compose command. |
|
14 | + */ |
|
15 | + public function testInstantiation() |
|
16 | + { |
|
17 | 17 | |
18 | - $version = '1.0.0'; |
|
18 | + $version = '1.0.0'; |
|
19 | 19 | |
20 | - $sut = new Application($version); |
|
20 | + $sut = new Application($version); |
|
21 | 21 | |
22 | - $commands = $sut->all(); |
|
22 | + $commands = $sut->all(); |
|
23 | 23 | |
24 | - $containsComposeCommand = array_reduce($commands, function ($carry, $item) { |
|
25 | - return $carry || $item instanceof Compose; |
|
26 | - }, false); |
|
24 | + $containsComposeCommand = array_reduce($commands, function ($carry, $item) { |
|
25 | + return $carry || $item instanceof Compose; |
|
26 | + }, false); |
|
27 | 27 | |
28 | - $this->assertTrue($containsComposeCommand); |
|
29 | - } |
|
28 | + $this->assertTrue($containsComposeCommand); |
|
29 | + } |
|
30 | 30 | } |
@@ -18,54 +18,54 @@ |
||
18 | 18 | */ |
19 | 19 | class IntegrationTestCase extends TestCase |
20 | 20 | { |
21 | - protected $testsWorkingDir; |
|
21 | + protected $testsWorkingDir; |
|
22 | 22 | |
23 | - public function setUp(): void |
|
24 | - { |
|
25 | - parent::setUp(); |
|
23 | + public function setUp(): void |
|
24 | + { |
|
25 | + parent::setUp(); |
|
26 | 26 | |
27 | - $this->testsWorkingDir = rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR |
|
28 | - . 'strausstestdir' . DIRECTORY_SEPARATOR; |
|
27 | + $this->testsWorkingDir = rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR |
|
28 | + . 'strausstestdir' . DIRECTORY_SEPARATOR; |
|
29 | 29 | |
30 | - if (file_exists($this->testsWorkingDir)) { |
|
31 | - $this->deleteDir($this->testsWorkingDir); |
|
32 | - } |
|
30 | + if (file_exists($this->testsWorkingDir)) { |
|
31 | + $this->deleteDir($this->testsWorkingDir); |
|
32 | + } |
|
33 | 33 | |
34 | - @mkdir($this->testsWorkingDir); |
|
35 | - } |
|
34 | + @mkdir($this->testsWorkingDir); |
|
35 | + } |
|
36 | 36 | |
37 | 37 | |
38 | - /** |
|
39 | - * Delete $this->testsWorkingDir after each test. |
|
40 | - * |
|
41 | - * @see https://stackoverflow.com/questions/3349753/delete-directory-with-files-in-it |
|
42 | - */ |
|
43 | - public function tearDown(): void |
|
44 | - { |
|
45 | - parent::tearDown(); |
|
38 | + /** |
|
39 | + * Delete $this->testsWorkingDir after each test. |
|
40 | + * |
|
41 | + * @see https://stackoverflow.com/questions/3349753/delete-directory-with-files-in-it |
|
42 | + */ |
|
43 | + public function tearDown(): void |
|
44 | + { |
|
45 | + parent::tearDown(); |
|
46 | 46 | |
47 | - $dir = $this->testsWorkingDir; |
|
47 | + $dir = $this->testsWorkingDir; |
|
48 | 48 | |
49 | - $this->deleteDir($dir); |
|
50 | - } |
|
49 | + $this->deleteDir($dir); |
|
50 | + } |
|
51 | 51 | |
52 | - protected function deleteDir($dir) |
|
53 | - { |
|
54 | - if (!file_exists($dir)) { |
|
55 | - return; |
|
56 | - } |
|
52 | + protected function deleteDir($dir) |
|
53 | + { |
|
54 | + if (!file_exists($dir)) { |
|
55 | + return; |
|
56 | + } |
|
57 | 57 | |
58 | - $it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS); |
|
59 | - $files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST); |
|
60 | - foreach ($files as $file) { |
|
61 | - if (is_link($file)) { |
|
62 | - unlink($file); |
|
63 | - } elseif ($file->isDir()) { |
|
64 | - rmdir($file->getRealPath()); |
|
65 | - } else { |
|
66 | - unlink($file->getRealPath()); |
|
67 | - } |
|
68 | - } |
|
69 | - rmdir($dir); |
|
70 | - } |
|
58 | + $it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS); |
|
59 | + $files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST); |
|
60 | + foreach ($files as $file) { |
|
61 | + if (is_link($file)) { |
|
62 | + unlink($file); |
|
63 | + } elseif ($file->isDir()) { |
|
64 | + rmdir($file->getRealPath()); |
|
65 | + } else { |
|
66 | + unlink($file->getRealPath()); |
|
67 | + } |
|
68 | + } |
|
69 | + rmdir($dir); |
|
70 | + } |
|
71 | 71 | } |