@@ -18,15 +18,15 @@ discard block |
||
18 | 18 | class MozartIssue90Test extends IntegrationTestCase |
19 | 19 | { |
20 | 20 | |
21 | - /** |
|
22 | - * Issue 90. Needs "iio/libmergepdf". |
|
23 | - * |
|
24 | - * Error: "File already exists at path: classmap_directory/tecnickcom/tcpdf/tcpdf.php". |
|
25 | - */ |
|
26 | - public function testLibpdfmergeSucceeds() |
|
27 | - { |
|
21 | + /** |
|
22 | + * Issue 90. Needs "iio/libmergepdf". |
|
23 | + * |
|
24 | + * Error: "File already exists at path: classmap_directory/tecnickcom/tcpdf/tcpdf.php". |
|
25 | + */ |
|
26 | + public function testLibpdfmergeSucceeds() |
|
27 | + { |
|
28 | 28 | |
29 | - $composerJsonString = <<<'EOD' |
|
29 | + $composerJsonString = <<<'EOD' |
|
30 | 30 | { |
31 | 31 | "name": "brianhenryie/mozart-issue-90", |
32 | 32 | "require": { |
@@ -41,24 +41,24 @@ discard block |
||
41 | 41 | } |
42 | 42 | EOD; |
43 | 43 | |
44 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
44 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
45 | 45 | |
46 | - chdir($this->testsWorkingDir); |
|
46 | + chdir($this->testsWorkingDir); |
|
47 | 47 | |
48 | - exec('composer install'); |
|
48 | + exec('composer install'); |
|
49 | 49 | |
50 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
51 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
50 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
51 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
52 | 52 | |
53 | - $mozartCompose = new Compose(); |
|
53 | + $mozartCompose = new Compose(); |
|
54 | 54 | |
55 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
55 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
56 | 56 | |
57 | - $this->assertEquals(0, $result); |
|
57 | + $this->assertEquals(0, $result); |
|
58 | 58 | |
59 | - // This test would only fail on Windows? |
|
60 | - $this->assertDirectoryDoesNotExist($this->testsWorkingDir .'strauss/iio/libmergepdf/vendor/iio/libmergepdf/tcpdi'); |
|
59 | + // This test would only fail on Windows? |
|
60 | + $this->assertDirectoryDoesNotExist($this->testsWorkingDir .'strauss/iio/libmergepdf/vendor/iio/libmergepdf/tcpdi'); |
|
61 | 61 | |
62 | - $this->assertFileExists($this->testsWorkingDir .'/strauss/iio/libmergepdf/tcpdi/tcpdi.php'); |
|
63 | - } |
|
62 | + $this->assertFileExists($this->testsWorkingDir .'/strauss/iio/libmergepdf/tcpdi/tcpdi.php'); |
|
63 | + } |
|
64 | 64 | } |
@@ -41,24 +41,24 @@ |
||
41 | 41 | } |
42 | 42 | EOD; |
43 | 43 | |
44 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
44 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
45 | 45 | |
46 | - chdir($this->testsWorkingDir); |
|
46 | + chdir( $this->testsWorkingDir ); |
|
47 | 47 | |
48 | - exec('composer install'); |
|
48 | + exec( 'composer install' ); |
|
49 | 49 | |
50 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
51 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
50 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
51 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
52 | 52 | |
53 | 53 | $mozartCompose = new Compose(); |
54 | 54 | |
55 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
55 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
56 | 56 | |
57 | - $this->assertEquals(0, $result); |
|
57 | + $this->assertEquals( 0, $result ); |
|
58 | 58 | |
59 | 59 | // This test would only fail on Windows? |
60 | - $this->assertDirectoryDoesNotExist($this->testsWorkingDir .'strauss/iio/libmergepdf/vendor/iio/libmergepdf/tcpdi'); |
|
60 | + $this->assertDirectoryDoesNotExist( $this->testsWorkingDir . 'strauss/iio/libmergepdf/vendor/iio/libmergepdf/tcpdi' ); |
|
61 | 61 | |
62 | - $this->assertFileExists($this->testsWorkingDir .'/strauss/iio/libmergepdf/tcpdi/tcpdi.php'); |
|
62 | + $this->assertFileExists( $this->testsWorkingDir . '/strauss/iio/libmergepdf/tcpdi/tcpdi.php' ); |
|
63 | 63 | } |
64 | 64 | } |
@@ -15,16 +15,14 @@ |
||
15 | 15 | * @package BrianHenryIE\Strauss\Tests\Issues |
16 | 16 | * @coversNothing |
17 | 17 | */ |
18 | -class MozartIssue90Test extends IntegrationTestCase |
|
19 | -{ |
|
18 | +class MozartIssue90Test extends IntegrationTestCase { |
|
20 | 19 | |
21 | 20 | /** |
22 | 21 | * Issue 90. Needs "iio/libmergepdf". |
23 | 22 | * |
24 | 23 | * Error: "File already exists at path: classmap_directory/tecnickcom/tcpdf/tcpdf.php". |
25 | 24 | */ |
26 | - public function testLibpdfmergeSucceeds() |
|
27 | - { |
|
25 | + public function testLibpdfmergeSucceeds() { |
|
28 | 26 | |
29 | 27 | $composerJsonString = <<<'EOD' |
30 | 28 | { |
@@ -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 |
@@ -19,15 +19,15 @@ discard block |
||
19 | 19 | class MozartIssue86Test extends IntegrationTestCase |
20 | 20 | { |
21 | 21 | |
22 | - /** |
|
23 | - * Issue #86 – "class as" appeared in a comment and later the keyword as was prefixed! |
|
24 | - * |
|
25 | - * Solved by https://github.com/ziodave |
|
26 | - */ |
|
27 | - public function test_do_not_parse_comments_to_classnames() |
|
28 | - { |
|
29 | - |
|
30 | - $composerJsonString = <<<'EOD' |
|
22 | + /** |
|
23 | + * Issue #86 – "class as" appeared in a comment and later the keyword as was prefixed! |
|
24 | + * |
|
25 | + * Solved by https://github.com/ziodave |
|
26 | + */ |
|
27 | + public function test_do_not_parse_comments_to_classnames() |
|
28 | + { |
|
29 | + |
|
30 | + $composerJsonString = <<<'EOD' |
|
31 | 31 | { |
32 | 32 | "name": "brianhenryie/mozart-issue-86", |
33 | 33 | "require": { |
@@ -50,38 +50,38 @@ 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 | - $php_string = file_get_contents($this->testsWorkingDir .'/strauss/pear/pear_exception/PEAR/Exception.php'); |
|
66 | + $php_string = file_get_contents($this->testsWorkingDir .'/strauss/pear/pear_exception/PEAR/Exception.php'); |
|
67 | 67 | |
68 | - // Confirm problem is gone. |
|
69 | - $this->assertStringNotContainsString('foreach (self::$_observers Mozart_as $func) {', $php_string); |
|
68 | + // Confirm problem is gone. |
|
69 | + $this->assertStringNotContainsString('foreach (self::$_observers Mozart_as $func) {', $php_string); |
|
70 | 70 | |
71 | - // Confirm solution is correct. |
|
72 | - $this->assertStringContainsString('foreach (self::$_observers as $func) {', $php_string); |
|
73 | - } |
|
71 | + // Confirm solution is correct. |
|
72 | + $this->assertStringContainsString('foreach (self::$_observers as $func) {', $php_string); |
|
73 | + } |
|
74 | 74 | |
75 | 75 | |
76 | - /** |
|
77 | - * Like issue #86, when prefixing WP_Dependency_Installer, words in comments were |
|
78 | - * |
|
79 | - * @see https://github.com/afragen/wp-dependency-installer/ |
|
80 | - */ |
|
81 | - public function test_do_not_parse_comments_to_classnames_wp_dependency_installer() |
|
82 | - { |
|
76 | + /** |
|
77 | + * Like issue #86, when prefixing WP_Dependency_Installer, words in comments were |
|
78 | + * |
|
79 | + * @see https://github.com/afragen/wp-dependency-installer/ |
|
80 | + */ |
|
81 | + public function test_do_not_parse_comments_to_classnames_wp_dependency_installer() |
|
82 | + { |
|
83 | 83 | |
84 | - $composerJsonString = <<<'EOD' |
|
84 | + $composerJsonString = <<<'EOD' |
|
85 | 85 | { |
86 | 86 | "name": "brianhenryie/mozart-issue-86-2", |
87 | 87 | "require": { |
@@ -96,25 +96,25 @@ discard block |
||
96 | 96 | } |
97 | 97 | EOD; |
98 | 98 | |
99 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
99 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
100 | 100 | |
101 | - chdir($this->testsWorkingDir); |
|
101 | + chdir($this->testsWorkingDir); |
|
102 | 102 | |
103 | - exec('composer install'); |
|
103 | + exec('composer install'); |
|
104 | 104 | |
105 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
106 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
105 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
106 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
107 | 107 | |
108 | - $mozartCompose = new Compose(); |
|
108 | + $mozartCompose = new Compose(); |
|
109 | 109 | |
110 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
110 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
111 | 111 | |
112 | - $php_string = file_get_contents($this->testsWorkingDir .'/strauss/afragen/wp-dependency-installer/wp-dependency-installer.php'); |
|
112 | + $php_string = file_get_contents($this->testsWorkingDir .'/strauss/afragen/wp-dependency-installer/wp-dependency-installer.php'); |
|
113 | 113 | |
114 | - // Confirm problem is gone. |
|
115 | - $this->assertStringNotContainsString('Path BrianHenryIE_Strauss_to plugin or theme', $php_string, 'Text in comment still prefixed.'); |
|
114 | + // Confirm problem is gone. |
|
115 | + $this->assertStringNotContainsString('Path BrianHenryIE_Strauss_to plugin or theme', $php_string, 'Text in comment still prefixed.'); |
|
116 | 116 | |
117 | - // Confirm solution is correct. |
|
118 | - $this->assertStringContainsString('BrianHenryIE_Strauss_WP_Dependency_Installer', $php_string, 'Class name not properly prefixed.'); |
|
119 | - } |
|
117 | + // Confirm solution is correct. |
|
118 | + $this->assertStringContainsString('BrianHenryIE_Strauss_WP_Dependency_Installer', $php_string, 'Class name not properly prefixed.'); |
|
119 | + } |
|
120 | 120 | } |
@@ -50,26 +50,26 @@ 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 | 62 | $mozartCompose = new Compose(); |
63 | 63 | |
64 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
64 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
65 | 65 | |
66 | - $php_string = file_get_contents($this->testsWorkingDir .'/strauss/pear/pear_exception/PEAR/Exception.php'); |
|
66 | + $php_string = file_get_contents( $this->testsWorkingDir . '/strauss/pear/pear_exception/PEAR/Exception.php' ); |
|
67 | 67 | |
68 | 68 | // Confirm problem is gone. |
69 | - $this->assertStringNotContainsString('foreach (self::$_observers Mozart_as $func) {', $php_string); |
|
69 | + $this->assertStringNotContainsString( 'foreach (self::$_observers Mozart_as $func) {', $php_string ); |
|
70 | 70 | |
71 | 71 | // Confirm solution is correct. |
72 | - $this->assertStringContainsString('foreach (self::$_observers as $func) {', $php_string); |
|
72 | + $this->assertStringContainsString( 'foreach (self::$_observers as $func) {', $php_string ); |
|
73 | 73 | } |
74 | 74 | |
75 | 75 | |
@@ -96,25 +96,25 @@ discard block |
||
96 | 96 | } |
97 | 97 | EOD; |
98 | 98 | |
99 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
99 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
100 | 100 | |
101 | - chdir($this->testsWorkingDir); |
|
101 | + chdir( $this->testsWorkingDir ); |
|
102 | 102 | |
103 | - exec('composer install'); |
|
103 | + exec( 'composer install' ); |
|
104 | 104 | |
105 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
106 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
105 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
106 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
107 | 107 | |
108 | 108 | $mozartCompose = new Compose(); |
109 | 109 | |
110 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
110 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
111 | 111 | |
112 | - $php_string = file_get_contents($this->testsWorkingDir .'/strauss/afragen/wp-dependency-installer/wp-dependency-installer.php'); |
|
112 | + $php_string = file_get_contents( $this->testsWorkingDir . '/strauss/afragen/wp-dependency-installer/wp-dependency-installer.php' ); |
|
113 | 113 | |
114 | 114 | // Confirm problem is gone. |
115 | - $this->assertStringNotContainsString('Path BrianHenryIE_Strauss_to plugin or theme', $php_string, 'Text in comment still prefixed.'); |
|
115 | + $this->assertStringNotContainsString( 'Path BrianHenryIE_Strauss_to plugin or theme', $php_string, 'Text in comment still prefixed.' ); |
|
116 | 116 | |
117 | 117 | // Confirm solution is correct. |
118 | - $this->assertStringContainsString('BrianHenryIE_Strauss_WP_Dependency_Installer', $php_string, 'Class name not properly prefixed.'); |
|
118 | + $this->assertStringContainsString( 'BrianHenryIE_Strauss_WP_Dependency_Installer', $php_string, 'Class name not properly prefixed.' ); |
|
119 | 119 | } |
120 | 120 | } |
@@ -16,16 +16,14 @@ discard block |
||
16 | 16 | * Class NamespaceReplacerIntegrationTest |
17 | 17 | * @coversNothing |
18 | 18 | */ |
19 | -class MozartIssue86Test extends IntegrationTestCase |
|
20 | -{ |
|
19 | +class MozartIssue86Test extends IntegrationTestCase { |
|
21 | 20 | |
22 | 21 | /** |
23 | 22 | * Issue #86 – "class as" appeared in a comment and later the keyword as was prefixed! |
24 | 23 | * |
25 | 24 | * Solved by https://github.com/ziodave |
26 | 25 | */ |
27 | - public function test_do_not_parse_comments_to_classnames() |
|
28 | - { |
|
26 | + public function test_do_not_parse_comments_to_classnames() { |
|
29 | 27 | |
30 | 28 | $composerJsonString = <<<'EOD' |
31 | 29 | { |
@@ -78,8 +76,7 @@ discard block |
||
78 | 76 | * |
79 | 77 | * @see https://github.com/afragen/wp-dependency-installer/ |
80 | 78 | */ |
81 | - public function test_do_not_parse_comments_to_classnames_wp_dependency_installer() |
|
82 | - { |
|
79 | + public function test_do_not_parse_comments_to_classnames_wp_dependency_installer() { |
|
83 | 80 | |
84 | 81 | $composerJsonString = <<<'EOD' |
85 | 82 | { |
@@ -48,7 +48,7 @@ discard block |
||
48 | 48 | } |
49 | 49 | } |
50 | 50 | } |
51 | -EOD; |
|
51 | +eod; |
|
52 | 52 | |
53 | 53 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
54 | 54 | |
@@ -94,7 +94,7 @@ discard block |
||
94 | 94 | } |
95 | 95 | } |
96 | 96 | } |
97 | -EOD; |
|
97 | +eod; |
|
98 | 98 | |
99 | 99 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
100 | 100 |
@@ -17,15 +17,15 @@ discard block |
||
17 | 17 | class StraussIssue19Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
18 | 18 | { |
19 | 19 | |
20 | - /** |
|
21 | - * Need to make the class finder in change enumerator stricter. |
|
22 | - * |
|
23 | - * @author BrianHenryIE |
|
24 | - */ |
|
25 | - public function testObjectIsNotPrefixed() |
|
26 | - { |
|
27 | - |
|
28 | - $composerJsonString = <<<'EOD' |
|
20 | + /** |
|
21 | + * Need to make the class finder in change enumerator stricter. |
|
22 | + * |
|
23 | + * @author BrianHenryIE |
|
24 | + */ |
|
25 | + public function testObjectIsNotPrefixed() |
|
26 | + { |
|
27 | + |
|
28 | + $composerJsonString = <<<'EOD' |
|
29 | 29 | { |
30 | 30 | "name": "brianhenryie/strauss-issue-19", |
31 | 31 | "require": { |
@@ -40,23 +40,23 @@ discard block |
||
40 | 40 | } |
41 | 41 | EOD; |
42 | 42 | |
43 | - file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
43 | + file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
44 | 44 | |
45 | - chdir($this->testsWorkingDir); |
|
45 | + chdir($this->testsWorkingDir); |
|
46 | 46 | |
47 | - exec('composer install'); |
|
47 | + exec('composer install'); |
|
48 | 48 | |
49 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
50 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
49 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
50 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
51 | 51 | |
52 | - $mozartCompose = new Compose(); |
|
52 | + $mozartCompose = new Compose(); |
|
53 | 53 | |
54 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
54 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
55 | 55 | |
56 | - $php_string = file_get_contents($this->testsWorkingDir . 'strauss/tecnickcom/tcpdf/include/tcpdf_static.php'); |
|
56 | + $php_string = file_get_contents($this->testsWorkingDir . 'strauss/tecnickcom/tcpdf/include/tcpdf_static.php'); |
|
57 | 57 | |
58 | - $this->assertStringNotContainsString('* Creates a copy of a class Strauss_Issue19_object', $php_string); |
|
58 | + $this->assertStringNotContainsString('* Creates a copy of a class Strauss_Issue19_object', $php_string); |
|
59 | 59 | |
60 | - $this->assertStringContainsString('* Creates a copy of a class object', $php_string); |
|
61 | - } |
|
60 | + $this->assertStringContainsString('* Creates a copy of a class object', $php_string); |
|
61 | + } |
|
62 | 62 | } |
@@ -40,23 +40,23 @@ |
||
40 | 40 | } |
41 | 41 | EOD; |
42 | 42 | |
43 | - file_put_contents($this->testsWorkingDir . 'composer.json', $composerJsonString); |
|
43 | + file_put_contents( $this->testsWorkingDir . 'composer.json', $composerJsonString ); |
|
44 | 44 | |
45 | - chdir($this->testsWorkingDir); |
|
45 | + chdir( $this->testsWorkingDir ); |
|
46 | 46 | |
47 | - exec('composer install'); |
|
47 | + exec( 'composer install' ); |
|
48 | 48 | |
49 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
50 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
49 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
50 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
51 | 51 | |
52 | 52 | $mozartCompose = new Compose(); |
53 | 53 | |
54 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
54 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
55 | 55 | |
56 | - $php_string = file_get_contents($this->testsWorkingDir . 'strauss/tecnickcom/tcpdf/include/tcpdf_static.php'); |
|
56 | + $php_string = file_get_contents( $this->testsWorkingDir . 'strauss/tecnickcom/tcpdf/include/tcpdf_static.php' ); |
|
57 | 57 | |
58 | - $this->assertStringNotContainsString('* Creates a copy of a class Strauss_Issue19_object', $php_string); |
|
58 | + $this->assertStringNotContainsString( '* Creates a copy of a class Strauss_Issue19_object', $php_string ); |
|
59 | 59 | |
60 | - $this->assertStringContainsString('* Creates a copy of a class object', $php_string); |
|
60 | + $this->assertStringContainsString( '* Creates a copy of a class object', $php_string ); |
|
61 | 61 | } |
62 | 62 | } |
@@ -14,16 +14,14 @@ |
||
14 | 14 | * @package BrianHenryIE\Strauss\Tests\Issues |
15 | 15 | * @coversNothing |
16 | 16 | */ |
17 | -class StraussIssue19Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
|
18 | -{ |
|
17 | +class StraussIssue19Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase { |
|
19 | 18 | |
20 | 19 | /** |
21 | 20 | * Need to make the class finder in change enumerator stricter. |
22 | 21 | * |
23 | 22 | * @author BrianHenryIE |
24 | 23 | */ |
25 | - public function testObjectIsNotPrefixed() |
|
26 | - { |
|
24 | + public function testObjectIsNotPrefixed() { |
|
27 | 25 | |
28 | 26 | $composerJsonString = <<<'EOD' |
29 | 27 | { |
@@ -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 |
@@ -15,16 +15,16 @@ discard block |
||
15 | 15 | class MozartIssue97Test extends IntegrationTestCase |
16 | 16 | { |
17 | 17 | |
18 | - /** |
|
19 | - * Issue 97. Package named "crewlabs/unsplash" is downloaded to `vendor/crewlabs/unsplash` but their composer.json |
|
20 | - * has the package name as "unsplash/unsplash". |
|
21 | - * |
|
22 | - * "The "/Users/BrianHenryIE/Sites/mozart-97/vendor/unsplash/unsplash/src" directory does not exist." |
|
23 | - */ |
|
24 | - public function testCrewlabsUnsplashSucceeds() |
|
25 | - { |
|
26 | - |
|
27 | - $composerJsonString = <<<'EOD' |
|
18 | + /** |
|
19 | + * Issue 97. Package named "crewlabs/unsplash" is downloaded to `vendor/crewlabs/unsplash` but their composer.json |
|
20 | + * has the package name as "unsplash/unsplash". |
|
21 | + * |
|
22 | + * "The "/Users/BrianHenryIE/Sites/mozart-97/vendor/unsplash/unsplash/src" directory does not exist." |
|
23 | + */ |
|
24 | + public function testCrewlabsUnsplashSucceeds() |
|
25 | + { |
|
26 | + |
|
27 | + $composerJsonString = <<<'EOD' |
|
28 | 28 | { |
29 | 29 | "name": "brianhenryie/mozart-issue-97", |
30 | 30 | "require": { |
@@ -39,19 +39,19 @@ discard block |
||
39 | 39 | } |
40 | 40 | EOD; |
41 | 41 | |
42 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
42 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
43 | 43 | |
44 | - chdir($this->testsWorkingDir); |
|
44 | + chdir($this->testsWorkingDir); |
|
45 | 45 | |
46 | - exec('composer install'); |
|
46 | + exec('composer install'); |
|
47 | 47 | |
48 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
49 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
48 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
49 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
50 | 50 | |
51 | - $mozartCompose = new Compose(); |
|
51 | + $mozartCompose = new Compose(); |
|
52 | 52 | |
53 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
53 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
54 | 54 | |
55 | - $this->assertEquals(0, $result); |
|
56 | - } |
|
55 | + $this->assertEquals(0, $result); |
|
56 | + } |
|
57 | 57 | } |
@@ -39,19 +39,19 @@ |
||
39 | 39 | } |
40 | 40 | EOD; |
41 | 41 | |
42 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
42 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
43 | 43 | |
44 | - chdir($this->testsWorkingDir); |
|
44 | + chdir( $this->testsWorkingDir ); |
|
45 | 45 | |
46 | - exec('composer install'); |
|
46 | + exec( 'composer install' ); |
|
47 | 47 | |
48 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
49 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
48 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
49 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
50 | 50 | |
51 | 51 | $mozartCompose = new Compose(); |
52 | 52 | |
53 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
53 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
54 | 54 | |
55 | - $this->assertEquals(0, $result); |
|
55 | + $this->assertEquals( 0, $result ); |
|
56 | 56 | } |
57 | 57 | } |
@@ -12,8 +12,7 @@ discard block |
||
12 | 12 | * @package BrianHenryIE\Strauss\Tests\Issues |
13 | 13 | * @coversNothing |
14 | 14 | */ |
15 | -class MozartIssue97Test extends IntegrationTestCase |
|
16 | -{ |
|
15 | +class MozartIssue97Test extends IntegrationTestCase { |
|
17 | 16 | |
18 | 17 | /** |
19 | 18 | * Issue 97. Package named "crewlabs/unsplash" is downloaded to `vendor/crewlabs/unsplash` but their composer.json |
@@ -21,8 +20,7 @@ discard block |
||
21 | 20 | * |
22 | 21 | * "The "/Users/BrianHenryIE/Sites/mozart-97/vendor/unsplash/unsplash/src" directory does not exist." |
23 | 22 | */ |
24 | - public function testCrewlabsUnsplashSucceeds() |
|
25 | - { |
|
23 | + public function testCrewlabsUnsplashSucceeds() { |
|
26 | 24 | |
27 | 25 | $composerJsonString = <<<'EOD' |
28 | 26 | { |
@@ -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 |
@@ -20,52 +20,52 @@ |
||
20 | 20 | class MozartIssue48Test extends IntegrationTestCase |
21 | 21 | { |
22 | 22 | |
23 | - /** |
|
24 | - * rubix/tensor |
|
25 | - * |
|
26 | - * Mozart was only processing one of the PSR-4 autoload paths, in which case it was not copying (amongst others) |
|
27 | - * `EigenvalueDecomposition.php` at all. Test for its presence. |
|
28 | - */ |
|
29 | - public function testRubixTensorBothPathsPersist() |
|
30 | - { |
|
23 | + /** |
|
24 | + * rubix/tensor |
|
25 | + * |
|
26 | + * Mozart was only processing one of the PSR-4 autoload paths, in which case it was not copying (amongst others) |
|
27 | + * `EigenvalueDecomposition.php` at all. Test for its presence. |
|
28 | + */ |
|
29 | + public function testRubixTensorBothPathsPersist() |
|
30 | + { |
|
31 | 31 | |
32 | - $composerJsonString = <<<'EOD' |
|
32 | + $composerJsonString = <<<'EOD' |
|
33 | 33 | { |
34 | 34 | "name": "brianhenryie/mozart-issue-48", |
35 | 35 | "require": { "rubix/tensor": "*" } |
36 | 36 | } |
37 | 37 | EOD; |
38 | 38 | |
39 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
39 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
40 | 40 | |
41 | - chdir($this->testsWorkingDir); |
|
41 | + chdir($this->testsWorkingDir); |
|
42 | 42 | |
43 | - exec('composer install'); |
|
43 | + exec('composer install'); |
|
44 | 44 | |
45 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
46 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
45 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
46 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
47 | 47 | |
48 | - $mozartCompose = new Compose(); |
|
48 | + $mozartCompose = new Compose(); |
|
49 | 49 | |
50 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
50 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
51 | 51 | |
52 | - // EigenvalueDecomposition.php |
|
53 | - // assert file exists somewhere in the tree |
|
52 | + // EigenvalueDecomposition.php |
|
53 | + // assert file exists somewhere in the tree |
|
54 | 54 | |
55 | - // https://stackoverflow.com/questions/17160696/php-glob-scan-in-subfolders-for-a-file |
|
56 | - $rsearch = function ($folder, $pattern) { |
|
57 | - $dir = new \RecursiveDirectoryIterator($folder); |
|
58 | - $ite = new \RecursiveIteratorIterator($dir); |
|
59 | - $files = new \RegexIterator($ite, $pattern, \RegexIterator::GET_MATCH); |
|
60 | - $fileList = array(); |
|
61 | - foreach ($files as $file) { |
|
62 | - $fileList = array_merge($fileList, $file); |
|
63 | - } |
|
64 | - return $fileList; |
|
65 | - }; |
|
55 | + // https://stackoverflow.com/questions/17160696/php-glob-scan-in-subfolders-for-a-file |
|
56 | + $rsearch = function ($folder, $pattern) { |
|
57 | + $dir = new \RecursiveDirectoryIterator($folder); |
|
58 | + $ite = new \RecursiveIteratorIterator($dir); |
|
59 | + $files = new \RegexIterator($ite, $pattern, \RegexIterator::GET_MATCH); |
|
60 | + $fileList = array(); |
|
61 | + foreach ($files as $file) { |
|
62 | + $fileList = array_merge($fileList, $file); |
|
63 | + } |
|
64 | + return $fileList; |
|
65 | + }; |
|
66 | 66 | |
67 | - $found = $rsearch($this->testsWorkingDir . 'strauss', '~EigenvalueDecomposition\.php~'); |
|
67 | + $found = $rsearch($this->testsWorkingDir . 'strauss', '~EigenvalueDecomposition\.php~'); |
|
68 | 68 | |
69 | - $this->assertNotEmpty($found); |
|
70 | - } |
|
69 | + $this->assertNotEmpty($found); |
|
70 | + } |
|
71 | 71 | } |
@@ -36,36 +36,36 @@ |
||
36 | 36 | } |
37 | 37 | EOD; |
38 | 38 | |
39 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
39 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
40 | 40 | |
41 | - chdir($this->testsWorkingDir); |
|
41 | + chdir( $this->testsWorkingDir ); |
|
42 | 42 | |
43 | - exec('composer install'); |
|
43 | + exec( 'composer install' ); |
|
44 | 44 | |
45 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
46 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
45 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
46 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
47 | 47 | |
48 | 48 | $mozartCompose = new Compose(); |
49 | 49 | |
50 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
50 | + $result = $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
51 | 51 | |
52 | 52 | // EigenvalueDecomposition.php |
53 | 53 | // assert file exists somewhere in the tree |
54 | 54 | |
55 | 55 | // https://stackoverflow.com/questions/17160696/php-glob-scan-in-subfolders-for-a-file |
56 | - $rsearch = function ($folder, $pattern) { |
|
57 | - $dir = new \RecursiveDirectoryIterator($folder); |
|
58 | - $ite = new \RecursiveIteratorIterator($dir); |
|
59 | - $files = new \RegexIterator($ite, $pattern, \RegexIterator::GET_MATCH); |
|
56 | + $rsearch = function( $folder, $pattern ) { |
|
57 | + $dir = new \RecursiveDirectoryIterator( $folder ); |
|
58 | + $ite = new \RecursiveIteratorIterator( $dir ); |
|
59 | + $files = new \RegexIterator( $ite, $pattern, \RegexIterator::GET_MATCH ); |
|
60 | 60 | $fileList = array(); |
61 | - foreach ($files as $file) { |
|
62 | - $fileList = array_merge($fileList, $file); |
|
61 | + foreach ( $files as $file ) { |
|
62 | + $fileList = array_merge( $fileList, $file ); |
|
63 | 63 | } |
64 | 64 | return $fileList; |
65 | 65 | }; |
66 | 66 | |
67 | - $found = $rsearch($this->testsWorkingDir . 'strauss', '~EigenvalueDecomposition\.php~'); |
|
67 | + $found = $rsearch( $this->testsWorkingDir . 'strauss', '~EigenvalueDecomposition\.php~' ); |
|
68 | 68 | |
69 | - $this->assertNotEmpty($found); |
|
69 | + $this->assertNotEmpty( $found ); |
|
70 | 70 | } |
71 | 71 | } |
@@ -17,8 +17,7 @@ discard block |
||
17 | 17 | * @package BrianHenryIE\Strauss\Tests\Issues |
18 | 18 | * @coversNothing |
19 | 19 | */ |
20 | -class MozartIssue48Test extends IntegrationTestCase |
|
21 | -{ |
|
20 | +class MozartIssue48Test extends IntegrationTestCase { |
|
22 | 21 | |
23 | 22 | /** |
24 | 23 | * rubix/tensor |
@@ -26,8 +25,7 @@ discard block |
||
26 | 25 | * Mozart was only processing one of the PSR-4 autoload paths, in which case it was not copying (amongst others) |
27 | 26 | * `EigenvalueDecomposition.php` at all. Test for its presence. |
28 | 27 | */ |
29 | - public function testRubixTensorBothPathsPersist() |
|
30 | - { |
|
28 | + public function testRubixTensorBothPathsPersist() { |
|
31 | 29 | |
32 | 30 | $composerJsonString = <<<'EOD' |
33 | 31 | { |
@@ -34,7 +34,7 @@ |
||
34 | 34 | "name": "brianhenryie/mozart-issue-48", |
35 | 35 | "require": { "rubix/tensor": "*" } |
36 | 36 | } |
37 | -EOD; |
|
37 | +eod; |
|
38 | 38 | |
39 | 39 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
40 | 40 |
@@ -32,14 +32,14 @@ discard block |
||
32 | 32 | class MozartIssue62Test extends IntegrationTestCase |
33 | 33 | { |
34 | 34 | |
35 | - /** |
|
36 | - * Just confirms `use Guzzle\Common\Collection;` is prefixed. |
|
37 | - */ |
|
38 | - public function testGuzzleNamespaceIsPrefixedInS3Client() |
|
39 | - { |
|
40 | - $this->markTestSkipped('Very slow to run'); |
|
35 | + /** |
|
36 | + * Just confirms `use Guzzle\Common\Collection;` is prefixed. |
|
37 | + */ |
|
38 | + public function testGuzzleNamespaceIsPrefixedInS3Client() |
|
39 | + { |
|
40 | + $this->markTestSkipped('Very slow to run'); |
|
41 | 41 | |
42 | - $composerJsonString = <<<'EOD' |
|
42 | + $composerJsonString = <<<'EOD' |
|
43 | 43 | { |
44 | 44 | "name": "brianhenryie/mozart-issue-62", |
45 | 45 | "require": { |
@@ -54,21 +54,21 @@ discard block |
||
54 | 54 | } |
55 | 55 | EOD; |
56 | 56 | |
57 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
57 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
58 | 58 | |
59 | - chdir($this->testsWorkingDir); |
|
59 | + chdir($this->testsWorkingDir); |
|
60 | 60 | |
61 | - exec('composer install'); |
|
61 | + exec('composer install'); |
|
62 | 62 | |
63 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
64 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
63 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
64 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
65 | 65 | |
66 | - $mozartCompose = new Compose(); |
|
66 | + $mozartCompose = new Compose(); |
|
67 | 67 | |
68 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
68 | + $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
69 | 69 | |
70 | - $phpString = file_get_contents($this->testsWorkingDir .'strauss/aws/aws-sdk-php/src/Aws/S3/S3Client.php'); |
|
70 | + $phpString = file_get_contents($this->testsWorkingDir .'strauss/aws/aws-sdk-php/src/Aws/S3/S3Client.php'); |
|
71 | 71 | |
72 | - $this->assertStringContainsString('use Strauss\\Guzzle\\Common\\Collection;', $phpString); |
|
73 | - } |
|
72 | + $this->assertStringContainsString('use Strauss\\Guzzle\\Common\\Collection;', $phpString); |
|
73 | + } |
|
74 | 74 | } |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | */ |
38 | 38 | public function testGuzzleNamespaceIsPrefixedInS3Client() |
39 | 39 | { |
40 | - $this->markTestSkipped('Very slow to run'); |
|
40 | + $this->markTestSkipped( 'Very slow to run' ); |
|
41 | 41 | |
42 | 42 | $composerJsonString = <<<'EOD' |
43 | 43 | { |
@@ -54,21 +54,21 @@ discard block |
||
54 | 54 | } |
55 | 55 | EOD; |
56 | 56 | |
57 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
57 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
58 | 58 | |
59 | - chdir($this->testsWorkingDir); |
|
59 | + chdir( $this->testsWorkingDir ); |
|
60 | 60 | |
61 | - exec('composer install'); |
|
61 | + exec( 'composer install' ); |
|
62 | 62 | |
63 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
64 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
63 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
64 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
65 | 65 | |
66 | 66 | $mozartCompose = new Compose(); |
67 | 67 | |
68 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
68 | + $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
69 | 69 | |
70 | - $phpString = file_get_contents($this->testsWorkingDir .'strauss/aws/aws-sdk-php/src/Aws/S3/S3Client.php'); |
|
70 | + $phpString = file_get_contents( $this->testsWorkingDir . 'strauss/aws/aws-sdk-php/src/Aws/S3/S3Client.php' ); |
|
71 | 71 | |
72 | - $this->assertStringContainsString('use Strauss\\Guzzle\\Common\\Collection;', $phpString); |
|
72 | + $this->assertStringContainsString( 'use Strauss\\Guzzle\\Common\\Collection;', $phpString ); |
|
73 | 73 | } |
74 | 74 | } |
@@ -29,14 +29,12 @@ |
||
29 | 29 | * @package BrianHenryIE\Strauss\Tests\Issues |
30 | 30 | * @coversNothing |
31 | 31 | */ |
32 | -class MozartIssue62Test extends IntegrationTestCase |
|
33 | -{ |
|
32 | +class MozartIssue62Test extends IntegrationTestCase { |
|
34 | 33 | |
35 | 34 | /** |
36 | 35 | * Just confirms `use Guzzle\Common\Collection;` is prefixed. |
37 | 36 | */ |
38 | - public function testGuzzleNamespaceIsPrefixedInS3Client() |
|
39 | - { |
|
37 | + public function testGuzzleNamespaceIsPrefixedInS3Client() { |
|
40 | 38 | $this->markTestSkipped('Very slow to run'); |
41 | 39 | |
42 | 40 | $composerJsonString = <<<'EOD' |
@@ -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 |
@@ -17,21 +17,21 @@ discard block |
||
17 | 17 | */ |
18 | 18 | class MozartIssue93Test extends IntegrationTestCase |
19 | 19 | { |
20 | - /** |
|
21 | - * Issue #93 shows a classname being updated inside a class whose namespace has also been updated |
|
22 | - * by Mozart. |
|
23 | - * |
|
24 | - * This is caused by the same files being loaded by both a PSR-4 autolaoder and classmap autoloader. |
|
25 | - * @see https://github.com/katzgrau/KLogger/blob/de2d3ab6777a393a9879e0496ebb8e0644066e3f/composer.json#L24-L29 |
|
26 | - * |
|
27 | - * @author BrianHenryIE |
|
28 | - */ |
|
29 | - public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
30 | - { |
|
20 | + /** |
|
21 | + * Issue #93 shows a classname being updated inside a class whose namespace has also been updated |
|
22 | + * by Mozart. |
|
23 | + * |
|
24 | + * This is caused by the same files being loaded by both a PSR-4 autolaoder and classmap autoloader. |
|
25 | + * @see https://github.com/katzgrau/KLogger/blob/de2d3ab6777a393a9879e0496ebb8e0644066e3f/composer.json#L24-L29 |
|
26 | + * |
|
27 | + * @author BrianHenryIE |
|
28 | + */ |
|
29 | + public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
30 | + { |
|
31 | 31 | |
32 | - $this->markTestSkipped('Not respecting the pinned commit.'); |
|
32 | + $this->markTestSkipped('Not respecting the pinned commit.'); |
|
33 | 33 | |
34 | - $composerJsonString = <<<'EOD' |
|
34 | + $composerJsonString = <<<'EOD' |
|
35 | 35 | { |
36 | 36 | "name": "brianhenryie/mozart-issue-93", |
37 | 37 | "repositories": [{ |
@@ -51,25 +51,25 @@ discard block |
||
51 | 51 | } |
52 | 52 | EOD; |
53 | 53 | |
54 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
54 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
55 | 55 | |
56 | - chdir($this->testsWorkingDir); |
|
56 | + chdir($this->testsWorkingDir); |
|
57 | 57 | |
58 | - exec('composer install'); |
|
58 | + exec('composer install'); |
|
59 | 59 | |
60 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
61 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
60 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
61 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
62 | 62 | |
63 | - $mozartCompose = new Compose(); |
|
63 | + $mozartCompose = new Compose(); |
|
64 | 64 | |
65 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
65 | + $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
66 | 66 | |
67 | - $php_string = file_get_contents($this->testsWorkingDir .'strauss/brianhenryie/wp-logger/src/class-logger.php'); |
|
67 | + $php_string = file_get_contents($this->testsWorkingDir .'strauss/brianhenryie/wp-logger/src/class-logger.php'); |
|
68 | 68 | |
69 | - // Confirm problem is gone. |
|
70 | - $this->assertStringNotContainsString('class BrianHenryIE_Strauss_Logger extends', $php_string); |
|
69 | + // Confirm problem is gone. |
|
70 | + $this->assertStringNotContainsString('class BrianHenryIE_Strauss_Logger extends', $php_string); |
|
71 | 71 | |
72 | - // Confirm solution is correct. |
|
73 | - $this->assertStringContainsString('class Logger extends', $php_string); |
|
74 | - } |
|
72 | + // Confirm solution is correct. |
|
73 | + $this->assertStringContainsString('class Logger extends', $php_string); |
|
74 | + } |
|
75 | 75 | } |
@@ -29,7 +29,7 @@ discard block |
||
29 | 29 | public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
30 | 30 | { |
31 | 31 | |
32 | - $this->markTestSkipped('Not respecting the pinned commit.'); |
|
32 | + $this->markTestSkipped( 'Not respecting the pinned commit.' ); |
|
33 | 33 | |
34 | 34 | $composerJsonString = <<<'EOD' |
35 | 35 | { |
@@ -51,25 +51,25 @@ discard block |
||
51 | 51 | } |
52 | 52 | EOD; |
53 | 53 | |
54 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
54 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
55 | 55 | |
56 | - chdir($this->testsWorkingDir); |
|
56 | + chdir( $this->testsWorkingDir ); |
|
57 | 57 | |
58 | - exec('composer install'); |
|
58 | + exec( 'composer install' ); |
|
59 | 59 | |
60 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
61 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
60 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
61 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
62 | 62 | |
63 | 63 | $mozartCompose = new Compose(); |
64 | 64 | |
65 | - $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
65 | + $mozartCompose->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
66 | 66 | |
67 | - $php_string = file_get_contents($this->testsWorkingDir .'strauss/brianhenryie/wp-logger/src/class-logger.php'); |
|
67 | + $php_string = file_get_contents( $this->testsWorkingDir . 'strauss/brianhenryie/wp-logger/src/class-logger.php' ); |
|
68 | 68 | |
69 | 69 | // Confirm problem is gone. |
70 | - $this->assertStringNotContainsString('class BrianHenryIE_Strauss_Logger extends', $php_string); |
|
70 | + $this->assertStringNotContainsString( 'class BrianHenryIE_Strauss_Logger extends', $php_string ); |
|
71 | 71 | |
72 | 72 | // Confirm solution is correct. |
73 | - $this->assertStringContainsString('class Logger extends', $php_string); |
|
73 | + $this->assertStringContainsString( 'class Logger extends', $php_string ); |
|
74 | 74 | } |
75 | 75 | } |
@@ -15,8 +15,7 @@ discard block |
||
15 | 15 | * Class MozartIssue93Test |
16 | 16 | * @coversNothing |
17 | 17 | */ |
18 | -class MozartIssue93Test extends IntegrationTestCase |
|
19 | -{ |
|
18 | +class MozartIssue93Test extends IntegrationTestCase { |
|
20 | 19 | /** |
21 | 20 | * Issue #93 shows a classname being updated inside a class whose namespace has also been updated |
22 | 21 | * by Mozart. |
@@ -26,8 +25,7 @@ discard block |
||
26 | 25 | * |
27 | 26 | * @author BrianHenryIE |
28 | 27 | */ |
29 | - public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
30 | - { |
|
28 | + public function test_it_does_not_make_classname_replacement_inside_namespaced_file() { |
|
31 | 29 | |
32 | 30 | $this->markTestSkipped('Not respecting the pinned commit.'); |
33 | 31 |
@@ -49,7 +49,7 @@ |
||
49 | 49 | }, |
50 | 50 | "minimum-stability": "dev" |
51 | 51 | } |
52 | -EOD; |
|
52 | +eod; |
|
53 | 53 | |
54 | 54 | file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
55 | 55 |
@@ -21,17 +21,17 @@ discard block |
||
21 | 21 | class MozartIssue43Test extends IntegrationTestCase |
22 | 22 | { |
23 | 23 | |
24 | - /** |
|
25 | - * Issue 43. Needs "aws/aws-sdk-php". |
|
26 | - * |
|
27 | - * League\Flysystem\FileExistsException : File already exists at path: |
|
28 | - * dep_directory/vendor/guzzle/guzzle/src/Guzzle/Cache/Zf1CacheAdapter.php |
|
29 | - */ |
|
30 | - public function testAwsSdkSucceeds() |
|
31 | - { |
|
32 | - $this->markTestSkipped('Very slow to run'); |
|
24 | + /** |
|
25 | + * Issue 43. Needs "aws/aws-sdk-php". |
|
26 | + * |
|
27 | + * League\Flysystem\FileExistsException : File already exists at path: |
|
28 | + * dep_directory/vendor/guzzle/guzzle/src/Guzzle/Cache/Zf1CacheAdapter.php |
|
29 | + */ |
|
30 | + public function testAwsSdkSucceeds() |
|
31 | + { |
|
32 | + $this->markTestSkipped('Very slow to run'); |
|
33 | 33 | |
34 | - $composerJsonString = <<<'EOD' |
|
34 | + $composerJsonString = <<<'EOD' |
|
35 | 35 | { |
36 | 36 | "name": "brianhenryie/mozart-issue-43", |
37 | 37 | "require": { |
@@ -53,21 +53,21 @@ discard block |
||
53 | 53 | } |
54 | 54 | EOD; |
55 | 55 | |
56 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
56 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
57 | 57 | |
58 | - chdir($this->testsWorkingDir); |
|
58 | + chdir($this->testsWorkingDir); |
|
59 | 59 | |
60 | - exec('composer install'); |
|
60 | + exec('composer install'); |
|
61 | 61 | |
62 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
63 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
62 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
63 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
64 | 64 | |
65 | - $strauss = new Compose(); |
|
65 | + $strauss = new Compose(); |
|
66 | 66 | |
67 | - $result = $strauss->run($inputInterfaceMock, $outputInterfaceMock); |
|
67 | + $result = $strauss->run($inputInterfaceMock, $outputInterfaceMock); |
|
68 | 68 | // |
69 | 69 | // $this->assertEquals(0, $result); |
70 | 70 | |
71 | - $this->assertFileExists($this->testsWorkingDir . '/strauss/aws/aws-sdk-php/src/AWS/Common/Aws.php'); |
|
72 | - } |
|
71 | + $this->assertFileExists($this->testsWorkingDir . '/strauss/aws/aws-sdk-php/src/AWS/Common/Aws.php'); |
|
72 | + } |
|
73 | 73 | } |
@@ -29,7 +29,7 @@ discard block |
||
29 | 29 | */ |
30 | 30 | public function testAwsSdkSucceeds() |
31 | 31 | { |
32 | - $this->markTestSkipped('Very slow to run'); |
|
32 | + $this->markTestSkipped( 'Very slow to run' ); |
|
33 | 33 | |
34 | 34 | $composerJsonString = <<<'EOD' |
35 | 35 | { |
@@ -53,21 +53,21 @@ discard block |
||
53 | 53 | } |
54 | 54 | EOD; |
55 | 55 | |
56 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
56 | + file_put_contents( $this->testsWorkingDir . '/composer.json', $composerJsonString ); |
|
57 | 57 | |
58 | - chdir($this->testsWorkingDir); |
|
58 | + chdir( $this->testsWorkingDir ); |
|
59 | 59 | |
60 | - exec('composer install'); |
|
60 | + exec( 'composer install' ); |
|
61 | 61 | |
62 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
63 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
62 | + $inputInterfaceMock = $this->createMock( InputInterface::class ); |
|
63 | + $outputInterfaceMock = $this->createMock( OutputInterface::class ); |
|
64 | 64 | |
65 | 65 | $strauss = new Compose(); |
66 | 66 | |
67 | - $result = $strauss->run($inputInterfaceMock, $outputInterfaceMock); |
|
67 | + $result = $strauss->run( $inputInterfaceMock, $outputInterfaceMock ); |
|
68 | 68 | // |
69 | 69 | // $this->assertEquals(0, $result); |
70 | 70 | |
71 | - $this->assertFileExists($this->testsWorkingDir . '/strauss/aws/aws-sdk-php/src/AWS/Common/Aws.php'); |
|
71 | + $this->assertFileExists( $this->testsWorkingDir . '/strauss/aws/aws-sdk-php/src/AWS/Common/Aws.php' ); |
|
72 | 72 | } |
73 | 73 | } |
@@ -18,8 +18,7 @@ discard block |
||
18 | 18 | * @package BrianHenryIE\Strauss\Tests\Issues |
19 | 19 | * @coversNothing |
20 | 20 | */ |
21 | -class MozartIssue43Test extends IntegrationTestCase |
|
22 | -{ |
|
21 | +class MozartIssue43Test extends IntegrationTestCase { |
|
23 | 22 | |
24 | 23 | /** |
25 | 24 | * Issue 43. Needs "aws/aws-sdk-php". |
@@ -27,8 +26,7 @@ discard block |
||
27 | 26 | * League\Flysystem\FileExistsException : File already exists at path: |
28 | 27 | * dep_directory/vendor/guzzle/guzzle/src/Guzzle/Cache/Zf1CacheAdapter.php |
29 | 28 | */ |
30 | - public function testAwsSdkSucceeds() |
|
31 | - { |
|
29 | + public function testAwsSdkSucceeds() { |
|
32 | 30 | $this->markTestSkipped('Very slow to run'); |
33 | 31 | |
34 | 32 | $composerJsonString = <<<'EOD' |
@@ -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 |
@@ -19,72 +19,72 @@ |
||
19 | 19 | |
20 | 20 | class Copier |
21 | 21 | { |
22 | - /** |
|
23 | - * The only path variable with a leading slash. |
|
24 | - * All directories in project end with a slash. |
|
25 | - * |
|
26 | - * @var string |
|
27 | - */ |
|
28 | - protected string $workingDir; |
|
29 | - |
|
30 | - protected string $targetDir; |
|
31 | - |
|
32 | - protected array $filepaths; |
|
33 | - |
|
34 | - /** @var Filesystem */ |
|
35 | - protected Filesystem $filesystem; |
|
36 | - |
|
37 | - /** |
|
38 | - * Copier constructor. |
|
39 | - * @param array<string, ComposerPackage> $filepaths |
|
40 | - * @param string $workingDir |
|
41 | - * @param string $relativeTargetDir |
|
42 | - */ |
|
43 | - public function __construct(array $filepaths, string $workingDir, string $relativeTargetDir) |
|
44 | - { |
|
45 | - $this->filepaths = array_keys($filepaths); |
|
46 | - |
|
47 | - $this->workingDir = $workingDir; |
|
48 | - |
|
49 | - $this->targetDir = $relativeTargetDir; |
|
50 | - |
|
51 | - $this->filesystem = new Filesystem(new Local($this->workingDir)); |
|
52 | - } |
|
53 | - |
|
54 | - /** |
|
55 | - * If the target dir does not exist, create it. |
|
56 | - * If it already exists, delete any files we're about to copy. |
|
57 | - * |
|
58 | - * @return void |
|
59 | - */ |
|
60 | - public function prepareTarget(): void |
|
61 | - { |
|
62 | - if (! $this->filesystem->has($this->targetDir)) { |
|
63 | - $this->filesystem->createDir($this->targetDir); |
|
64 | - } else { |
|
65 | - foreach ($this->filepaths as $vendorRelativeFilepath) { |
|
66 | - $projectRelativeFilepath = $this->targetDir . $vendorRelativeFilepath; |
|
67 | - |
|
68 | - if ($this->filesystem->has($projectRelativeFilepath)) { |
|
69 | - $this->filesystem->delete($projectRelativeFilepath); |
|
70 | - } |
|
71 | - } |
|
72 | - } |
|
73 | - } |
|
74 | - |
|
75 | - |
|
76 | - /** |
|
77 | - * |
|
78 | - */ |
|
79 | - public function copy(): void |
|
80 | - { |
|
81 | - |
|
82 | - foreach ($this->filepaths as $relativeFilepath) { |
|
83 | - $sourceFileRelativePath = 'vendor' . DIRECTORY_SEPARATOR . $relativeFilepath; |
|
84 | - |
|
85 | - $targetFileRelativePath = $this->targetDir . $relativeFilepath; |
|
86 | - |
|
87 | - $this->filesystem->copy($sourceFileRelativePath, $targetFileRelativePath); |
|
88 | - } |
|
89 | - } |
|
22 | + /** |
|
23 | + * The only path variable with a leading slash. |
|
24 | + * All directories in project end with a slash. |
|
25 | + * |
|
26 | + * @var string |
|
27 | + */ |
|
28 | + protected string $workingDir; |
|
29 | + |
|
30 | + protected string $targetDir; |
|
31 | + |
|
32 | + protected array $filepaths; |
|
33 | + |
|
34 | + /** @var Filesystem */ |
|
35 | + protected Filesystem $filesystem; |
|
36 | + |
|
37 | + /** |
|
38 | + * Copier constructor. |
|
39 | + * @param array<string, ComposerPackage> $filepaths |
|
40 | + * @param string $workingDir |
|
41 | + * @param string $relativeTargetDir |
|
42 | + */ |
|
43 | + public function __construct(array $filepaths, string $workingDir, string $relativeTargetDir) |
|
44 | + { |
|
45 | + $this->filepaths = array_keys($filepaths); |
|
46 | + |
|
47 | + $this->workingDir = $workingDir; |
|
48 | + |
|
49 | + $this->targetDir = $relativeTargetDir; |
|
50 | + |
|
51 | + $this->filesystem = new Filesystem(new Local($this->workingDir)); |
|
52 | + } |
|
53 | + |
|
54 | + /** |
|
55 | + * If the target dir does not exist, create it. |
|
56 | + * If it already exists, delete any files we're about to copy. |
|
57 | + * |
|
58 | + * @return void |
|
59 | + */ |
|
60 | + public function prepareTarget(): void |
|
61 | + { |
|
62 | + if (! $this->filesystem->has($this->targetDir)) { |
|
63 | + $this->filesystem->createDir($this->targetDir); |
|
64 | + } else { |
|
65 | + foreach ($this->filepaths as $vendorRelativeFilepath) { |
|
66 | + $projectRelativeFilepath = $this->targetDir . $vendorRelativeFilepath; |
|
67 | + |
|
68 | + if ($this->filesystem->has($projectRelativeFilepath)) { |
|
69 | + $this->filesystem->delete($projectRelativeFilepath); |
|
70 | + } |
|
71 | + } |
|
72 | + } |
|
73 | + } |
|
74 | + |
|
75 | + |
|
76 | + /** |
|
77 | + * |
|
78 | + */ |
|
79 | + public function copy(): void |
|
80 | + { |
|
81 | + |
|
82 | + foreach ($this->filepaths as $relativeFilepath) { |
|
83 | + $sourceFileRelativePath = 'vendor' . DIRECTORY_SEPARATOR . $relativeFilepath; |
|
84 | + |
|
85 | + $targetFileRelativePath = $this->targetDir . $relativeFilepath; |
|
86 | + |
|
87 | + $this->filesystem->copy($sourceFileRelativePath, $targetFileRelativePath); |
|
88 | + } |
|
89 | + } |
|
90 | 90 | } |
@@ -40,15 +40,15 @@ discard block |
||
40 | 40 | * @param string $workingDir |
41 | 41 | * @param string $relativeTargetDir |
42 | 42 | */ |
43 | - public function __construct(array $filepaths, string $workingDir, string $relativeTargetDir) |
|
43 | + public function __construct( array $filepaths, string $workingDir, string $relativeTargetDir ) |
|
44 | 44 | { |
45 | - $this->filepaths = array_keys($filepaths); |
|
45 | + $this->filepaths = array_keys( $filepaths ); |
|
46 | 46 | |
47 | 47 | $this->workingDir = $workingDir; |
48 | 48 | |
49 | 49 | $this->targetDir = $relativeTargetDir; |
50 | 50 | |
51 | - $this->filesystem = new Filesystem(new Local($this->workingDir)); |
|
51 | + $this->filesystem = new Filesystem( new Local( $this->workingDir ) ); |
|
52 | 52 | } |
53 | 53 | |
54 | 54 | /** |
@@ -59,14 +59,14 @@ discard block |
||
59 | 59 | */ |
60 | 60 | public function prepareTarget(): void |
61 | 61 | { |
62 | - if (! $this->filesystem->has($this->targetDir)) { |
|
63 | - $this->filesystem->createDir($this->targetDir); |
|
62 | + if ( ! $this->filesystem->has( $this->targetDir ) ) { |
|
63 | + $this->filesystem->createDir( $this->targetDir ); |
|
64 | 64 | } else { |
65 | - foreach ($this->filepaths as $vendorRelativeFilepath) { |
|
65 | + foreach ( $this->filepaths as $vendorRelativeFilepath ) { |
|
66 | 66 | $projectRelativeFilepath = $this->targetDir . $vendorRelativeFilepath; |
67 | 67 | |
68 | - if ($this->filesystem->has($projectRelativeFilepath)) { |
|
69 | - $this->filesystem->delete($projectRelativeFilepath); |
|
68 | + if ( $this->filesystem->has( $projectRelativeFilepath ) ) { |
|
69 | + $this->filesystem->delete( $projectRelativeFilepath ); |
|
70 | 70 | } |
71 | 71 | } |
72 | 72 | } |
@@ -79,12 +79,12 @@ discard block |
||
79 | 79 | public function copy(): void |
80 | 80 | { |
81 | 81 | |
82 | - foreach ($this->filepaths as $relativeFilepath) { |
|
82 | + foreach ( $this->filepaths as $relativeFilepath ) { |
|
83 | 83 | $sourceFileRelativePath = 'vendor' . DIRECTORY_SEPARATOR . $relativeFilepath; |
84 | 84 | |
85 | 85 | $targetFileRelativePath = $this->targetDir . $relativeFilepath; |
86 | 86 | |
87 | - $this->filesystem->copy($sourceFileRelativePath, $targetFileRelativePath); |
|
87 | + $this->filesystem->copy( $sourceFileRelativePath, $targetFileRelativePath ); |
|
88 | 88 | } |
89 | 89 | } |
90 | 90 | } |
@@ -17,8 +17,7 @@ discard block |
||
17 | 17 | use League\Flysystem\Adapter\Local; |
18 | 18 | use League\Flysystem\Filesystem; |
19 | 19 | |
20 | -class Copier |
|
21 | -{ |
|
20 | +class Copier { |
|
22 | 21 | /** |
23 | 22 | * The only path variable with a leading slash. |
24 | 23 | * All directories in project end with a slash. |
@@ -40,8 +39,7 @@ discard block |
||
40 | 39 | * @param string $workingDir |
41 | 40 | * @param string $relativeTargetDir |
42 | 41 | */ |
43 | - public function __construct(array $filepaths, string $workingDir, string $relativeTargetDir) |
|
44 | - { |
|
42 | + public function __construct(array $filepaths, string $workingDir, string $relativeTargetDir) { |
|
45 | 43 | $this->filepaths = array_keys($filepaths); |
46 | 44 | |
47 | 45 | $this->workingDir = $workingDir; |