@@ -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 | } |
@@ -18,10 +18,10 @@ discard block |
||
18 | 18 | class CopierIntegrationTest extends IntegrationTestCase |
19 | 19 | { |
20 | 20 | |
21 | - public function testsPrepareTarget() |
|
22 | - { |
|
21 | + public function testsPrepareTarget() |
|
22 | + { |
|
23 | 23 | |
24 | - $composerJsonString = <<<'EOD' |
|
24 | + $composerJsonString = <<<'EOD' |
|
25 | 25 | { |
26 | 26 | "name": "brianhenryie/strauss", |
27 | 27 | "require": { |
@@ -37,50 +37,50 @@ discard block |
||
37 | 37 | } |
38 | 38 | EOD; |
39 | 39 | |
40 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
40 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
41 | 41 | |
42 | - chdir($this->testsWorkingDir); |
|
42 | + chdir($this->testsWorkingDir); |
|
43 | 43 | |
44 | - exec('composer install'); |
|
44 | + exec('composer install'); |
|
45 | 45 | |
46 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
46 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
47 | 47 | |
48 | - $dependencies = array_map(function ($element) { |
|
49 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
50 | - return new ComposerPackage($dir); |
|
51 | - }, $projectComposerPackage->getRequiresNames()); |
|
48 | + $dependencies = array_map(function ($element) { |
|
49 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
50 | + return new ComposerPackage($dir); |
|
51 | + }, $projectComposerPackage->getRequiresNames()); |
|
52 | 52 | |
53 | - $workingDir = $this->testsWorkingDir; |
|
54 | - $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
53 | + $workingDir = $this->testsWorkingDir; |
|
54 | + $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
55 | 55 | |
56 | - $config = $this->createStub(StraussConfig::class); |
|
56 | + $config = $this->createStub(StraussConfig::class); |
|
57 | 57 | |
58 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
59 | - $fileEnumerator->compileFileList(); |
|
60 | - $filepaths = $fileEnumerator->getAllFilesAndDependencyList(); |
|
58 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
59 | + $fileEnumerator->compileFileList(); |
|
60 | + $filepaths = $fileEnumerator->getAllFilesAndDependencyList(); |
|
61 | 61 | |
62 | - $copier = new Copier($filepaths, $workingDir, $relativeTargetDir); |
|
62 | + $copier = new Copier($filepaths, $workingDir, $relativeTargetDir); |
|
63 | 63 | |
64 | - $file = 'ContainerAwareTrait.php'; |
|
65 | - $relativePath = 'league/container/src/'; |
|
66 | - $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath; |
|
67 | - $targetFile = $targetPath . $file; |
|
64 | + $file = 'ContainerAwareTrait.php'; |
|
65 | + $relativePath = 'league/container/src/'; |
|
66 | + $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath; |
|
67 | + $targetFile = $targetPath . $file; |
|
68 | 68 | |
69 | - mkdir(rtrim($targetPath, DIRECTORY_SEPARATOR), 0777, true); |
|
69 | + mkdir(rtrim($targetPath, DIRECTORY_SEPARATOR), 0777, true); |
|
70 | 70 | |
71 | - file_put_contents($targetFile, 'dummy file'); |
|
71 | + file_put_contents($targetFile, 'dummy file'); |
|
72 | 72 | |
73 | - assert(file_exists($targetFile)); |
|
73 | + assert(file_exists($targetFile)); |
|
74 | 74 | |
75 | - $copier->prepareTarget(); |
|
75 | + $copier->prepareTarget(); |
|
76 | 76 | |
77 | - $this->assertFileDoesNotExist($targetFile); |
|
78 | - } |
|
77 | + $this->assertFileDoesNotExist($targetFile); |
|
78 | + } |
|
79 | 79 | |
80 | - public function testsCopy() |
|
81 | - { |
|
80 | + public function testsCopy() |
|
81 | + { |
|
82 | 82 | |
83 | - $composerJsonString = <<<'EOD' |
|
83 | + $composerJsonString = <<<'EOD' |
|
84 | 84 | { |
85 | 85 | "name": "brianhenryie/strauss", |
86 | 86 | "require": { |
@@ -96,183 +96,183 @@ discard block |
||
96 | 96 | } |
97 | 97 | EOD; |
98 | 98 | |
99 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
99 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
100 | 100 | |
101 | - chdir($this->testsWorkingDir); |
|
101 | + chdir($this->testsWorkingDir); |
|
102 | 102 | |
103 | - exec('composer install'); |
|
103 | + exec('composer install'); |
|
104 | 104 | |
105 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
105 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
106 | 106 | |
107 | - $dependencies = array_map(function ($element) { |
|
108 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
109 | - return new ComposerPackage($dir); |
|
110 | - }, $projectComposerPackage->getRequiresNames()); |
|
107 | + $dependencies = array_map(function ($element) { |
|
108 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
109 | + return new ComposerPackage($dir); |
|
110 | + }, $projectComposerPackage->getRequiresNames()); |
|
111 | 111 | |
112 | - $workingDir = $this->testsWorkingDir; |
|
113 | - $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
114 | - $config = $this->createStub(StraussConfig::class); |
|
115 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
116 | - $fileEnumerator->compileFileList(); |
|
117 | - $filepaths = $fileEnumerator->getAllFilesAndDependencyList(); |
|
112 | + $workingDir = $this->testsWorkingDir; |
|
113 | + $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
114 | + $config = $this->createStub(StraussConfig::class); |
|
115 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
116 | + $fileEnumerator->compileFileList(); |
|
117 | + $filepaths = $fileEnumerator->getAllFilesAndDependencyList(); |
|
118 | 118 | |
119 | - $copier = new Copier($filepaths, $workingDir, $relativeTargetDir); |
|
119 | + $copier = new Copier($filepaths, $workingDir, $relativeTargetDir); |
|
120 | 120 | |
121 | - $file = 'Client.php'; |
|
122 | - $relativePath = 'google/apiclient/src/'; |
|
123 | - $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath; |
|
124 | - $targetFile = $targetPath . $file; |
|
121 | + $file = 'Client.php'; |
|
122 | + $relativePath = 'google/apiclient/src/'; |
|
123 | + $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath; |
|
124 | + $targetFile = $targetPath . $file; |
|
125 | 125 | |
126 | - $copier->prepareTarget(); |
|
126 | + $copier->prepareTarget(); |
|
127 | 127 | |
128 | - $copier->copy(); |
|
128 | + $copier->copy(); |
|
129 | 129 | |
130 | - $this->assertFileExists($targetFile); |
|
131 | - } |
|
130 | + $this->assertFileExists($targetFile); |
|
131 | + } |
|
132 | 132 | |
133 | 133 | |
134 | 134 | |
135 | 135 | |
136 | - /** |
|
137 | - * Set up a common settings object. |
|
138 | - * @see MoverTest.php |
|
139 | - */ |
|
140 | - protected function createComposer(): void |
|
141 | - { |
|
136 | + /** |
|
137 | + * Set up a common settings object. |
|
138 | + * @see MoverTest.php |
|
139 | + */ |
|
140 | + protected function createComposer(): void |
|
141 | + { |
|
142 | 142 | // parent::setUp(); |
143 | 143 | |
144 | - $this->testsWorkingDir = __DIR__ . '/temptestdir/'; |
|
145 | - if (!file_exists($this->testsWorkingDir)) { |
|
146 | - mkdir($this->testsWorkingDir); |
|
147 | - } |
|
148 | - |
|
149 | - $mozartConfig = new stdClass(); |
|
150 | - $mozartConfig->dep_directory = "/dep_directory/"; |
|
151 | - $mozartConfig->classmap_directory = "/classmap_directory/"; |
|
152 | - $mozartConfig->packages = array( |
|
153 | - "pimple/pimple", |
|
154 | - "ezyang/htmlpurifier" |
|
155 | - ); |
|
156 | - |
|
157 | - $pimpleAutoload = new stdClass(); |
|
158 | - $pimpleAutoload->{'psr-0'} = new stdClass(); |
|
159 | - $pimpleAutoload->{'psr-0'}->Pimple = "src/"; |
|
160 | - |
|
161 | - $htmlpurifierAutoload = new stdClass(); |
|
162 | - $htmlpurifierAutoload->classmap = new stdClass(); |
|
163 | - $htmlpurifierAutoload->classmap->Pimple = "library/"; |
|
164 | - |
|
165 | - $mozartConfig->override_autoload = array(); |
|
166 | - $mozartConfig->override_autoload["pimple/pimple"] = $pimpleAutoload; |
|
167 | - $mozartConfig->override_autoload["ezyang/htmlpurifier"] = $htmlpurifierAutoload; |
|
168 | - |
|
169 | - $composer = new stdClass(); |
|
170 | - $composer->extra = new stdClass(); |
|
171 | - $composer->extra->mozart = $mozartConfig; |
|
172 | - |
|
173 | - $composerFilepath = $this->testsWorkingDir . '/composer.json'; |
|
174 | - $composerJson = json_encode($composer) ; |
|
175 | - file_put_contents($composerFilepath, $composerJson); |
|
176 | - |
|
177 | - $this->config = StraussConfig::loadFromFile($composerFilepath); |
|
178 | - } |
|
179 | - |
|
180 | - /** |
|
181 | - * If the specified `dep_directory` or `classmap_directory` are absent, create them. |
|
182 | - * @see MoverTest.php |
|
183 | - * @test |
|
184 | - */ |
|
185 | - public function it_creates_absent_dirs(): void |
|
186 | - { |
|
187 | - $this->markTestIncomplete(); |
|
188 | - |
|
189 | - $mover = new Mover($this->testsWorkingDir, $this->config); |
|
190 | - |
|
191 | - // Make sure the directories don't exist. |
|
192 | - assert(! file_exists($this->testsWorkingDir . $this->config->gett()), "{$this->testsWorkingDir}{$this->config->getDepDirectory()} already exists"); |
|
193 | - assert(! file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())); |
|
194 | - |
|
195 | - $packages = array(); |
|
196 | - |
|
197 | - $mover->deleteTargetDirs($packages); |
|
198 | - |
|
199 | - $this->assertTrue(file_exists($this->testsWorkingDir |
|
200 | - . $this->config->getDepDirectory())); |
|
201 | - $this->assertTrue(file_exists($this->testsWorkingDir |
|
202 | - . $this->config->getClassmapDirectory())); |
|
203 | - } |
|
204 | - |
|
205 | - /** |
|
206 | - * If the specified `dep_directory` or `classmap_directory` already exists with contents, it is not an issue. |
|
207 | - * |
|
208 | - * @see MoverTest.php |
|
209 | - * |
|
210 | - * @test |
|
211 | - */ |
|
212 | - public function it_is_unpertrubed_by_existing_dirs(): void |
|
213 | - { |
|
214 | - $this->markTestIncomplete(); |
|
215 | - |
|
216 | - $mover = new Mover($this->testsWorkingDir, $this->config); |
|
217 | - |
|
218 | - if (!file_exists($this->testsWorkingDir . $this->config->getDepDirectory())) { |
|
219 | - mkdir($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
220 | - } |
|
221 | - if (!file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())) { |
|
222 | - mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
223 | - } |
|
224 | - |
|
225 | - $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
226 | - $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
227 | - |
|
228 | - $packages = array(); |
|
229 | - |
|
230 | - ob_start(); |
|
231 | - |
|
232 | - $mover->deleteTargetDirs($packages); |
|
233 | - |
|
234 | - $output = ob_get_clean(); |
|
235 | - |
|
236 | - $this->assertEmpty($output); |
|
237 | - } |
|
238 | - |
|
239 | - /** |
|
240 | - * If the specified `dep_directory` or `classmap_directory` contains a subdir we are going to need when moving, |
|
241 | - * delete the subdir. aka: If subfolders exist for dependencies we are about to manage, delete those subfolders. |
|
242 | - * |
|
243 | - * @see MoverTest.php |
|
244 | - * |
|
245 | - * @test |
|
246 | - */ |
|
247 | - public function it_deletes_subdirs_for_packages_about_to_be_moved(): void |
|
248 | - { |
|
249 | - $this->markTestIncomplete(); |
|
250 | - |
|
251 | - $mover = new Mover($this->testsWorkingDir, $this->config); |
|
252 | - |
|
253 | - @mkdir($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
254 | - @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
255 | - |
|
256 | - @mkdir($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple'); |
|
257 | - @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory() . 'ezyang'); |
|
258 | - |
|
259 | - $packages = array(); |
|
260 | - foreach ($this->config->getPackages() as $packageString) { |
|
261 | - $testDummyComposerDir = $this->testsWorkingDir . 'vendor' |
|
262 | - . DIRECTORY_SEPARATOR . $packageString; |
|
263 | - @mkdir($testDummyComposerDir, 0777, true); |
|
264 | - $testDummyComposerPath = $testDummyComposerDir . DIRECTORY_SEPARATOR . 'composer.json'; |
|
265 | - $testDummyComposerContents = json_encode(new stdClass()); |
|
266 | - |
|
267 | - file_put_contents($testDummyComposerPath, $testDummyComposerContents); |
|
268 | - $parsedPackage = new ComposerPackageConfig($testDummyComposerDir, $this->config->getOverrideAutoload()[$packageString]); |
|
269 | - $parsedPackage->findAutoloaders(); |
|
270 | - $packages[] = $parsedPackage; |
|
271 | - } |
|
272 | - |
|
273 | - $mover->deleteTargetDirs($packages); |
|
274 | - |
|
275 | - $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple'); |
|
276 | - $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'ezyang'); |
|
277 | - } |
|
144 | + $this->testsWorkingDir = __DIR__ . '/temptestdir/'; |
|
145 | + if (!file_exists($this->testsWorkingDir)) { |
|
146 | + mkdir($this->testsWorkingDir); |
|
147 | + } |
|
148 | + |
|
149 | + $mozartConfig = new stdClass(); |
|
150 | + $mozartConfig->dep_directory = "/dep_directory/"; |
|
151 | + $mozartConfig->classmap_directory = "/classmap_directory/"; |
|
152 | + $mozartConfig->packages = array( |
|
153 | + "pimple/pimple", |
|
154 | + "ezyang/htmlpurifier" |
|
155 | + ); |
|
156 | + |
|
157 | + $pimpleAutoload = new stdClass(); |
|
158 | + $pimpleAutoload->{'psr-0'} = new stdClass(); |
|
159 | + $pimpleAutoload->{'psr-0'}->Pimple = "src/"; |
|
160 | + |
|
161 | + $htmlpurifierAutoload = new stdClass(); |
|
162 | + $htmlpurifierAutoload->classmap = new stdClass(); |
|
163 | + $htmlpurifierAutoload->classmap->Pimple = "library/"; |
|
164 | + |
|
165 | + $mozartConfig->override_autoload = array(); |
|
166 | + $mozartConfig->override_autoload["pimple/pimple"] = $pimpleAutoload; |
|
167 | + $mozartConfig->override_autoload["ezyang/htmlpurifier"] = $htmlpurifierAutoload; |
|
168 | + |
|
169 | + $composer = new stdClass(); |
|
170 | + $composer->extra = new stdClass(); |
|
171 | + $composer->extra->mozart = $mozartConfig; |
|
172 | + |
|
173 | + $composerFilepath = $this->testsWorkingDir . '/composer.json'; |
|
174 | + $composerJson = json_encode($composer) ; |
|
175 | + file_put_contents($composerFilepath, $composerJson); |
|
176 | + |
|
177 | + $this->config = StraussConfig::loadFromFile($composerFilepath); |
|
178 | + } |
|
179 | + |
|
180 | + /** |
|
181 | + * If the specified `dep_directory` or `classmap_directory` are absent, create them. |
|
182 | + * @see MoverTest.php |
|
183 | + * @test |
|
184 | + */ |
|
185 | + public function it_creates_absent_dirs(): void |
|
186 | + { |
|
187 | + $this->markTestIncomplete(); |
|
188 | + |
|
189 | + $mover = new Mover($this->testsWorkingDir, $this->config); |
|
190 | + |
|
191 | + // Make sure the directories don't exist. |
|
192 | + assert(! file_exists($this->testsWorkingDir . $this->config->gett()), "{$this->testsWorkingDir}{$this->config->getDepDirectory()} already exists"); |
|
193 | + assert(! file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())); |
|
194 | + |
|
195 | + $packages = array(); |
|
196 | + |
|
197 | + $mover->deleteTargetDirs($packages); |
|
198 | + |
|
199 | + $this->assertTrue(file_exists($this->testsWorkingDir |
|
200 | + . $this->config->getDepDirectory())); |
|
201 | + $this->assertTrue(file_exists($this->testsWorkingDir |
|
202 | + . $this->config->getClassmapDirectory())); |
|
203 | + } |
|
204 | + |
|
205 | + /** |
|
206 | + * If the specified `dep_directory` or `classmap_directory` already exists with contents, it is not an issue. |
|
207 | + * |
|
208 | + * @see MoverTest.php |
|
209 | + * |
|
210 | + * @test |
|
211 | + */ |
|
212 | + public function it_is_unpertrubed_by_existing_dirs(): void |
|
213 | + { |
|
214 | + $this->markTestIncomplete(); |
|
215 | + |
|
216 | + $mover = new Mover($this->testsWorkingDir, $this->config); |
|
217 | + |
|
218 | + if (!file_exists($this->testsWorkingDir . $this->config->getDepDirectory())) { |
|
219 | + mkdir($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
220 | + } |
|
221 | + if (!file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())) { |
|
222 | + mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
223 | + } |
|
224 | + |
|
225 | + $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
226 | + $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
227 | + |
|
228 | + $packages = array(); |
|
229 | + |
|
230 | + ob_start(); |
|
231 | + |
|
232 | + $mover->deleteTargetDirs($packages); |
|
233 | + |
|
234 | + $output = ob_get_clean(); |
|
235 | + |
|
236 | + $this->assertEmpty($output); |
|
237 | + } |
|
238 | + |
|
239 | + /** |
|
240 | + * If the specified `dep_directory` or `classmap_directory` contains a subdir we are going to need when moving, |
|
241 | + * delete the subdir. aka: If subfolders exist for dependencies we are about to manage, delete those subfolders. |
|
242 | + * |
|
243 | + * @see MoverTest.php |
|
244 | + * |
|
245 | + * @test |
|
246 | + */ |
|
247 | + public function it_deletes_subdirs_for_packages_about_to_be_moved(): void |
|
248 | + { |
|
249 | + $this->markTestIncomplete(); |
|
250 | + |
|
251 | + $mover = new Mover($this->testsWorkingDir, $this->config); |
|
252 | + |
|
253 | + @mkdir($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
254 | + @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
255 | + |
|
256 | + @mkdir($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple'); |
|
257 | + @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory() . 'ezyang'); |
|
258 | + |
|
259 | + $packages = array(); |
|
260 | + foreach ($this->config->getPackages() as $packageString) { |
|
261 | + $testDummyComposerDir = $this->testsWorkingDir . 'vendor' |
|
262 | + . DIRECTORY_SEPARATOR . $packageString; |
|
263 | + @mkdir($testDummyComposerDir, 0777, true); |
|
264 | + $testDummyComposerPath = $testDummyComposerDir . DIRECTORY_SEPARATOR . 'composer.json'; |
|
265 | + $testDummyComposerContents = json_encode(new stdClass()); |
|
266 | + |
|
267 | + file_put_contents($testDummyComposerPath, $testDummyComposerContents); |
|
268 | + $parsedPackage = new ComposerPackageConfig($testDummyComposerDir, $this->config->getOverrideAutoload()[$packageString]); |
|
269 | + $parsedPackage->findAutoloaders(); |
|
270 | + $packages[] = $parsedPackage; |
|
271 | + } |
|
272 | + |
|
273 | + $mover->deleteTargetDirs($packages); |
|
274 | + |
|
275 | + $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple'); |
|
276 | + $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'ezyang'); |
|
277 | + } |
|
278 | 278 | } |
@@ -23,10 +23,10 @@ discard block |
||
23 | 23 | class ReplacerIntegrationTest extends IntegrationTestCase |
24 | 24 | { |
25 | 25 | |
26 | - public function testReplaceNamespace() |
|
27 | - { |
|
26 | + public function testReplaceNamespace() |
|
27 | + { |
|
28 | 28 | |
29 | - $composerJsonString = <<<'EOD' |
|
29 | + $composerJsonString = <<<'EOD' |
|
30 | 30 | { |
31 | 31 | "name": "brianhenryie/strauss", |
32 | 32 | "require": { |
@@ -41,56 +41,56 @@ discard block |
||
41 | 41 | } |
42 | 42 | EOD; |
43 | 43 | |
44 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
44 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
45 | 45 | |
46 | - chdir($this->testsWorkingDir); |
|
46 | + chdir($this->testsWorkingDir); |
|
47 | 47 | |
48 | - exec('composer install'); |
|
48 | + exec('composer install'); |
|
49 | 49 | |
50 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
51 | - $config = $projectComposerPackage->getStraussConfig(); |
|
50 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
51 | + $config = $projectComposerPackage->getStraussConfig(); |
|
52 | 52 | |
53 | - $dependencies = array_map(function ($element) { |
|
54 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
55 | - return new ComposerPackage($dir); |
|
56 | - }, $projectComposerPackage->getRequiresNames()); |
|
53 | + $dependencies = array_map(function ($element) { |
|
54 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
55 | + return new ComposerPackage($dir); |
|
56 | + }, $projectComposerPackage->getRequiresNames()); |
|
57 | 57 | |
58 | - $workingDir = $this->testsWorkingDir; |
|
59 | - $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
60 | - $absoluteTargetDir = $workingDir . $relativeTargetDir; |
|
58 | + $workingDir = $this->testsWorkingDir; |
|
59 | + $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
60 | + $absoluteTargetDir = $workingDir . $relativeTargetDir; |
|
61 | 61 | |
62 | 62 | // $config = $this->createStub(StraussConfig::class); |
63 | 63 | // $config->method('getTargetDirectory')->willReturn('strauss' . DIRECTORY_SEPARATOR); |
64 | 64 | |
65 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
66 | - $fileEnumerator->compileFileList(); |
|
67 | - $fileList = $fileEnumerator->getAllFilesAndDependencyList(); |
|
68 | - $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList(); |
|
65 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
66 | + $fileEnumerator->compileFileList(); |
|
67 | + $fileList = $fileEnumerator->getAllFilesAndDependencyList(); |
|
68 | + $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList(); |
|
69 | 69 | |
70 | - $copier = new Copier($fileList, $workingDir, $relativeTargetDir); |
|
71 | - $copier->prepareTarget(); |
|
72 | - $copier->copy(); |
|
70 | + $copier = new Copier($fileList, $workingDir, $relativeTargetDir); |
|
71 | + $copier->prepareTarget(); |
|
72 | + $copier->copy(); |
|
73 | 73 | |
74 | - $changeEnumerator = new ChangeEnumerator($config); |
|
75 | - $changeEnumerator->findInFiles($absoluteTargetDir, $phpFileList); |
|
74 | + $changeEnumerator = new ChangeEnumerator($config); |
|
75 | + $changeEnumerator->findInFiles($absoluteTargetDir, $phpFileList); |
|
76 | 76 | |
77 | - $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements(); |
|
78 | - $classes = $changeEnumerator->getDiscoveredClasses(); |
|
77 | + $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements(); |
|
78 | + $classes = $changeEnumerator->getDiscoveredClasses(); |
|
79 | 79 | |
80 | - $replacer = new Prefixer($config, $workingDir); |
|
80 | + $replacer = new Prefixer($config, $workingDir); |
|
81 | 81 | |
82 | - $replacer->replaceInFiles($namespaces, $classes, $phpFileList); |
|
82 | + $replacer->replaceInFiles($namespaces, $classes, $phpFileList); |
|
83 | 83 | |
84 | - $updatedFile = file_get_contents($absoluteTargetDir . 'google/apiclient/src/Client.php'); |
|
84 | + $updatedFile = file_get_contents($absoluteTargetDir . 'google/apiclient/src/Client.php'); |
|
85 | 85 | |
86 | - $this->assertStringContainsString('use BrianHenryIE\Strauss\Google\AccessToken\Revoke;', $updatedFile); |
|
87 | - } |
|
86 | + $this->assertStringContainsString('use BrianHenryIE\Strauss\Google\AccessToken\Revoke;', $updatedFile); |
|
87 | + } |
|
88 | 88 | |
89 | 89 | |
90 | - public function testReplaceClass() |
|
91 | - { |
|
90 | + public function testReplaceClass() |
|
91 | + { |
|
92 | 92 | |
93 | - $composerJsonString = <<<'EOD' |
|
93 | + $composerJsonString = <<<'EOD' |
|
94 | 94 | { |
95 | 95 | "name": "brianhenryie/strauss", |
96 | 96 | "require": { |
@@ -106,18 +106,18 @@ discard block |
||
106 | 106 | } |
107 | 107 | EOD; |
108 | 108 | |
109 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
109 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
110 | 110 | |
111 | - chdir($this->testsWorkingDir); |
|
111 | + chdir($this->testsWorkingDir); |
|
112 | 112 | |
113 | - exec('composer install'); |
|
113 | + exec('composer install'); |
|
114 | 114 | |
115 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
116 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
115 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
116 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
117 | 117 | |
118 | - $mozartCompose = new Compose(); |
|
118 | + $mozartCompose = new Compose(); |
|
119 | 119 | |
120 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
120 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
121 | 121 | |
122 | 122 | |
123 | 123 | // $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
@@ -150,8 +150,8 @@ discard block |
||
150 | 150 | // |
151 | 151 | // $replacer->replaceInFiles($namespaces, $classes, $phpFileList); |
152 | 152 | |
153 | - $updatedFile = file_get_contents($this->testsWorkingDir . '/strauss/' . 'setasign/fpdf/fpdf.php'); |
|
153 | + $updatedFile = file_get_contents($this->testsWorkingDir . '/strauss/' . 'setasign/fpdf/fpdf.php'); |
|
154 | 154 | |
155 | - $this->assertStringContainsString('class BrianHenryIE_Strauss_FPDF', $updatedFile); |
|
156 | - } |
|
155 | + $this->assertStringContainsString('class BrianHenryIE_Strauss_FPDF', $updatedFile); |
|
156 | + } |
|
157 | 157 | } |
@@ -19,10 +19,10 @@ discard block |
||
19 | 19 | class FileEnumeratorIntegrationTest extends IntegrationTestCase |
20 | 20 | { |
21 | 21 | |
22 | - public function testBuildFileList() |
|
23 | - { |
|
22 | + public function testBuildFileList() |
|
23 | + { |
|
24 | 24 | |
25 | - $composerJsonString = <<<'EOD' |
|
25 | + $composerJsonString = <<<'EOD' |
|
26 | 26 | { |
27 | 27 | "name": "brianhenryie/strauss", |
28 | 28 | "require": { |
@@ -38,41 +38,41 @@ discard block |
||
38 | 38 | } |
39 | 39 | EOD; |
40 | 40 | |
41 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
41 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
42 | 42 | |
43 | - chdir($this->testsWorkingDir); |
|
43 | + chdir($this->testsWorkingDir); |
|
44 | 44 | |
45 | - exec('composer install'); |
|
45 | + exec('composer install'); |
|
46 | 46 | |
47 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
47 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
48 | 48 | |
49 | - // Only one because we haven't run "flat dependency list". |
|
50 | - $dependencies = array_map(function ($element) { |
|
51 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
52 | - return new ComposerPackage($dir); |
|
53 | - }, $projectComposerPackage->getRequiresNames()); |
|
49 | + // Only one because we haven't run "flat dependency list". |
|
50 | + $dependencies = array_map(function ($element) { |
|
51 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
52 | + return new ComposerPackage($dir); |
|
53 | + }, $projectComposerPackage->getRequiresNames()); |
|
54 | 54 | |
55 | - $workingDir = $this->testsWorkingDir; |
|
56 | - $config = $this->createStub(StraussConfig::class); |
|
55 | + $workingDir = $this->testsWorkingDir; |
|
56 | + $config = $this->createStub(StraussConfig::class); |
|
57 | 57 | |
58 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
58 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
59 | 59 | |
60 | - $fileEnumerator->compileFileList(); |
|
60 | + $fileEnumerator->compileFileList(); |
|
61 | 61 | |
62 | - $list = array_keys($fileEnumerator->getAllFilesAndDependencyList()); |
|
62 | + $list = array_keys($fileEnumerator->getAllFilesAndDependencyList()); |
|
63 | 63 | |
64 | - $this->assertContains('google/apiclient/src/aliases.php', $list); |
|
65 | - } |
|
64 | + $this->assertContains('google/apiclient/src/aliases.php', $list); |
|
65 | + } |
|
66 | 66 | |
67 | 67 | |
68 | - public function testClassmapAutoloader() |
|
69 | - { |
|
70 | - $this->markTestIncomplete(); |
|
71 | - } |
|
68 | + public function testClassmapAutoloader() |
|
69 | + { |
|
70 | + $this->markTestIncomplete(); |
|
71 | + } |
|
72 | 72 | |
73 | 73 | |
74 | - public function testFilesAutoloader() |
|
75 | - { |
|
76 | - $this->markTestIncomplete(); |
|
77 | - } |
|
74 | + public function testFilesAutoloader() |
|
75 | + { |
|
76 | + $this->markTestIncomplete(); |
|
77 | + } |
|
78 | 78 | } |
@@ -18,13 +18,13 @@ discard block |
||
18 | 18 | class ChangeEnumeratorIntegrationTest extends IntegrationTestCase |
19 | 19 | { |
20 | 20 | |
21 | - /** |
|
22 | - * Given a list of files, find all the global classes and the namespaces. |
|
23 | - */ |
|
24 | - public function testOne() |
|
25 | - { |
|
21 | + /** |
|
22 | + * Given a list of files, find all the global classes and the namespaces. |
|
23 | + */ |
|
24 | + public function testOne() |
|
25 | + { |
|
26 | 26 | |
27 | - $composerJsonString = <<<'EOD' |
|
27 | + $composerJsonString = <<<'EOD' |
|
28 | 28 | { |
29 | 29 | "name": "brianhenryie/strauss", |
30 | 30 | "require": { |
@@ -40,53 +40,53 @@ discard block |
||
40 | 40 | } |
41 | 41 | EOD; |
42 | 42 | |
43 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
43 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
44 | 44 | |
45 | - chdir($this->testsWorkingDir); |
|
45 | + chdir($this->testsWorkingDir); |
|
46 | 46 | |
47 | - exec('composer install'); |
|
47 | + exec('composer install'); |
|
48 | 48 | |
49 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
49 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
50 | 50 | |
51 | - $dependencies = array_map(function ($element) { |
|
52 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
53 | - return new ComposerPackage($dir); |
|
54 | - }, $projectComposerPackage->getRequiresNames()); |
|
51 | + $dependencies = array_map(function ($element) { |
|
52 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
53 | + return new ComposerPackage($dir); |
|
54 | + }, $projectComposerPackage->getRequiresNames()); |
|
55 | 55 | |
56 | - $workingDir = $this->testsWorkingDir; |
|
57 | - $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
56 | + $workingDir = $this->testsWorkingDir; |
|
57 | + $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
58 | 58 | |
59 | - $config = $this->createStub(StraussConfig::class); |
|
59 | + $config = $this->createStub(StraussConfig::class); |
|
60 | 60 | |
61 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
61 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
62 | 62 | |
63 | - $fileEnumerator->compileFileList(); |
|
63 | + $fileEnumerator->compileFileList(); |
|
64 | 64 | |
65 | - $copier = new Copier($fileEnumerator->getAllFilesAndDependencyList(), $workingDir, $relativeTargetDir); |
|
65 | + $copier = new Copier($fileEnumerator->getAllFilesAndDependencyList(), $workingDir, $relativeTargetDir); |
|
66 | 66 | |
67 | - $copier->prepareTarget(); |
|
67 | + $copier->prepareTarget(); |
|
68 | 68 | |
69 | - $copier->copy(); |
|
69 | + $copier->copy(); |
|
70 | 70 | |
71 | - $config = $this->createStub(StraussConfig::class); |
|
71 | + $config = $this->createStub(StraussConfig::class); |
|
72 | 72 | |
73 | - $config->method('getExcludePackagesFromPrefixing')->willReturn(array()); |
|
74 | - $config->method('getExcludeNamespacesFromPrefixing')->willReturn(array()); |
|
73 | + $config->method('getExcludePackagesFromPrefixing')->willReturn(array()); |
|
74 | + $config->method('getExcludeNamespacesFromPrefixing')->willReturn(array()); |
|
75 | 75 | |
76 | - $changeEnumerator = new ChangeEnumerator($config); |
|
76 | + $changeEnumerator = new ChangeEnumerator($config); |
|
77 | 77 | |
78 | - $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList(); |
|
78 | + $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList(); |
|
79 | 79 | |
80 | - $changeEnumerator->findInFiles($workingDir . $relativeTargetDir, $phpFileList); |
|
80 | + $changeEnumerator->findInFiles($workingDir . $relativeTargetDir, $phpFileList); |
|
81 | 81 | |
82 | 82 | |
83 | - $classes = $changeEnumerator->getDiscoveredClasses(); |
|
83 | + $classes = $changeEnumerator->getDiscoveredClasses(); |
|
84 | 84 | |
85 | - $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements(); |
|
85 | + $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements(); |
|
86 | 86 | |
87 | - $this->assertNotEmpty($classes); |
|
88 | - $this->assertNotEmpty($namespaces); |
|
87 | + $this->assertNotEmpty($classes); |
|
88 | + $this->assertNotEmpty($namespaces); |
|
89 | 89 | |
90 | - $this->assertContains('Google_Task_Composer', $classes); |
|
91 | - } |
|
90 | + $this->assertContains('Google_Task_Composer', $classes); |
|
91 | + } |
|
92 | 92 | } |