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