@@ -22,23 +22,23 @@ discard block |
||
22 | 22 | class MozartIssue66Test extends IntegrationTestCase |
23 | 23 | { |
24 | 24 | |
25 | - /** |
|
26 | - * |
|
27 | - * php-di's composer.json's autoload key: |
|
28 | - * |
|
29 | - * "autoload": { |
|
30 | - * "psr-4": { |
|
31 | - * "DI\\": "src/" |
|
32 | - * }, |
|
33 | - * "files": [ |
|
34 | - * "src/functions.php" |
|
35 | - * ] |
|
36 | - * }, |
|
37 | - */ |
|
38 | - public function testFilesAutoloaderIsUsed() |
|
39 | - { |
|
25 | + /** |
|
26 | + * |
|
27 | + * php-di's composer.json's autoload key: |
|
28 | + * |
|
29 | + * "autoload": { |
|
30 | + * "psr-4": { |
|
31 | + * "DI\\": "src/" |
|
32 | + * }, |
|
33 | + * "files": [ |
|
34 | + * "src/functions.php" |
|
35 | + * ] |
|
36 | + * }, |
|
37 | + */ |
|
38 | + public function testFilesAutoloaderIsUsed() |
|
39 | + { |
|
40 | 40 | |
41 | - $composerJsonString = <<<'EOD' |
|
41 | + $composerJsonString = <<<'EOD' |
|
42 | 42 | { |
43 | 43 | "name": "markjaquith/mozart-bug-example", |
44 | 44 | "require": { |
@@ -64,19 +64,19 @@ discard block |
||
64 | 64 | |
65 | 65 | EOD; |
66 | 66 | |
67 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
67 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
68 | 68 | |
69 | - chdir($this->testsWorkingDir); |
|
69 | + chdir($this->testsWorkingDir); |
|
70 | 70 | |
71 | - exec('composer install'); |
|
71 | + exec('composer install'); |
|
72 | 72 | |
73 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
74 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
73 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
74 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
75 | 75 | |
76 | - $mozartCompose = new Compose(); |
|
76 | + $mozartCompose = new Compose(); |
|
77 | 77 | |
78 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
78 | + $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
79 | 79 | |
80 | - $this->assertFileExists($this->testsWorkingDir . 'strauss/php-di/php-di/src/functions.php'); |
|
81 | - } |
|
80 | + $this->assertFileExists($this->testsWorkingDir . 'strauss/php-di/php-di/src/functions.php'); |
|
81 | + } |
|
82 | 82 | } |
@@ -64,19 +64,19 @@ |
||
64 | 64 | |
65 | 65 | EOD; |
66 | 66 | |
67 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
67 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
68 | 68 | |
69 | - chdir($this->testsWorkingDir); |
|
69 | + chdir( $this->testsWorkingDir ); |
|
70 | 70 | |
71 | - exec('composer install'); |
|
71 | + exec( 'composer install' ); |
|
72 | 72 | |
73 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
74 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
73 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
74 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
75 | 75 | |
76 | 76 | $mozartCompose = new Compose(); |
77 | 77 | |
78 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
78 | + $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
79 | 79 | |
80 | - $this->assertFileExists($this->testsWorkingDir . 'strauss/php-di/php-di/src/functions.php'); |
|
80 | + $this->assertFileExists( $this->testsWorkingDir . 'strauss/php-di/php-di/src/functions.php' ); |
|
81 | 81 | } |
82 | 82 | } |
@@ -19,8 +19,7 @@ discard block |
||
19 | 19 | * @package BrianHenryIE\Strauss\Tests\Issues |
20 | 20 | * @coversNothing |
21 | 21 | */ |
22 | -class MozartIssue66Test extends IntegrationTestCase |
|
23 | -{ |
|
22 | +class MozartIssue66Test extends IntegrationTestCase { |
|
24 | 23 | |
25 | 24 | /** |
26 | 25 | * |
@@ -35,8 +34,7 @@ discard block |
||
35 | 34 | * ] |
36 | 35 | * }, |
37 | 36 | */ |
38 | - public function testFilesAutoloaderIsUsed() |
|
39 | - { |
|
37 | + public function testFilesAutoloaderIsUsed() { |
|
40 | 38 | |
41 | 39 | $composerJsonString = <<<'EOD' |
42 | 40 | { |
@@ -62,7 +62,7 @@ |
||
62 | 62 | } |
63 | 63 | } |
64 | 64 | |
65 | -EOD; |
|
65 | +eod; |
|
66 | 66 | |
67 | 67 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
68 | 68 |
@@ -17,13 +17,13 @@ discard block |
||
17 | 17 | class StraussIssue8Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
18 | 18 | { |
19 | 19 | |
20 | - /** |
|
21 | - * @author BrianHenryIE |
|
22 | - */ |
|
23 | - public function test_delete_vendor_files() |
|
24 | - { |
|
20 | + /** |
|
21 | + * @author BrianHenryIE |
|
22 | + */ |
|
23 | + public function test_delete_vendor_files() |
|
24 | + { |
|
25 | 25 | |
26 | - $composerJsonString = <<<'EOD' |
|
26 | + $composerJsonString = <<<'EOD' |
|
27 | 27 | { |
28 | 28 | "name": "brianhenryie/strauss-issue-8", |
29 | 29 | "require": { |
@@ -37,21 +37,21 @@ discard block |
||
37 | 37 | } |
38 | 38 | EOD; |
39 | 39 | |
40 | - file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
40 | + file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
41 | 41 | |
42 | - chdir($this->testsWorkingDir); |
|
42 | + chdir($this->testsWorkingDir); |
|
43 | 43 | |
44 | - exec('composer install'); |
|
44 | + exec('composer install'); |
|
45 | 45 | |
46 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
47 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
46 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
47 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
48 | 48 | |
49 | - $mozartCompose = new Compose(); |
|
49 | + $mozartCompose = new Compose(); |
|
50 | 50 | |
51 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
51 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
52 | 52 | |
53 | - $this->assertEquals(0, $result); |
|
53 | + $this->assertEquals(0, $result); |
|
54 | 54 | |
55 | - $this->assertFileDoesNotExist($this->testsWorkingDir. 'vendor/htmlburger/carbon-fields/core/Carbon_Fields.php'); |
|
56 | - } |
|
55 | + $this->assertFileDoesNotExist($this->testsWorkingDir. 'vendor/htmlburger/carbon-fields/core/Carbon_Fields.php'); |
|
56 | + } |
|
57 | 57 | } |
@@ -37,21 +37,21 @@ |
||
37 | 37 | } |
38 | 38 | EOD; |
39 | 39 | |
40 | - file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
40 | + file_put_contents( $this->testsWorkingDir . 'composer.json', $composerJsonString ); |
|
41 | 41 | |
42 | - chdir($this->testsWorkingDir); |
|
42 | + chdir( $this->testsWorkingDir ); |
|
43 | 43 | |
44 | - exec('composer install'); |
|
44 | + exec( 'composer install' ); |
|
45 | 45 | |
46 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
47 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
46 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
47 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
48 | 48 | |
49 | 49 | $mozartCompose = new Compose(); |
50 | 50 | |
51 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
51 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
52 | 52 | |
53 | - $this->assertEquals(0, $result); |
|
53 | + $this->assertEquals( 0, $result ); |
|
54 | 54 | |
55 | - $this->assertFileDoesNotExist($this->testsWorkingDir. 'vendor/htmlburger/carbon-fields/core/Carbon_Fields.php'); |
|
55 | + $this->assertFileDoesNotExist( $this->testsWorkingDir . 'vendor/htmlburger/carbon-fields/core/Carbon_Fields.php' ); |
|
56 | 56 | } |
57 | 57 | } |
@@ -14,14 +14,12 @@ |
||
14 | 14 | * @package BrianHenryIE\Strauss\Tests\Issues |
15 | 15 | * @coversNothing |
16 | 16 | */ |
17 | -class StraussIssue8Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
|
18 | -{ |
|
17 | +class StraussIssue8Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase { |
|
19 | 18 | |
20 | 19 | /** |
21 | 20 | * @author BrianHenryIE |
22 | 21 | */ |
23 | - public function test_delete_vendor_files() |
|
24 | - { |
|
22 | + public function test_delete_vendor_files() { |
|
25 | 23 | |
26 | 24 | $composerJsonString = <<<'EOD' |
27 | 25 | { |
@@ -35,7 +35,7 @@ |
||
35 | 35 | } |
36 | 36 | } |
37 | 37 | } |
38 | -EOD; |
|
38 | +eod; |
|
39 | 39 | |
40 | 40 | file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
41 | 41 |
@@ -22,13 +22,13 @@ discard block |
||
22 | 22 | class MozartIssue130Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
23 | 23 | { |
24 | 24 | |
25 | - /** |
|
26 | - * @author BrianHenryIE |
|
27 | - */ |
|
28 | - public function test_config_copied() |
|
29 | - { |
|
25 | + /** |
|
26 | + * @author BrianHenryIE |
|
27 | + */ |
|
28 | + public function test_config_copied() |
|
29 | + { |
|
30 | 30 | |
31 | - $composerJsonString = <<<'EOD' |
|
31 | + $composerJsonString = <<<'EOD' |
|
32 | 32 | { |
33 | 33 | "name": "brianhenryie/mozart-issue-130", |
34 | 34 | "require": { |
@@ -56,19 +56,19 @@ discard block |
||
56 | 56 | } |
57 | 57 | EOD; |
58 | 58 | |
59 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
59 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
60 | 60 | |
61 | - chdir($this->testsWorkingDir); |
|
61 | + chdir($this->testsWorkingDir); |
|
62 | 62 | |
63 | - exec('composer install'); |
|
63 | + exec('composer install'); |
|
64 | 64 | |
65 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
66 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
65 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
66 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
67 | 67 | |
68 | - $mozartCompose = new Compose(); |
|
68 | + $mozartCompose = new Compose(); |
|
69 | 69 | |
70 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
70 | + $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
71 | 71 | |
72 | - $this->assertFileExists($this->testsWorkingDir .'strauss/htmlburger/carbon-fields/config.php'); |
|
73 | - } |
|
72 | + $this->assertFileExists($this->testsWorkingDir .'strauss/htmlburger/carbon-fields/config.php'); |
|
73 | + } |
|
74 | 74 | } |
@@ -56,19 +56,19 @@ |
||
56 | 56 | } |
57 | 57 | EOD; |
58 | 58 | |
59 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
59 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
60 | 60 | |
61 | - chdir($this->testsWorkingDir); |
|
61 | + chdir( $this->testsWorkingDir ); |
|
62 | 62 | |
63 | - exec('composer install'); |
|
63 | + exec( 'composer install' ); |
|
64 | 64 | |
65 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
66 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
65 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
66 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
67 | 67 | |
68 | 68 | $mozartCompose = new Compose(); |
69 | 69 | |
70 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
70 | + $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
71 | 71 | |
72 | - $this->assertFileExists($this->testsWorkingDir .'strauss/htmlburger/carbon-fields/config.php'); |
|
72 | + $this->assertFileExists( $this->testsWorkingDir . 'strauss/htmlburger/carbon-fields/config.php' ); |
|
73 | 73 | } |
74 | 74 | } |
@@ -19,14 +19,12 @@ |
||
19 | 19 | * @package BrianHenryIE\Strauss\Tests\Issues |
20 | 20 | * @coversNothing |
21 | 21 | */ |
22 | -class MozartIssue130Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
|
23 | -{ |
|
22 | +class MozartIssue130Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase { |
|
24 | 23 | |
25 | 24 | /** |
26 | 25 | * @author BrianHenryIE |
27 | 26 | */ |
28 | - public function test_config_copied() |
|
29 | - { |
|
27 | + public function test_config_copied() { |
|
30 | 28 | |
31 | 29 | $composerJsonString = <<<'EOD' |
32 | 30 | { |
@@ -36,7 +36,7 @@ |
||
36 | 36 | } |
37 | 37 | } |
38 | 38 | } |
39 | -EOD; |
|
39 | +eod; |
|
40 | 40 | |
41 | 41 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
42 | 42 |
@@ -18,16 +18,16 @@ discard block |
||
18 | 18 | class MozartIssue128Test extends IntegrationTestCase |
19 | 19 | { |
20 | 20 | |
21 | - /** |
|
22 | - * Because the neither package was a sub-package of the other, the replacing was not occurring |
|
23 | - * throughout. |
|
24 | - */ |
|
25 | - public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
26 | - { |
|
21 | + /** |
|
22 | + * Because the neither package was a sub-package of the other, the replacing was not occurring |
|
23 | + * throughout. |
|
24 | + */ |
|
25 | + public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
26 | + { |
|
27 | 27 | |
28 | - $this->markTestSkipped("Failing on PHP 8"); |
|
28 | + $this->markTestSkipped("Failing on PHP 8"); |
|
29 | 29 | |
30 | - $composerJsonString = <<<'EOD' |
|
30 | + $composerJsonString = <<<'EOD' |
|
31 | 31 | { |
32 | 32 | "require": { |
33 | 33 | "setasign/fpdf": "1.8", |
@@ -45,27 +45,27 @@ discard block |
||
45 | 45 | } |
46 | 46 | EOD; |
47 | 47 | |
48 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
48 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
49 | 49 | |
50 | - chdir($this->testsWorkingDir); |
|
50 | + chdir($this->testsWorkingDir); |
|
51 | 51 | |
52 | - exec('composer install'); |
|
52 | + exec('composer install'); |
|
53 | 53 | |
54 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
55 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
54 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
55 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
56 | 56 | |
57 | - $mozartCompose = new Compose(); |
|
57 | + $mozartCompose = new Compose(); |
|
58 | 58 | |
59 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
59 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
60 | 60 | |
61 | - assert(0 === $result); |
|
61 | + assert(0 === $result); |
|
62 | 62 | |
63 | - $mpdf_php = file_get_contents($this->testsWorkingDir .'strauss/setasign/fpdi/src/FpdfTpl.php'); |
|
63 | + $mpdf_php = file_get_contents($this->testsWorkingDir .'strauss/setasign/fpdi/src/FpdfTpl.php'); |
|
64 | 64 | |
65 | - // Confirm problem is gone. |
|
66 | - $this->assertStringNotContainsString('class FpdfTpl extends \FPDF', $mpdf_php); |
|
65 | + // Confirm problem is gone. |
|
66 | + $this->assertStringNotContainsString('class FpdfTpl extends \FPDF', $mpdf_php); |
|
67 | 67 | |
68 | - // Confirm solution is correct. |
|
69 | - $this->assertStringContainsString('class FpdfTpl extends \Strauss_FPDF', $mpdf_php); |
|
70 | - } |
|
68 | + // Confirm solution is correct. |
|
69 | + $this->assertStringContainsString('class FpdfTpl extends \Strauss_FPDF', $mpdf_php); |
|
70 | + } |
|
71 | 71 | } |
@@ -25,7 +25,7 @@ discard block |
||
25 | 25 | public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
26 | 26 | { |
27 | 27 | |
28 | - $this->markTestSkipped("Failing on PHP 8"); |
|
28 | + $this->markTestSkipped( "Failing on PHP 8" ); |
|
29 | 29 | |
30 | 30 | $composerJsonString = <<<'EOD' |
31 | 31 | { |
@@ -45,27 +45,27 @@ discard block |
||
45 | 45 | } |
46 | 46 | EOD; |
47 | 47 | |
48 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
48 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
49 | 49 | |
50 | - chdir($this->testsWorkingDir); |
|
50 | + chdir( $this->testsWorkingDir ); |
|
51 | 51 | |
52 | - exec('composer install'); |
|
52 | + exec( 'composer install' ); |
|
53 | 53 | |
54 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
55 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
54 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
55 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
56 | 56 | |
57 | 57 | $mozartCompose = new Compose(); |
58 | 58 | |
59 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
59 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
60 | 60 | |
61 | - assert(0 === $result); |
|
61 | + assert( 0 === $result ); |
|
62 | 62 | |
63 | - $mpdf_php = file_get_contents($this->testsWorkingDir .'strauss/setasign/fpdi/src/FpdfTpl.php'); |
|
63 | + $mpdf_php = file_get_contents( $this->testsWorkingDir . 'strauss/setasign/fpdi/src/FpdfTpl.php' ); |
|
64 | 64 | |
65 | 65 | // Confirm problem is gone. |
66 | - $this->assertStringNotContainsString('class FpdfTpl extends \FPDF', $mpdf_php); |
|
66 | + $this->assertStringNotContainsString( 'class FpdfTpl extends \FPDF', $mpdf_php ); |
|
67 | 67 | |
68 | 68 | // Confirm solution is correct. |
69 | - $this->assertStringContainsString('class FpdfTpl extends \Strauss_FPDF', $mpdf_php); |
|
69 | + $this->assertStringContainsString( 'class FpdfTpl extends \Strauss_FPDF', $mpdf_php ); |
|
70 | 70 | } |
71 | 71 | } |
@@ -15,15 +15,13 @@ |
||
15 | 15 | * Class MozartIssue128Test |
16 | 16 | * @coversNothing |
17 | 17 | */ |
18 | -class MozartIssue128Test extends IntegrationTestCase |
|
19 | -{ |
|
18 | +class MozartIssue128Test extends IntegrationTestCase { |
|
20 | 19 | |
21 | 20 | /** |
22 | 21 | * Because the neither package was a sub-package of the other, the replacing was not occurring |
23 | 22 | * throughout. |
24 | 23 | */ |
25 | - public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
26 | - { |
|
24 | + public function test_it_does_not_make_classname_replacement_inside_namespaced_file() { |
|
27 | 25 | |
28 | 26 | $this->markTestSkipped("Failing on PHP 8"); |
29 | 27 |
@@ -36,7 +36,7 @@ |
||
36 | 36 | } |
37 | 37 | } |
38 | 38 | } |
39 | -EOD; |
|
39 | +eod; |
|
40 | 40 | |
41 | 41 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
42 | 42 |
@@ -20,21 +20,21 @@ discard block |
||
20 | 20 | class MozartIssue124Test extends IntegrationTestCase |
21 | 21 | { |
22 | 22 | |
23 | - /** |
|
24 | - * After PR #84, running Mozart on Mpdf began prefixing the class name inside the namespaced file. |
|
25 | - * |
|
26 | - * The problem coming from the filename matching the namespace name? |
|
27 | - * |
|
28 | - * dev-master#5d8041fdefc94ff57edcbe83ab468a9988c4fc11 |
|
29 | - * |
|
30 | - * @see https://github.com/coenjacobs/mozart/pull/84/files |
|
31 | - * |
|
32 | - * Should be: "class Mpdf implements" because its namespace has already been prefixed. |
|
33 | - */ |
|
34 | - public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
35 | - { |
|
36 | - |
|
37 | - $composerJsonString = <<<'EOD' |
|
23 | + /** |
|
24 | + * After PR #84, running Mozart on Mpdf began prefixing the class name inside the namespaced file. |
|
25 | + * |
|
26 | + * The problem coming from the filename matching the namespace name? |
|
27 | + * |
|
28 | + * dev-master#5d8041fdefc94ff57edcbe83ab468a9988c4fc11 |
|
29 | + * |
|
30 | + * @see https://github.com/coenjacobs/mozart/pull/84/files |
|
31 | + * |
|
32 | + * Should be: "class Mpdf implements" because its namespace has already been prefixed. |
|
33 | + */ |
|
34 | + public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
35 | + { |
|
36 | + |
|
37 | + $composerJsonString = <<<'EOD' |
|
38 | 38 | { |
39 | 39 | "name": "brianhenryie/mozart-issue-124", |
40 | 40 | "require": { |
@@ -49,44 +49,44 @@ discard block |
||
49 | 49 | } |
50 | 50 | EOD; |
51 | 51 | |
52 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
52 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
53 | 53 | |
54 | - chdir($this->testsWorkingDir); |
|
54 | + chdir($this->testsWorkingDir); |
|
55 | 55 | |
56 | - exec('composer install'); |
|
56 | + exec('composer install'); |
|
57 | 57 | |
58 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
59 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
58 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
59 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
60 | 60 | |
61 | - $mozartCompose = new Compose(); |
|
61 | + $mozartCompose = new Compose(); |
|
62 | 62 | |
63 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
63 | + $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
64 | 64 | |
65 | - $mpdf_php = file_get_contents($this->testsWorkingDir .'strauss/mpdf/mpdf/src/Mpdf.php'); |
|
65 | + $mpdf_php = file_get_contents($this->testsWorkingDir .'strauss/mpdf/mpdf/src/Mpdf.php'); |
|
66 | 66 | |
67 | - // Confirm problem is gone. |
|
68 | - $this->assertStringNotContainsString('class BrianHenryIE\Strauss\Mpdf implements', $mpdf_php); |
|
67 | + // Confirm problem is gone. |
|
68 | + $this->assertStringNotContainsString('class BrianHenryIE\Strauss\Mpdf implements', $mpdf_php); |
|
69 | 69 | |
70 | - // Confirm solution is correct. |
|
71 | - $this->assertStringContainsString('class Mpdf implements', $mpdf_php); |
|
72 | - } |
|
70 | + // Confirm solution is correct. |
|
71 | + $this->assertStringContainsString('class Mpdf implements', $mpdf_php); |
|
72 | + } |
|
73 | 73 | |
74 | 74 | |
75 | - /** |
|
76 | - * Another Mpdf problem, presumably down to the classname matching the namespace. |
|
77 | - * |
|
78 | - * Typed function properties whose class type (name) match the namespace being replaced are |
|
79 | - * unintentionally prefixed, causing PHP to crash. |
|
80 | - * |
|
81 | - * Occurring at dev-master#3b1243ca8505fa6436569800dc34269178930f39 |
|
82 | - * |
|
83 | - * @see https://github.com/coenjacobs/mozart/issues/124 |
|
84 | - */ |
|
85 | - public function test_it_does_not_prefix_function_argument_types_whose_classname_matches_the_namespace() |
|
86 | - { |
|
75 | + /** |
|
76 | + * Another Mpdf problem, presumably down to the classname matching the namespace. |
|
77 | + * |
|
78 | + * Typed function properties whose class type (name) match the namespace being replaced are |
|
79 | + * unintentionally prefixed, causing PHP to crash. |
|
80 | + * |
|
81 | + * Occurring at dev-master#3b1243ca8505fa6436569800dc34269178930f39 |
|
82 | + * |
|
83 | + * @see https://github.com/coenjacobs/mozart/issues/124 |
|
84 | + */ |
|
85 | + public function test_it_does_not_prefix_function_argument_types_whose_classname_matches_the_namespace() |
|
86 | + { |
|
87 | 87 | |
88 | 88 | |
89 | - $composerJsonString = <<<'EOD' |
|
89 | + $composerJsonString = <<<'EOD' |
|
90 | 90 | { |
91 | 91 | "name": "brianhenryie/mozart-issue-124", |
92 | 92 | "require": { |
@@ -101,41 +101,41 @@ discard block |
||
101 | 101 | } |
102 | 102 | EOD; |
103 | 103 | |
104 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
104 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
105 | 105 | |
106 | - chdir($this->testsWorkingDir); |
|
106 | + chdir($this->testsWorkingDir); |
|
107 | 107 | |
108 | - exec('composer install'); |
|
108 | + exec('composer install'); |
|
109 | 109 | |
110 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
111 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
110 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
111 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
112 | 112 | |
113 | - $mozartCompose = new Compose(); |
|
113 | + $mozartCompose = new Compose(); |
|
114 | 114 | |
115 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
115 | + $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
116 | 116 | |
117 | - $mpdf_php = file_get_contents($this->testsWorkingDir .'/strauss/mpdf/mpdf/src/Conversion/DecToOther.php'); |
|
117 | + $mpdf_php = file_get_contents($this->testsWorkingDir .'/strauss/mpdf/mpdf/src/Conversion/DecToOther.php'); |
|
118 | 118 | |
119 | - // Confirm problem is gone. |
|
120 | - $this->assertStringNotContainsString('public function __construct(BrianHenryIE\Strauss\Mpdf $mpdf)', $mpdf_php); |
|
119 | + // Confirm problem is gone. |
|
120 | + $this->assertStringNotContainsString('public function __construct(BrianHenryIE\Strauss\Mpdf $mpdf)', $mpdf_php); |
|
121 | 121 | |
122 | - // Confirm solution is correct. |
|
123 | - $this->assertStringContainsString('public function __construct(Mpdf $mpdf)', $mpdf_php); |
|
124 | - } |
|
125 | - // src/strauss/mpdf/mpdf/src/Barcode/BarcodeException.php |
|
122 | + // Confirm solution is correct. |
|
123 | + $this->assertStringContainsString('public function __construct(Mpdf $mpdf)', $mpdf_php); |
|
124 | + } |
|
125 | + // src/strauss/mpdf/mpdf/src/Barcode/BarcodeException.php |
|
126 | 126 | |
127 | 127 | |
128 | 128 | |
129 | - /** |
|
130 | - * Another Mpdf problem, presumably down to the classname matching the namespace. |
|
131 | - * |
|
132 | - * @see mpdf/mpdf/src/Barcode/BarcodeException.php |
|
133 | - */ |
|
134 | - public function testItDoesPrefixNamespacedExtends() |
|
135 | - { |
|
129 | + /** |
|
130 | + * Another Mpdf problem, presumably down to the classname matching the namespace. |
|
131 | + * |
|
132 | + * @see mpdf/mpdf/src/Barcode/BarcodeException.php |
|
133 | + */ |
|
134 | + public function testItDoesPrefixNamespacedExtends() |
|
135 | + { |
|
136 | 136 | |
137 | 137 | |
138 | - $composerJsonString = <<<'EOD' |
|
138 | + $composerJsonString = <<<'EOD' |
|
139 | 139 | { |
140 | 140 | "name": "brianhenryie/mozart-issue-124", |
141 | 141 | "require": { |
@@ -150,25 +150,25 @@ discard block |
||
150 | 150 | } |
151 | 151 | EOD; |
152 | 152 | |
153 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
153 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
154 | 154 | |
155 | - chdir($this->testsWorkingDir); |
|
155 | + chdir($this->testsWorkingDir); |
|
156 | 156 | |
157 | - exec('composer install'); |
|
157 | + exec('composer install'); |
|
158 | 158 | |
159 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
160 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
159 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
160 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
161 | 161 | |
162 | - $mozartCompose = new Compose(); |
|
162 | + $mozartCompose = new Compose(); |
|
163 | 163 | |
164 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
164 | + $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
165 | 165 | |
166 | - $mpdf_php = file_get_contents($this->testsWorkingDir .'/strauss/mpdf/mpdf/src/Barcode/BarcodeException.php'); |
|
166 | + $mpdf_php = file_get_contents($this->testsWorkingDir .'/strauss/mpdf/mpdf/src/Barcode/BarcodeException.php'); |
|
167 | 167 | |
168 | - // Confirm problem is gone. |
|
169 | - $this->assertStringNotContainsString('class BarcodeException extends \Mpdf\MpdfException', $mpdf_php); |
|
168 | + // Confirm problem is gone. |
|
169 | + $this->assertStringNotContainsString('class BarcodeException extends \Mpdf\MpdfException', $mpdf_php); |
|
170 | 170 | |
171 | - // Confirm solution is correct. |
|
172 | - $this->assertStringContainsString('class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException', $mpdf_php); |
|
173 | - } |
|
171 | + // Confirm solution is correct. |
|
172 | + $this->assertStringContainsString('class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException', $mpdf_php); |
|
173 | + } |
|
174 | 174 | } |
@@ -49,26 +49,26 @@ discard block |
||
49 | 49 | } |
50 | 50 | EOD; |
51 | 51 | |
52 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
52 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
53 | 53 | |
54 | - chdir($this->testsWorkingDir); |
|
54 | + chdir( $this->testsWorkingDir ); |
|
55 | 55 | |
56 | - exec('composer install'); |
|
56 | + exec( 'composer install' ); |
|
57 | 57 | |
58 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
59 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
58 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
59 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
60 | 60 | |
61 | 61 | $mozartCompose = new Compose(); |
62 | 62 | |
63 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
63 | + $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
64 | 64 | |
65 | - $mpdf_php = file_get_contents($this->testsWorkingDir .'strauss/mpdf/mpdf/src/Mpdf.php'); |
|
65 | + $mpdf_php = file_get_contents( $this->testsWorkingDir . 'strauss/mpdf/mpdf/src/Mpdf.php' ); |
|
66 | 66 | |
67 | 67 | // Confirm problem is gone. |
68 | - $this->assertStringNotContainsString('class BrianHenryIE\Strauss\Mpdf implements', $mpdf_php); |
|
68 | + $this->assertStringNotContainsString( 'class BrianHenryIE\Strauss\Mpdf implements', $mpdf_php ); |
|
69 | 69 | |
70 | 70 | // Confirm solution is correct. |
71 | - $this->assertStringContainsString('class Mpdf implements', $mpdf_php); |
|
71 | + $this->assertStringContainsString( 'class Mpdf implements', $mpdf_php ); |
|
72 | 72 | } |
73 | 73 | |
74 | 74 | |
@@ -101,26 +101,26 @@ discard block |
||
101 | 101 | } |
102 | 102 | EOD; |
103 | 103 | |
104 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
104 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
105 | 105 | |
106 | - chdir($this->testsWorkingDir); |
|
106 | + chdir( $this->testsWorkingDir ); |
|
107 | 107 | |
108 | - exec('composer install'); |
|
108 | + exec( 'composer install' ); |
|
109 | 109 | |
110 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
111 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
110 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
111 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
112 | 112 | |
113 | 113 | $mozartCompose = new Compose(); |
114 | 114 | |
115 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
115 | + $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
116 | 116 | |
117 | - $mpdf_php = file_get_contents($this->testsWorkingDir .'/strauss/mpdf/mpdf/src/Conversion/DecToOther.php'); |
|
117 | + $mpdf_php = file_get_contents( $this->testsWorkingDir . '/strauss/mpdf/mpdf/src/Conversion/DecToOther.php' ); |
|
118 | 118 | |
119 | 119 | // Confirm problem is gone. |
120 | - $this->assertStringNotContainsString('public function __construct(BrianHenryIE\Strauss\Mpdf $mpdf)', $mpdf_php); |
|
120 | + $this->assertStringNotContainsString( 'public function __construct(BrianHenryIE\Strauss\Mpdf $mpdf)', $mpdf_php ); |
|
121 | 121 | |
122 | 122 | // Confirm solution is correct. |
123 | - $this->assertStringContainsString('public function __construct(Mpdf $mpdf)', $mpdf_php); |
|
123 | + $this->assertStringContainsString( 'public function __construct(Mpdf $mpdf)', $mpdf_php ); |
|
124 | 124 | } |
125 | 125 | // src/strauss/mpdf/mpdf/src/Barcode/BarcodeException.php |
126 | 126 | |
@@ -150,25 +150,25 @@ discard block |
||
150 | 150 | } |
151 | 151 | EOD; |
152 | 152 | |
153 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
153 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
154 | 154 | |
155 | - chdir($this->testsWorkingDir); |
|
155 | + chdir( $this->testsWorkingDir ); |
|
156 | 156 | |
157 | - exec('composer install'); |
|
157 | + exec( 'composer install' ); |
|
158 | 158 | |
159 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
160 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
159 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
160 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
161 | 161 | |
162 | 162 | $mozartCompose = new Compose(); |
163 | 163 | |
164 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
164 | + $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
165 | 165 | |
166 | - $mpdf_php = file_get_contents($this->testsWorkingDir .'/strauss/mpdf/mpdf/src/Barcode/BarcodeException.php'); |
|
166 | + $mpdf_php = file_get_contents( $this->testsWorkingDir . '/strauss/mpdf/mpdf/src/Barcode/BarcodeException.php' ); |
|
167 | 167 | |
168 | 168 | // Confirm problem is gone. |
169 | - $this->assertStringNotContainsString('class BarcodeException extends \Mpdf\MpdfException', $mpdf_php); |
|
169 | + $this->assertStringNotContainsString( 'class BarcodeException extends \Mpdf\MpdfException', $mpdf_php ); |
|
170 | 170 | |
171 | 171 | // Confirm solution is correct. |
172 | - $this->assertStringContainsString('class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException', $mpdf_php); |
|
172 | + $this->assertStringContainsString( 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException', $mpdf_php ); |
|
173 | 173 | } |
174 | 174 | } |
@@ -17,8 +17,7 @@ discard block |
||
17 | 17 | * Class MozartIssue124Test |
18 | 18 | * @coversNothing |
19 | 19 | */ |
20 | -class MozartIssue124Test extends IntegrationTestCase |
|
21 | -{ |
|
20 | +class MozartIssue124Test extends IntegrationTestCase { |
|
22 | 21 | |
23 | 22 | /** |
24 | 23 | * After PR #84, running Mozart on Mpdf began prefixing the class name inside the namespaced file. |
@@ -31,8 +30,7 @@ discard block |
||
31 | 30 | * |
32 | 31 | * Should be: "class Mpdf implements" because its namespace has already been prefixed. |
33 | 32 | */ |
34 | - public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
35 | - { |
|
33 | + public function test_it_does_not_make_classname_replacement_inside_namespaced_file() { |
|
36 | 34 | |
37 | 35 | $composerJsonString = <<<'EOD' |
38 | 36 | { |
@@ -82,8 +80,7 @@ discard block |
||
82 | 80 | * |
83 | 81 | * @see https://github.com/coenjacobs/mozart/issues/124 |
84 | 82 | */ |
85 | - public function test_it_does_not_prefix_function_argument_types_whose_classname_matches_the_namespace() |
|
86 | - { |
|
83 | + public function test_it_does_not_prefix_function_argument_types_whose_classname_matches_the_namespace() { |
|
87 | 84 | |
88 | 85 | |
89 | 86 | $composerJsonString = <<<'EOD' |
@@ -131,8 +128,7 @@ discard block |
||
131 | 128 | * |
132 | 129 | * @see mpdf/mpdf/src/Barcode/BarcodeException.php |
133 | 130 | */ |
134 | - public function testItDoesPrefixNamespacedExtends() |
|
135 | - { |
|
131 | + public function testItDoesPrefixNamespacedExtends() { |
|
136 | 132 | |
137 | 133 | |
138 | 134 | $composerJsonString = <<<'EOD' |
@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | } |
48 | 48 | } |
49 | 49 | } |
50 | -EOD; |
|
50 | +eod; |
|
51 | 51 | |
52 | 52 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
53 | 53 | |
@@ -99,7 +99,7 @@ discard block |
||
99 | 99 | } |
100 | 100 | } |
101 | 101 | } |
102 | -EOD; |
|
102 | +eod; |
|
103 | 103 | |
104 | 104 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
105 | 105 | |
@@ -148,7 +148,7 @@ discard block |
||
148 | 148 | } |
149 | 149 | } |
150 | 150 | } |
151 | -EOD; |
|
151 | +eod; |
|
152 | 152 | |
153 | 153 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
154 | 154 |
@@ -22,10 +22,10 @@ discard block |
||
22 | 22 | class MozartIssue109Test extends IntegrationTestCase |
23 | 23 | { |
24 | 24 | |
25 | - public function testTheOutputDoesNotPrefixComments() |
|
26 | - { |
|
25 | + public function testTheOutputDoesNotPrefixComments() |
|
26 | + { |
|
27 | 27 | |
28 | - $composerJsonString = <<<'EOD' |
|
28 | + $composerJsonString = <<<'EOD' |
|
29 | 29 | { |
30 | 30 | "minimum-stability": "dev", |
31 | 31 | "require": { |
@@ -52,21 +52,21 @@ discard block |
||
52 | 52 | } |
53 | 53 | EOD; |
54 | 54 | |
55 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
55 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
56 | 56 | |
57 | - chdir($this->testsWorkingDir); |
|
57 | + chdir($this->testsWorkingDir); |
|
58 | 58 | |
59 | - exec('composer install'); |
|
59 | + exec('composer install'); |
|
60 | 60 | |
61 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
62 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
61 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
62 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
63 | 63 | |
64 | - $mozartCompose = new Compose(); |
|
64 | + $mozartCompose = new Compose(); |
|
65 | 65 | |
66 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
66 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
67 | 67 | |
68 | - $phpString = file_get_contents($this->testsWorkingDir .'strauss/nesbot/carbon/src/Carbon/Carbon.php'); |
|
68 | + $phpString = file_get_contents($this->testsWorkingDir .'strauss/nesbot/carbon/src/Carbon/Carbon.php'); |
|
69 | 69 | |
70 | - $this->assertStringNotContainsString('*Mozart\\ This file is part of the Carbon package.Mozart\\', $phpString); |
|
71 | - } |
|
70 | + $this->assertStringNotContainsString('*Mozart\\ This file is part of the Carbon package.Mozart\\', $phpString); |
|
71 | + } |
|
72 | 72 | } |
@@ -52,21 +52,21 @@ |
||
52 | 52 | } |
53 | 53 | EOD; |
54 | 54 | |
55 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
55 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
56 | 56 | |
57 | - chdir($this->testsWorkingDir); |
|
57 | + chdir( $this->testsWorkingDir ); |
|
58 | 58 | |
59 | - exec('composer install'); |
|
59 | + exec( 'composer install' ); |
|
60 | 60 | |
61 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
62 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
61 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
62 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
63 | 63 | |
64 | 64 | $mozartCompose = new Compose(); |
65 | 65 | |
66 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
66 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
67 | 67 | |
68 | - $phpString = file_get_contents($this->testsWorkingDir .'strauss/nesbot/carbon/src/Carbon/Carbon.php'); |
|
68 | + $phpString = file_get_contents( $this->testsWorkingDir . 'strauss/nesbot/carbon/src/Carbon/Carbon.php' ); |
|
69 | 69 | |
70 | - $this->assertStringNotContainsString('*Mozart\\ This file is part of the Carbon package.Mozart\\', $phpString); |
|
70 | + $this->assertStringNotContainsString( '*Mozart\\ This file is part of the Carbon package.Mozart\\', $phpString ); |
|
71 | 71 | } |
72 | 72 | } |
@@ -19,11 +19,9 @@ |
||
19 | 19 | * @package BrianHenryIE\Strauss\Tests\Issues |
20 | 20 | * @coversNothing |
21 | 21 | */ |
22 | -class MozartIssue109Test extends IntegrationTestCase |
|
23 | -{ |
|
22 | +class MozartIssue109Test extends IntegrationTestCase { |
|
24 | 23 | |
25 | - public function testTheOutputDoesNotPrefixComments() |
|
26 | - { |
|
24 | + public function testTheOutputDoesNotPrefixComments() { |
|
27 | 25 | |
28 | 26 | $composerJsonString = <<<'EOD' |
29 | 27 | { |
@@ -36,7 +36,7 @@ |
||
36 | 36 | } |
37 | 37 | } |
38 | 38 | } |
39 | -EOD; |
|
39 | +eod; |
|
40 | 40 | |
41 | 41 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
42 | 42 |
@@ -18,50 +18,50 @@ discard block |
||
18 | 18 | class MozartIssue89Test extends IntegrationTestCase |
19 | 19 | { |
20 | 20 | |
21 | - /** |
|
22 | - * If a file is specified more than once in an autoloader, e.g. is explicitly listed and is also in a folder listed, |
|
23 | - * a "File already exists at path" error occurs. |
|
24 | - * |
|
25 | - * To fix this, we enumerate the files to be copied using a dictionary indexed with the source file path, then loop |
|
26 | - * and copy, thus only copying each one once. |
|
27 | - * |
|
28 | - * Original error: |
|
29 | - * "League\Flysystem\FileExistsException : File already exists at path: lib/classes/tecnickcom/tcpdf/tcpdf.php" |
|
30 | - * |
|
31 | - * Test is using a known problematic autoloader: |
|
32 | - * "iio/libmergepdf": { |
|
33 | - * "classmap": [ |
|
34 | - * "config", |
|
35 | - * "include", |
|
36 | - * "tcpdf.php", |
|
37 | - * "tcpdf_parser.php", |
|
38 | - * "tcpdf_import.php", |
|
39 | - * "tcpdf_barcodes_1d.php", |
|
40 | - * "tcpdf_barcodes_2d.php", |
|
41 | - * "include/tcpdf_colors.php", |
|
42 | - * "include/tcpdf_filters.php", |
|
43 | - * "include/tcpdf_font_data.php", |
|
44 | - * "include/tcpdf_fonts.php", |
|
45 | - * "include/tcpdf_images.php", |
|
46 | - * "include/tcpdf_static.php", |
|
47 | - * "include/barcodes/datamatrix.php", |
|
48 | - * "include/barcodes/pdf417.php", |
|
49 | - * "include/barcodes/qrcode.php" |
|
50 | - * ] |
|
51 | - * } |
|
52 | - * |
|
53 | - * @see https://github.com/coenjacobs/mozart/issues/89 |
|
54 | - * |
|
55 | - * @test |
|
56 | - */ |
|
57 | - public function it_moves_each_file_once_per_namespace() |
|
58 | - { |
|
59 | - |
|
60 | - if (version_compare(phpversion(), '8.0.0', '>=')) { |
|
61 | - $this->markTestSkipped("Package specified for test is not PHP 8 compatible"); |
|
62 | - } |
|
63 | - |
|
64 | - $composerJsonString = <<<'EOD' |
|
21 | + /** |
|
22 | + * If a file is specified more than once in an autoloader, e.g. is explicitly listed and is also in a folder listed, |
|
23 | + * a "File already exists at path" error occurs. |
|
24 | + * |
|
25 | + * To fix this, we enumerate the files to be copied using a dictionary indexed with the source file path, then loop |
|
26 | + * and copy, thus only copying each one once. |
|
27 | + * |
|
28 | + * Original error: |
|
29 | + * "League\Flysystem\FileExistsException : File already exists at path: lib/classes/tecnickcom/tcpdf/tcpdf.php" |
|
30 | + * |
|
31 | + * Test is using a known problematic autoloader: |
|
32 | + * "iio/libmergepdf": { |
|
33 | + * "classmap": [ |
|
34 | + * "config", |
|
35 | + * "include", |
|
36 | + * "tcpdf.php", |
|
37 | + * "tcpdf_parser.php", |
|
38 | + * "tcpdf_import.php", |
|
39 | + * "tcpdf_barcodes_1d.php", |
|
40 | + * "tcpdf_barcodes_2d.php", |
|
41 | + * "include/tcpdf_colors.php", |
|
42 | + * "include/tcpdf_filters.php", |
|
43 | + * "include/tcpdf_font_data.php", |
|
44 | + * "include/tcpdf_fonts.php", |
|
45 | + * "include/tcpdf_images.php", |
|
46 | + * "include/tcpdf_static.php", |
|
47 | + * "include/barcodes/datamatrix.php", |
|
48 | + * "include/barcodes/pdf417.php", |
|
49 | + * "include/barcodes/qrcode.php" |
|
50 | + * ] |
|
51 | + * } |
|
52 | + * |
|
53 | + * @see https://github.com/coenjacobs/mozart/issues/89 |
|
54 | + * |
|
55 | + * @test |
|
56 | + */ |
|
57 | + public function it_moves_each_file_once_per_namespace() |
|
58 | + { |
|
59 | + |
|
60 | + if (version_compare(phpversion(), '8.0.0', '>=')) { |
|
61 | + $this->markTestSkipped("Package specified for test is not PHP 8 compatible"); |
|
62 | + } |
|
63 | + |
|
64 | + $composerJsonString = <<<'EOD' |
|
65 | 65 | { |
66 | 66 | "name": "brianhenryie/mozart-issue-89", |
67 | 67 | "require": { |
@@ -76,30 +76,30 @@ discard block |
||
76 | 76 | } |
77 | 77 | EOD; |
78 | 78 | |
79 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
79 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
80 | 80 | |
81 | - chdir($this->testsWorkingDir); |
|
81 | + chdir($this->testsWorkingDir); |
|
82 | 82 | |
83 | - exec('composer install'); |
|
83 | + exec('composer install'); |
|
84 | 84 | |
85 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
86 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
85 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
86 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
87 | 87 | |
88 | - $mozartCompose = new Compose(); |
|
88 | + $mozartCompose = new Compose(); |
|
89 | 89 | |
90 | - // $this->expectException(League\Flysystem\FileExistsException::class); |
|
90 | + // $this->expectException(League\Flysystem\FileExistsException::class); |
|
91 | 91 | |
92 | - $exception = null; |
|
92 | + $exception = null; |
|
93 | 93 | |
94 | - try { |
|
95 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
96 | - } catch (\League\Flysystem\FileExistsException $e) { |
|
97 | - $exception = $e; |
|
98 | - } |
|
94 | + try { |
|
95 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
96 | + } catch (\League\Flysystem\FileExistsException $e) { |
|
97 | + $exception = $e; |
|
98 | + } |
|
99 | 99 | |
100 | - // On the failing test, an exception was thrown and this line was not reached. |
|
101 | - $this->assertEquals(0, $result); |
|
100 | + // On the failing test, an exception was thrown and this line was not reached. |
|
101 | + $this->assertEquals(0, $result); |
|
102 | 102 | |
103 | - $this->assertNull($exception); |
|
104 | - } |
|
103 | + $this->assertNull($exception); |
|
104 | + } |
|
105 | 105 | } |
@@ -57,8 +57,8 @@ discard block |
||
57 | 57 | public function it_moves_each_file_once_per_namespace() |
58 | 58 | { |
59 | 59 | |
60 | - if (version_compare(phpversion(), '8.0.0', '>=')) { |
|
61 | - $this->markTestSkipped("Package specified for test is not PHP 8 compatible"); |
|
60 | + if ( version_compare( phpversion(), '8.0.0', '>=' ) ) { |
|
61 | + $this->markTestSkipped( "Package specified for test is not PHP 8 compatible" ); |
|
62 | 62 | } |
63 | 63 | |
64 | 64 | $composerJsonString = <<<'EOD' |
@@ -76,14 +76,14 @@ discard block |
||
76 | 76 | } |
77 | 77 | EOD; |
78 | 78 | |
79 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
79 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
80 | 80 | |
81 | - chdir($this->testsWorkingDir); |
|
81 | + chdir( $this->testsWorkingDir ); |
|
82 | 82 | |
83 | - exec('composer install'); |
|
83 | + exec( 'composer install' ); |
|
84 | 84 | |
85 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
86 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
85 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
86 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
87 | 87 | |
88 | 88 | $mozartCompose = new Compose(); |
89 | 89 | |
@@ -92,14 +92,14 @@ discard block |
||
92 | 92 | $exception = null; |
93 | 93 | |
94 | 94 | try { |
95 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
96 | - } catch (\League\Flysystem\FileExistsException $e) { |
|
97 | - $exception = $e; |
|
95 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
96 | + } catch ( \League\Flysystem\FileExistsException $e ) { |
|
97 | + $exception = $e; |
|
98 | 98 | } |
99 | 99 | |
100 | 100 | // On the failing test, an exception was thrown and this line was not reached. |
101 | - $this->assertEquals(0, $result); |
|
101 | + $this->assertEquals( 0, $result ); |
|
102 | 102 | |
103 | - $this->assertNull($exception); |
|
103 | + $this->assertNull( $exception ); |
|
104 | 104 | } |
105 | 105 | } |
@@ -15,8 +15,7 @@ discard block |
||
15 | 15 | * @package BrianHenryIE\Strauss\Tests\Issues |
16 | 16 | * @coversNothing |
17 | 17 | */ |
18 | -class MozartIssue89Test extends IntegrationTestCase |
|
19 | -{ |
|
18 | +class MozartIssue89Test extends IntegrationTestCase { |
|
20 | 19 | |
21 | 20 | /** |
22 | 21 | * If a file is specified more than once in an autoloader, e.g. is explicitly listed and is also in a folder listed, |
@@ -54,8 +53,7 @@ discard block |
||
54 | 53 | * |
55 | 54 | * @test |
56 | 55 | */ |
57 | - public function it_moves_each_file_once_per_namespace() |
|
58 | - { |
|
56 | + public function it_moves_each_file_once_per_namespace() { |
|
59 | 57 | |
60 | 58 | if (version_compare(phpversion(), '8.0.0', '>=')) { |
61 | 59 | $this->markTestSkipped("Package specified for test is not PHP 8 compatible"); |
@@ -74,7 +74,7 @@ |
||
74 | 74 | } |
75 | 75 | } |
76 | 76 | } |
77 | -EOD; |
|
77 | +eod; |
|
78 | 78 | |
79 | 79 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
80 | 80 |
@@ -21,17 +21,17 @@ discard block |
||
21 | 21 | class MozartIssue13Test extends IntegrationTestCase |
22 | 22 | { |
23 | 23 | |
24 | - /** |
|
25 | - * |
|
26 | - * "paypal/rest-api-sdk-php" |
|
27 | - * |
|
28 | - */ |
|
29 | - public function testPaypalStringReplacement() |
|
30 | - { |
|
24 | + /** |
|
25 | + * |
|
26 | + * "paypal/rest-api-sdk-php" |
|
27 | + * |
|
28 | + */ |
|
29 | + public function testPaypalStringReplacement() |
|
30 | + { |
|
31 | 31 | |
32 | 32 | // $this->markTestSkipped('This test was passing until I excluded the PSR namespace'); |
33 | 33 | |
34 | - $composerJsonString = <<<'EOD' |
|
34 | + $composerJsonString = <<<'EOD' |
|
35 | 35 | { |
36 | 36 | "name": "brianhenryie/mozart-issue-13", |
37 | 37 | "require": { |
@@ -50,22 +50,22 @@ discard block |
||
50 | 50 | } |
51 | 51 | EOD; |
52 | 52 | |
53 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
53 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
54 | 54 | |
55 | - chdir($this->testsWorkingDir); |
|
55 | + chdir($this->testsWorkingDir); |
|
56 | 56 | |
57 | - exec('composer install'); |
|
57 | + exec('composer install'); |
|
58 | 58 | |
59 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
60 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
59 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
60 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
61 | 61 | |
62 | - $mozartCompose = new Compose(); |
|
62 | + $mozartCompose = new Compose(); |
|
63 | 63 | |
64 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
64 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
65 | 65 | |
66 | - $phpString = file_get_contents($this->testsWorkingDir .'/strauss/paypal/rest-api-sdk-php/lib/PayPal/Log/PayPalLogger.php'); |
|
66 | + $phpString = file_get_contents($this->testsWorkingDir .'/strauss/paypal/rest-api-sdk-php/lib/PayPal/Log/PayPalLogger.php'); |
|
67 | 67 | |
68 | - // Confirm solution is correct. |
|
69 | - $this->assertStringContainsString('constant("\\\\BrianHenryIE\\\\Strauss\\\\Psr\\\\Log\\\\LogLevel::$loggingLevel")', $phpString); |
|
70 | - } |
|
68 | + // Confirm solution is correct. |
|
69 | + $this->assertStringContainsString('constant("\\\\BrianHenryIE\\\\Strauss\\\\Psr\\\\Log\\\\LogLevel::$loggingLevel")', $phpString); |
|
70 | + } |
|
71 | 71 | } |
@@ -50,22 +50,22 @@ |
||
50 | 50 | } |
51 | 51 | EOD; |
52 | 52 | |
53 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
53 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
54 | 54 | |
55 | - chdir($this->testsWorkingDir); |
|
55 | + chdir( $this->testsWorkingDir ); |
|
56 | 56 | |
57 | - exec('composer install'); |
|
57 | + exec( 'composer install' ); |
|
58 | 58 | |
59 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
60 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
59 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
60 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
61 | 61 | |
62 | 62 | $mozartCompose = new Compose(); |
63 | 63 | |
64 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
64 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
65 | 65 | |
66 | - $phpString = file_get_contents($this->testsWorkingDir .'/strauss/paypal/rest-api-sdk-php/lib/PayPal/Log/PayPalLogger.php'); |
|
66 | + $phpString = file_get_contents( $this->testsWorkingDir . '/strauss/paypal/rest-api-sdk-php/lib/PayPal/Log/PayPalLogger.php' ); |
|
67 | 67 | |
68 | 68 | // Confirm solution is correct. |
69 | - $this->assertStringContainsString('constant("\\\\BrianHenryIE\\\\Strauss\\\\Psr\\\\Log\\\\LogLevel::$loggingLevel")', $phpString); |
|
69 | + $this->assertStringContainsString( 'constant("\\\\BrianHenryIE\\\\Strauss\\\\Psr\\\\Log\\\\LogLevel::$loggingLevel")', $phpString ); |
|
70 | 70 | } |
71 | 71 | } |
@@ -18,16 +18,14 @@ |
||
18 | 18 | * @package BrianHenryIE\Strauss\Tests\Issues |
19 | 19 | * @coversNothing |
20 | 20 | */ |
21 | -class MozartIssue13Test extends IntegrationTestCase |
|
22 | -{ |
|
21 | +class MozartIssue13Test extends IntegrationTestCase { |
|
23 | 22 | |
24 | 23 | /** |
25 | 24 | * |
26 | 25 | * "paypal/rest-api-sdk-php" |
27 | 26 | * |
28 | 27 | */ |
29 | - public function testPaypalStringReplacement() |
|
30 | - { |
|
28 | + public function testPaypalStringReplacement() { |
|
31 | 29 | |
32 | 30 | // $this->markTestSkipped('This test was passing until I excluded the PSR namespace'); |
33 | 31 |
@@ -74,7 +74,7 @@ |
||
74 | 74 | } |
75 | 75 | } |
76 | 76 | } |
77 | -EOD; |
|
77 | +eod; |
|
78 | 78 | |
79 | 79 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
80 | 80 |
@@ -17,15 +17,15 @@ discard block |
||
17 | 17 | class StraussIssue14Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
18 | 18 | { |
19 | 19 | |
20 | - /** |
|
21 | - * Looks like the exclude_from_prefix regex for psr is not specific enough. |
|
22 | - * |
|
23 | - * @author BrianHenryIE |
|
24 | - */ |
|
25 | - public function test_guzzle_http_is_prefixed() |
|
26 | - { |
|
27 | - |
|
28 | - $composerJsonString = <<<'EOD' |
|
20 | + /** |
|
21 | + * Looks like the exclude_from_prefix regex for psr is not specific enough. |
|
22 | + * |
|
23 | + * @author BrianHenryIE |
|
24 | + */ |
|
25 | + public function test_guzzle_http_is_prefixed() |
|
26 | + { |
|
27 | + |
|
28 | + $composerJsonString = <<<'EOD' |
|
29 | 29 | { |
30 | 30 | "name": "brianhenryie/strauss-issue-14", |
31 | 31 | "require":{ |
@@ -42,31 +42,31 @@ discard block |
||
42 | 42 | } |
43 | 43 | EOD; |
44 | 44 | |
45 | - file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
45 | + file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
46 | 46 | |
47 | - chdir($this->testsWorkingDir); |
|
47 | + chdir($this->testsWorkingDir); |
|
48 | 48 | |
49 | - exec('composer install'); |
|
49 | + exec('composer install'); |
|
50 | 50 | |
51 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
52 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
51 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
52 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
53 | 53 | |
54 | - $mozartCompose = new Compose(); |
|
54 | + $mozartCompose = new Compose(); |
|
55 | 55 | |
56 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
56 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
57 | 57 | |
58 | - $php_string = file_get_contents($this->testsWorkingDir . '/strauss/guzzlehttp/psr7/src/AppendStream.php'); |
|
58 | + $php_string = file_get_contents($this->testsWorkingDir . '/strauss/guzzlehttp/psr7/src/AppendStream.php'); |
|
59 | 59 | |
60 | - // was namespace GuzzleHttp\Psr7; |
|
60 | + // was namespace GuzzleHttp\Psr7; |
|
61 | 61 | |
62 | - // Confirm solution is correct. |
|
63 | - $this->assertStringContainsString('namespace BrianHenryIE\Strauss\GuzzleHttp\Psr7;', $php_string); |
|
64 | - } |
|
62 | + // Confirm solution is correct. |
|
63 | + $this->assertStringContainsString('namespace BrianHenryIE\Strauss\GuzzleHttp\Psr7;', $php_string); |
|
64 | + } |
|
65 | 65 | |
66 | - public function testFilesAutoloaderIsGenerated() |
|
67 | - { |
|
66 | + public function testFilesAutoloaderIsGenerated() |
|
67 | + { |
|
68 | 68 | |
69 | - $composerJsonString = <<<'EOD' |
|
69 | + $composerJsonString = <<<'EOD' |
|
70 | 70 | { |
71 | 71 | "name": "brianhenryie/strauss-issue-14", |
72 | 72 | "require":{ |
@@ -83,19 +83,19 @@ discard block |
||
83 | 83 | } |
84 | 84 | EOD; |
85 | 85 | |
86 | - file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
86 | + file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
87 | 87 | |
88 | - chdir($this->testsWorkingDir); |
|
88 | + chdir($this->testsWorkingDir); |
|
89 | 89 | |
90 | - exec('composer install'); |
|
90 | + exec('composer install'); |
|
91 | 91 | |
92 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
93 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
92 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
93 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
94 | 94 | |
95 | - $mozartCompose = new Compose(); |
|
95 | + $mozartCompose = new Compose(); |
|
96 | 96 | |
97 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
97 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
98 | 98 | |
99 | - $this->assertFileExists($this->testsWorkingDir . '/strauss/autoload-files.php'); |
|
100 | - } |
|
99 | + $this->assertFileExists($this->testsWorkingDir . '/strauss/autoload-files.php'); |
|
100 | + } |
|
101 | 101 | } |
@@ -42,25 +42,25 @@ discard block |
||
42 | 42 | } |
43 | 43 | EOD; |
44 | 44 | |
45 | - file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
45 | + file_put_contents( $this->testsWorkingDir . 'composer.json', $composerJsonString ); |
|
46 | 46 | |
47 | - chdir($this->testsWorkingDir); |
|
47 | + chdir( $this->testsWorkingDir ); |
|
48 | 48 | |
49 | - exec('composer install'); |
|
49 | + exec( 'composer install' ); |
|
50 | 50 | |
51 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
52 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
51 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
52 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
53 | 53 | |
54 | 54 | $mozartCompose = new Compose(); |
55 | 55 | |
56 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
56 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
57 | 57 | |
58 | - $php_string = file_get_contents($this->testsWorkingDir . '/strauss/guzzlehttp/psr7/src/AppendStream.php'); |
|
58 | + $php_string = file_get_contents( $this->testsWorkingDir . '/strauss/guzzlehttp/psr7/src/AppendStream.php' ); |
|
59 | 59 | |
60 | 60 | // was namespace GuzzleHttp\Psr7; |
61 | 61 | |
62 | 62 | // Confirm solution is correct. |
63 | - $this->assertStringContainsString('namespace BrianHenryIE\Strauss\GuzzleHttp\Psr7;', $php_string); |
|
63 | + $this->assertStringContainsString( 'namespace BrianHenryIE\Strauss\GuzzleHttp\Psr7;', $php_string ); |
|
64 | 64 | } |
65 | 65 | |
66 | 66 | public function testFilesAutoloaderIsGenerated() |
@@ -83,19 +83,19 @@ discard block |
||
83 | 83 | } |
84 | 84 | EOD; |
85 | 85 | |
86 | - file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
86 | + file_put_contents( $this->testsWorkingDir . 'composer.json', $composerJsonString ); |
|
87 | 87 | |
88 | - chdir($this->testsWorkingDir); |
|
88 | + chdir( $this->testsWorkingDir ); |
|
89 | 89 | |
90 | - exec('composer install'); |
|
90 | + exec( 'composer install' ); |
|
91 | 91 | |
92 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
93 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
92 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
93 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
94 | 94 | |
95 | 95 | $mozartCompose = new Compose(); |
96 | 96 | |
97 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
97 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
98 | 98 | |
99 | - $this->assertFileExists($this->testsWorkingDir . '/strauss/autoload-files.php'); |
|
99 | + $this->assertFileExists( $this->testsWorkingDir . '/strauss/autoload-files.php' ); |
|
100 | 100 | } |
101 | 101 | } |
@@ -14,16 +14,14 @@ discard block |
||
14 | 14 | * @package BrianHenryIE\Strauss\Tests\Issues |
15 | 15 | * @coversNothing |
16 | 16 | */ |
17 | -class StraussIssue14Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
|
18 | -{ |
|
17 | +class StraussIssue14Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase { |
|
19 | 18 | |
20 | 19 | /** |
21 | 20 | * Looks like the exclude_from_prefix regex for psr is not specific enough. |
22 | 21 | * |
23 | 22 | * @author BrianHenryIE |
24 | 23 | */ |
25 | - public function test_guzzle_http_is_prefixed() |
|
26 | - { |
|
24 | + public function test_guzzle_http_is_prefixed() { |
|
27 | 25 | |
28 | 26 | $composerJsonString = <<<'EOD' |
29 | 27 | { |
@@ -63,8 +61,7 @@ discard block |
||
63 | 61 | $this->assertStringContainsString('namespace BrianHenryIE\Strauss\GuzzleHttp\Psr7;', $php_string); |
64 | 62 | } |
65 | 63 | |
66 | - public function testFilesAutoloaderIsGenerated() |
|
67 | - { |
|
64 | + public function testFilesAutoloaderIsGenerated() { |
|
68 | 65 | |
69 | 66 | $composerJsonString = <<<'EOD' |
70 | 67 | { |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | } |
41 | 41 | } |
42 | 42 | } |
43 | -EOD; |
|
43 | +eod; |
|
44 | 44 | |
45 | 45 | file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
46 | 46 | |
@@ -81,7 +81,7 @@ discard block |
||
81 | 81 | } |
82 | 82 | } |
83 | 83 | } |
84 | -EOD; |
|
84 | +eod; |
|
85 | 85 | |
86 | 86 | file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
87 | 87 |