@@ -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 | |
@@ -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' |
@@ -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 | { |
@@ -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"); |
@@ -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 | |
@@ -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 | { |
@@ -22,14 +22,12 @@ |
||
| 22 | 22 | * Class MozartIssue99Test |
| 23 | 23 | * @coversNothing |
| 24 | 24 | */ |
| 25 | -class MozartIssue99Test extends IntegrationTestCase |
|
| 26 | -{ |
|
| 25 | +class MozartIssue99Test extends IntegrationTestCase { |
|
| 27 | 26 | |
| 28 | 27 | /** |
| 29 | 28 | * WooCommerce Action Scheduler ... has no autoload key. But also needs some Mozart patches to work correctly. |
| 30 | 29 | */ |
| 31 | - public function test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
| 32 | - { |
|
| 30 | + public function test_it_does_not_make_classname_replacement_inside_namespaced_file() { |
|
| 33 | 31 | |
| 34 | 32 | $composerJsonString = <<<'EOD' |
| 35 | 33 | { |
@@ -20,14 +20,12 @@ discard block |
||
| 20 | 20 | * @package BrianHenryIE\Strauss\Tests\Issues |
| 21 | 21 | * @coversNothing |
| 22 | 22 | */ |
| 23 | -class StraussIssue11Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase |
|
| 24 | -{ |
|
| 23 | +class StraussIssue11Test extends \BrianHenryIE\Strauss\Tests\Integration\Util\IntegrationTestCase { |
|
| 25 | 24 | |
| 26 | 25 | /** |
| 27 | 26 | * @author BrianHenryIE |
| 28 | 27 | */ |
| 29 | - public function test_migrate_mozart_config() |
|
| 30 | - { |
|
| 28 | + public function test_migrate_mozart_config() { |
|
| 31 | 29 | |
| 32 | 30 | $composerExtraStraussJson = <<<'EOD' |
| 33 | 31 | { |
@@ -78,8 +76,7 @@ discard block |
||
| 78 | 76 | /** |
| 79 | 77 | * @author BrianHenryIE |
| 80 | 78 | */ |
| 81 | - public function test_carbon_fields() |
|
| 82 | - { |
|
| 79 | + public function test_carbon_fields() { |
|
| 83 | 80 | |
| 84 | 81 | $composerJsonString = <<<'EOD' |
| 85 | 82 | { |
@@ -139,8 +136,7 @@ discard block |
||
| 139 | 136 | /** |
| 140 | 137 | * @author BrianHenryIE |
| 141 | 138 | */ |
| 142 | - public function test_static_namespace() |
|
| 143 | - { |
|
| 139 | + public function test_static_namespace() { |
|
| 144 | 140 | |
| 145 | 141 | $composerJsonString = <<<'EOD' |
| 146 | 142 | { |
@@ -23,16 +23,14 @@ discard block |
||
| 23 | 23 | * @package BrianHenryIE\Strauss\Tests\Issues |
| 24 | 24 | * @coversNothing |
| 25 | 25 | */ |
| 26 | -class MozartIssue129Test extends TestCase |
|
| 27 | -{ |
|
| 26 | +class MozartIssue129Test extends TestCase { |
|
| 28 | 27 | |
| 29 | 28 | /** |
| 30 | 29 | * @author BrianHenryIE |
| 31 | 30 | * |
| 32 | 31 | * @dataProvider pairTestDataProvider |
| 33 | 32 | */ |
| 34 | - public function test_test($phpString, $expected) |
|
| 35 | - { |
|
| 33 | + public function test_test($phpString, $expected) { |
|
| 36 | 34 | |
| 37 | 35 | $config = $this->createMock(StraussConfig::class); |
| 38 | 36 | |
@@ -46,8 +44,7 @@ discard block |
||
| 46 | 44 | $this->assertEquals($expected, $result); |
| 47 | 45 | } |
| 48 | 46 | |
| 49 | - public function pairTestDataProvider() |
|
| 50 | - { |
|
| 47 | + public function pairTestDataProvider() { |
|
| 51 | 48 | |
| 52 | 49 | $fromTo = []; |
| 53 | 50 | |