@@ -11,14 +11,12 @@ discard block |
||
| 11 | 11 | use Composer\IO\NullIO; |
| 12 | 12 | use PHPUnit\Framework\TestCase; |
| 13 | 13 | |
| 14 | -class StraussConfigTest extends TestCase |
|
| 15 | -{ |
|
| 14 | +class StraussConfigTest extends TestCase { |
|
| 16 | 15 | |
| 17 | 16 | /** |
| 18 | 17 | * With a full (at time of writing) config, test the getters. |
| 19 | 18 | */ |
| 20 | - public function testGetters() |
|
| 21 | - { |
|
| 19 | + public function testGetters() { |
|
| 22 | 20 | |
| 23 | 21 | $composerExtraStraussJson = <<<'EOD' |
| 24 | 22 | { |
@@ -78,8 +76,7 @@ discard block |
||
| 78 | 76 | * |
| 79 | 77 | * Turns out it just ignores it... good! |
| 80 | 78 | */ |
| 81 | - public function testExtraKey() |
|
| 82 | - { |
|
| 79 | + public function testExtraKey() { |
|
| 83 | 80 | |
| 84 | 81 | $composerExtraStraussJson = <<<'EOD' |
| 85 | 82 | { |
@@ -133,8 +130,7 @@ discard block |
||
| 133 | 130 | * |
| 134 | 131 | * If no target_dir is specified, used "strauss/" |
| 135 | 132 | */ |
| 136 | - public function testDefaultTargetDir() |
|
| 137 | - { |
|
| 133 | + public function testDefaultTargetDir() { |
|
| 138 | 134 | |
| 139 | 135 | $composerExtraStraussJson = <<<'EOD' |
| 140 | 136 | { |
@@ -176,8 +172,7 @@ discard block |
||
| 176 | 172 | /** |
| 177 | 173 | * When the namespace prefix isn't provided, use the PSR-4 autoload key name. |
| 178 | 174 | */ |
| 179 | - public function testDefaultNamespacePrefixFromAutoloaderPsr4() |
|
| 180 | - { |
|
| 175 | + public function testDefaultNamespacePrefixFromAutoloaderPsr4() { |
|
| 181 | 176 | |
| 182 | 177 | $composerExtraStraussJson = <<<'EOD' |
| 183 | 178 | { |
@@ -207,8 +202,7 @@ discard block |
||
| 207 | 202 | /** |
| 208 | 203 | * When the namespace prefix isn't provided, use the PSR-0 autoload key name. |
| 209 | 204 | */ |
| 210 | - public function testDefaultNamespacePrefixFromAutoloaderPsr0() |
|
| 211 | - { |
|
| 205 | + public function testDefaultNamespacePrefixFromAutoloaderPsr0() { |
|
| 212 | 206 | |
| 213 | 207 | $composerExtraStraussJson = <<<'EOD' |
| 214 | 208 | { |
@@ -239,8 +233,7 @@ discard block |
||
| 239 | 233 | * |
| 240 | 234 | * brianhenryie/strauss-config-test |
| 241 | 235 | */ |
| 242 | - public function testDefaultNamespacePrefixWithNoAutoloader() |
|
| 243 | - { |
|
| 236 | + public function testDefaultNamespacePrefixWithNoAutoloader() { |
|
| 244 | 237 | |
| 245 | 238 | $composerExtraStraussJson = <<<'EOD' |
| 246 | 239 | { |
@@ -264,8 +257,7 @@ discard block |
||
| 264 | 257 | /** |
| 265 | 258 | * When the classmap prefix isn't provided, use the PSR-4 autoload key name. |
| 266 | 259 | */ |
| 267 | - public function testDefaultClassmapPrefixFromAutoloaderPsr4() |
|
| 268 | - { |
|
| 260 | + public function testDefaultClassmapPrefixFromAutoloaderPsr4() { |
|
| 269 | 261 | |
| 270 | 262 | $composerExtraStraussJson = <<<'EOD' |
| 271 | 263 | { |
@@ -295,8 +287,7 @@ discard block |
||
| 295 | 287 | /** |
| 296 | 288 | * When the classmap prefix isn't provided, use the PSR-0 autoload key name. |
| 297 | 289 | */ |
| 298 | - public function testDefaultClassmapPrefixFromAutoloaderPsr0() |
|
| 299 | - { |
|
| 290 | + public function testDefaultClassmapPrefixFromAutoloaderPsr0() { |
|
| 300 | 291 | |
| 301 | 292 | $composerExtraStraussJson = <<<'EOD' |
| 302 | 293 | { |
@@ -328,8 +319,7 @@ discard block |
||
| 328 | 319 | * |
| 329 | 320 | * brianhenryie/strauss-config-test |
| 330 | 321 | */ |
| 331 | - public function testDefaultClassmapPrefixWithNoAutoloader() |
|
| 332 | - { |
|
| 322 | + public function testDefaultClassmapPrefixWithNoAutoloader() { |
|
| 333 | 323 | |
| 334 | 324 | $composerExtraStraussJson = <<<'EOD' |
| 335 | 325 | { |
@@ -353,8 +343,7 @@ discard block |
||
| 353 | 343 | /** |
| 354 | 344 | * When Strauss config has packages specified, obviously use them. |
| 355 | 345 | */ |
| 356 | - public function testGetPackagesFromConfig() |
|
| 357 | - { |
|
| 346 | + public function testGetPackagesFromConfig() { |
|
| 358 | 347 | |
| 359 | 348 | $composerExtraStraussJson = <<<'EOD' |
| 360 | 349 | { |
@@ -399,8 +388,7 @@ discard block |
||
| 399 | 388 | /** |
| 400 | 389 | * When Strauss config has no packages specified, use composer.json's require list. |
| 401 | 390 | */ |
| 402 | - public function testGetPackagesNoConfig() |
|
| 403 | - { |
|
| 391 | + public function testGetPackagesNoConfig() { |
|
| 404 | 392 | |
| 405 | 393 | $composerExtraStraussJson = <<<'EOD' |
| 406 | 394 | { |
@@ -441,8 +429,7 @@ discard block |
||
| 441 | 429 | /** |
| 442 | 430 | * For backwards compatibility, if a Mozart config is present, use it. |
| 443 | 431 | */ |
| 444 | - public function testMapMozartConfig() |
|
| 445 | - { |
|
| 432 | + public function testMapMozartConfig() { |
|
| 446 | 433 | |
| 447 | 434 | $composerExtraStraussJson = <<<'EOD' |
| 448 | 435 | { |
@@ -500,8 +487,7 @@ discard block |
||
| 500 | 487 | * |
| 501 | 488 | * @covers \BrianHenryIE\Strauss\Composer\Extra\StraussConfig::getNamespacePrefix |
| 502 | 489 | */ |
| 503 | - public function testNamespacePrefixHasNoSlash() |
|
| 504 | - { |
|
| 490 | + public function testNamespacePrefixHasNoSlash() { |
|
| 505 | 491 | |
| 506 | 492 | $composerExtraStraussJson = <<<'EOD' |
| 507 | 493 | { |
@@ -5,14 +5,12 @@ discard block |
||
| 5 | 5 | use BrianHenryIE\Strauss\Composer\ComposerPackage; |
| 6 | 6 | use PHPUnit\Framework\TestCase; |
| 7 | 7 | |
| 8 | -class ComposerPackageTest extends TestCase |
|
| 9 | -{ |
|
| 8 | +class ComposerPackageTest extends TestCase { |
|
| 10 | 9 | |
| 11 | 10 | /** |
| 12 | 11 | * A simple test to check the getters all work. |
| 13 | 12 | */ |
| 14 | - public function testParseJson() |
|
| 15 | - { |
|
| 13 | + public function testParseJson() { |
|
| 16 | 14 | |
| 17 | 15 | $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
| 18 | 16 | |
@@ -28,8 +26,7 @@ discard block |
||
| 28 | 26 | /** |
| 29 | 27 | * Test the dependencies' names are returned. |
| 30 | 28 | */ |
| 31 | - public function testGetRequiresNames() |
|
| 32 | - { |
|
| 29 | + public function testGetRequiresNames() { |
|
| 33 | 30 | |
| 34 | 31 | $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
| 35 | 32 | |
@@ -44,8 +41,7 @@ discard block |
||
| 44 | 41 | /** |
| 45 | 42 | * Test PHP and ext- are not returned, since we won't be dealing with them. |
| 46 | 43 | */ |
| 47 | - public function testGetRequiresNamesDoesNotContain() |
|
| 48 | - { |
|
| 44 | + public function testGetRequiresNamesDoesNotContain() { |
|
| 49 | 45 | |
| 50 | 46 | $testFile = __DIR__ . '/composerpackage-test-easypost-php.json'; |
| 51 | 47 | |
@@ -61,8 +57,7 @@ discard block |
||
| 61 | 57 | /** |
| 62 | 58 | * |
| 63 | 59 | */ |
| 64 | - public function testAutoloadPsr0() |
|
| 65 | - { |
|
| 60 | + public function testAutoloadPsr0() { |
|
| 66 | 61 | |
| 67 | 62 | $testFile = __DIR__ . '/composerpackage-test-easypost-php.json'; |
| 68 | 63 | |
@@ -78,8 +73,7 @@ discard block |
||
| 78 | 73 | /** |
| 79 | 74 | * |
| 80 | 75 | */ |
| 81 | - public function testAutoloadPsr4() |
|
| 82 | - { |
|
| 76 | + public function testAutoloadPsr4() { |
|
| 83 | 77 | |
| 84 | 78 | $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
| 85 | 79 | |
@@ -95,8 +89,7 @@ discard block |
||
| 95 | 89 | /** |
| 96 | 90 | * |
| 97 | 91 | */ |
| 98 | - public function testAutoloadClassmap() |
|
| 99 | - { |
|
| 92 | + public function testAutoloadClassmap() { |
|
| 100 | 93 | |
| 101 | 94 | $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json'; |
| 102 | 95 | |
@@ -112,8 +105,7 @@ discard block |
||
| 112 | 105 | /** |
| 113 | 106 | * |
| 114 | 107 | */ |
| 115 | - public function testAutoloadFiles() |
|
| 116 | - { |
|
| 108 | + public function testAutoloadFiles() { |
|
| 117 | 109 | |
| 118 | 110 | $testFile = __DIR__ . '/composerpackage-test-php-di.json'; |
| 119 | 111 | |
@@ -126,16 +118,14 @@ discard block |
||
| 126 | 118 | $this->assertIsArray($autoload['files']); |
| 127 | 119 | } |
| 128 | 120 | |
| 129 | - public function testOverrideAutoload() |
|
| 130 | - { |
|
| 121 | + public function testOverrideAutoload() { |
|
| 131 | 122 | $this->markTestIncomplete(); |
| 132 | 123 | } |
| 133 | 124 | |
| 134 | 125 | /** |
| 135 | 126 | * When composer.json is not where it was specified, what error message (via Exception) should be returned? |
| 136 | 127 | */ |
| 137 | - public function testMissingComposer() |
|
| 138 | - { |
|
| 128 | + public function testMissingComposer() { |
|
| 139 | 129 | $this->markTestIncomplete(); |
| 140 | 130 | } |
| 141 | 131 | } |
@@ -6,14 +6,12 @@ |
||
| 6 | 6 | use BrianHenryIE\Strauss\Composer\ProjectComposerPackage; |
| 7 | 7 | use PHPUnit\Framework\TestCase; |
| 8 | 8 | |
| 9 | -class ProjectComposerPackageTest extends TestCase |
|
| 10 | -{ |
|
| 9 | +class ProjectComposerPackageTest extends TestCase { |
|
| 11 | 10 | |
| 12 | 11 | /** |
| 13 | 12 | * A simple test to check the getters all work. |
| 14 | 13 | */ |
| 15 | - public function testParseJson() |
|
| 16 | - { |
|
| 14 | + public function testParseJson() { |
|
| 17 | 15 | |
| 18 | 16 | $testFile = __DIR__ . '/projectcomposerpackage-test-1.json'; |
| 19 | 17 | |
@@ -9,8 +9,7 @@ discard block |
||
| 9 | 9 | use Composer\Composer; |
| 10 | 10 | use PHPUnit\Framework\TestCase; |
| 11 | 11 | |
| 12 | -class ChangeEnumeratorTest extends TestCase |
|
| 13 | -{ |
|
| 12 | +class ChangeEnumeratorTest extends TestCase { |
|
| 14 | 13 | |
| 15 | 14 | // PREG_BACKTRACK_LIMIT_ERROR |
| 16 | 15 | |
@@ -21,8 +20,7 @@ discard block |
||
| 21 | 20 | |
| 22 | 21 | |
| 23 | 22 | |
| 24 | - public function testSingleNamespace() |
|
| 25 | - { |
|
| 23 | + public function testSingleNamespace() { |
|
| 26 | 24 | |
| 27 | 25 | $validPhp = <<<'EOD' |
| 28 | 26 | <?php |
@@ -44,8 +42,7 @@ discard block |
||
| 44 | 42 | $this->assertNotContains('MyClass', $sut->getDiscoveredClasses()); |
| 45 | 43 | } |
| 46 | 44 | |
| 47 | - public function testGlobalNamespace() |
|
| 48 | - { |
|
| 45 | + public function testGlobalNamespace() { |
|
| 49 | 46 | |
| 50 | 47 | $validPhp = <<<'EOD' |
| 51 | 48 | <?php |
@@ -66,8 +63,7 @@ discard block |
||
| 66 | 63 | /** |
| 67 | 64 | * |
| 68 | 65 | */ |
| 69 | - public function testMultipleNamespace() |
|
| 70 | - { |
|
| 66 | + public function testMultipleNamespace() { |
|
| 71 | 67 | |
| 72 | 68 | $validPhp = <<<'EOD' |
| 73 | 69 | <?php |
@@ -93,8 +89,7 @@ discard block |
||
| 93 | 89 | /** |
| 94 | 90 | * |
| 95 | 91 | */ |
| 96 | - public function testMultipleNamespaceGlobalFirst() |
|
| 97 | - { |
|
| 92 | + public function testMultipleNamespaceGlobalFirst() { |
|
| 98 | 93 | |
| 99 | 94 | $validPhp = <<<'EOD' |
| 100 | 95 | <?php |
@@ -124,8 +119,7 @@ discard block |
||
| 124 | 119 | /** |
| 125 | 120 | * |
| 126 | 121 | */ |
| 127 | - public function testMultipleClasses() |
|
| 128 | - { |
|
| 122 | + public function testMultipleClasses() { |
|
| 129 | 123 | |
| 130 | 124 | $validPhp = <<<'EOD' |
| 131 | 125 | <?php |
@@ -149,8 +143,7 @@ discard block |
||
| 149 | 143 | * |
| 150 | 144 | * @author BrianHenryIE |
| 151 | 145 | */ |
| 152 | - public function test_it_does_not_treat_comments_as_classes() |
|
| 153 | - { |
|
| 146 | + public function test_it_does_not_treat_comments_as_classes() { |
|
| 154 | 147 | $contents = " |
| 155 | 148 | // A class as good as any. |
| 156 | 149 | class Whatever { |
@@ -170,8 +163,7 @@ discard block |
||
| 170 | 163 | * |
| 171 | 164 | * @author BrianHenryIE |
| 172 | 165 | */ |
| 173 | - public function test_it_does_not_treat_multiline_comments_as_classes() |
|
| 174 | - { |
|
| 166 | + public function test_it_does_not_treat_multiline_comments_as_classes() { |
|
| 175 | 167 | $contents = " |
| 176 | 168 | /** |
| 177 | 169 | * A class as good as any; class as. |
@@ -195,8 +187,7 @@ discard block |
||
| 195 | 187 | * |
| 196 | 188 | * @author BrianHenryIE |
| 197 | 189 | */ |
| 198 | - public function test_it_does_not_treat_multiline_comments_opening_line_as_classes() |
|
| 199 | - { |
|
| 190 | + public function test_it_does_not_treat_multiline_comments_opening_line_as_classes() { |
|
| 200 | 191 | $contents = " |
| 201 | 192 | /** A class as good as any; class as. |
| 202 | 193 | * |
@@ -218,8 +209,7 @@ discard block |
||
| 218 | 209 | * |
| 219 | 210 | * @author BrianHenryIE |
| 220 | 211 | */ |
| 221 | - public function test_it_does_not_treat_multiline_comments_on_one_line_as_classes() |
|
| 222 | - { |
|
| 212 | + public function test_it_does_not_treat_multiline_comments_on_one_line_as_classes() { |
|
| 223 | 213 | $contents = " |
| 224 | 214 | /** A class as good as any; class as. */ class Whatever_Trevor { |
| 225 | 215 | } |
@@ -240,8 +230,7 @@ discard block |
||
| 240 | 230 | * |
| 241 | 231 | * @test |
| 242 | 232 | */ |
| 243 | - public function test_it_does_not_treat_comments_with_semicolons_as_classes() |
|
| 244 | - { |
|
| 233 | + public function test_it_does_not_treat_comments_with_semicolons_as_classes() { |
|
| 245 | 234 | $contents = " |
| 246 | 235 | // A class as good as any; class as versatile as any. |
| 247 | 236 | class Whatever_Ever { |
@@ -260,8 +249,7 @@ discard block |
||
| 260 | 249 | /** |
| 261 | 250 | * @author BrianHenryIE |
| 262 | 251 | */ |
| 263 | - public function test_it_parses_classes_after_semicolon() |
|
| 264 | - { |
|
| 252 | + public function test_it_parses_classes_after_semicolon() { |
|
| 265 | 253 | |
| 266 | 254 | $contents = " |
| 267 | 255 | myvar = 123; class Pear { }; |
@@ -278,8 +266,7 @@ discard block |
||
| 278 | 266 | /** |
| 279 | 267 | * @author BrianHenryIE |
| 280 | 268 | */ |
| 281 | - public function test_it_parses_classes_followed_by_comment() |
|
| 282 | - { |
|
| 269 | + public function test_it_parses_classes_followed_by_comment() { |
|
| 283 | 270 | |
| 284 | 271 | $contents = <<<'EOD' |
| 285 | 272 | class WP_Dependency_Installer { |
@@ -325,8 +312,7 @@ discard block |
||
| 325 | 312 | $this->assertContains('B_Class', $changeEnumerator->getDiscoveredClasses()); |
| 326 | 313 | } |
| 327 | 314 | |
| 328 | - public function testExcludePackagesFromPrefix() |
|
| 329 | - { |
|
| 315 | + public function testExcludePackagesFromPrefix() { |
|
| 330 | 316 | |
| 331 | 317 | $config = $this->createMock(StraussConfig::class); |
| 332 | 318 | $config->method('getExcludePackagesFromPrefixing')->willReturn( |
@@ -345,8 +331,7 @@ discard block |
||
| 345 | 331 | } |
| 346 | 332 | |
| 347 | 333 | |
| 348 | - public function testExcludeFilePatternsFromPrefix() |
|
| 349 | - { |
|
| 334 | + public function testExcludeFilePatternsFromPrefix() { |
|
| 350 | 335 | $config = $this->createMock(StraussConfig::class); |
| 351 | 336 | $config->method('getExcludeFilePatternsFromPrefixing')->willReturn( |
| 352 | 337 | array('/to/') |
@@ -366,8 +351,7 @@ discard block |
||
| 366 | 351 | /** |
| 367 | 352 | * Test custom replacements |
| 368 | 353 | */ |
| 369 | - public function testNamespaceReplacementPatterns() |
|
| 370 | - { |
|
| 354 | + public function testNamespaceReplacementPatterns() { |
|
| 371 | 355 | |
| 372 | 356 | $contents = " |
| 373 | 357 | namespace BrianHenryIE\PdfHelpers { |
@@ -392,8 +376,7 @@ discard block |
||
| 392 | 376 | /** |
| 393 | 377 | * @see https://github.com/BrianHenryIE/strauss/issues/19 |
| 394 | 378 | */ |
| 395 | - public function testPhraseClassObjectIsNotMistaken() |
|
| 396 | - { |
|
| 379 | + public function testPhraseClassObjectIsNotMistaken() { |
|
| 397 | 380 | |
| 398 | 381 | $contents = <<<'EOD' |
| 399 | 382 | <?php |
@@ -20,8 +20,7 @@ discard block |
||
| 20 | 20 | * @package BrianHenryIE\Strauss |
| 21 | 21 | * @covers \BrianHenryIE\Strauss\Prefixer |
| 22 | 22 | */ |
| 23 | -class PrefixerTest extends TestCase |
|
| 24 | -{ |
|
| 23 | +class PrefixerTest extends TestCase { |
|
| 25 | 24 | |
| 26 | 25 | protected StraussConfig $config; |
| 27 | 26 | |
@@ -46,8 +45,7 @@ discard block |
||
| 46 | 45 | $this->config = new StraussConfig($composer); |
| 47 | 46 | } |
| 48 | 47 | |
| 49 | - public function testNamespaceReplacer() |
|
| 50 | - { |
|
| 48 | + public function testNamespaceReplacer() { |
|
| 51 | 49 | |
| 52 | 50 | $contents = <<<'EOD' |
| 53 | 51 | <?php |
@@ -137,8 +135,7 @@ discard block |
||
| 137 | 135 | } |
| 138 | 136 | |
| 139 | 137 | |
| 140 | - public function testClassnameReplacer() |
|
| 141 | - { |
|
| 138 | + public function testClassnameReplacer() { |
|
| 142 | 139 | |
| 143 | 140 | $contents = <<<'EOD' |
| 144 | 141 | <?php |
@@ -178,8 +175,7 @@ discard block |
||
| 178 | 175 | /** |
| 179 | 176 | * PHP 7.4 typed parameters were being prefixed. |
| 180 | 177 | */ |
| 181 | - public function testTypeFunctionParameter() |
|
| 182 | - { |
|
| 178 | + public function testTypeFunctionParameter() { |
|
| 183 | 179 | $this->markTestIncomplete(); |
| 184 | 180 | } |
| 185 | 181 | |
@@ -552,8 +548,7 @@ discard block |
||
| 552 | 548 | /** |
| 553 | 549 | * @author BrianHenryIE |
| 554 | 550 | */ |
| 555 | - public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace() |
|
| 556 | - { |
|
| 551 | + public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace() { |
|
| 557 | 552 | $namespace = 'Mpdf'; |
| 558 | 553 | $prefix = "Mozart"; |
| 559 | 554 | $contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
@@ -568,8 +563,7 @@ discard block |
||
| 568 | 563 | $this->assertEquals($expected, $result); |
| 569 | 564 | } |
| 570 | 565 | |
| 571 | - public function testLeadingSlashInString() |
|
| 572 | - { |
|
| 566 | + public function testLeadingSlashInString() { |
|
| 573 | 567 | $originalNamespace = "Strauss\\Test"; |
| 574 | 568 | $replacement = "Prefix\\Strauss\\Test"; |
| 575 | 569 | $contents = '$mentionedClass = "\\Strauss\\Test\\Classname";'; |
@@ -584,8 +578,7 @@ discard block |
||
| 584 | 578 | $this->assertEquals($expected, $result); |
| 585 | 579 | } |
| 586 | 580 | |
| 587 | - public function testDoubleLeadingSlashInString() |
|
| 588 | - { |
|
| 581 | + public function testDoubleLeadingSlashInString() { |
|
| 589 | 582 | $originalNamespace = "Strauss\\Test"; |
| 590 | 583 | $replacement = "Prefix\\Strauss\\Test"; |
| 591 | 584 | $contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";'; |
@@ -600,8 +593,7 @@ discard block |
||
| 600 | 593 | $this->assertEquals($expected, $result); |
| 601 | 594 | } |
| 602 | 595 | |
| 603 | - public function testItReplacesSlashedNamespaceInFunctionParameter() |
|
| 604 | - { |
|
| 596 | + public function testItReplacesSlashedNamespaceInFunctionParameter() { |
|
| 605 | 597 | |
| 606 | 598 | $originalNamespace = "net\\authorize\\api\\contract\\v1"; |
| 607 | 599 | $replacement = "Prefix\\net\\authorize\\api\\contract\\v1"; |
@@ -618,8 +610,7 @@ discard block |
||
| 618 | 610 | } |
| 619 | 611 | |
| 620 | 612 | |
| 621 | - public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue() |
|
| 622 | - { |
|
| 613 | + public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue() { |
|
| 623 | 614 | |
| 624 | 615 | $originalNamespace = "net\\authorize\\api\constants"; |
| 625 | 616 | $replacement = "Prefix\\net\\authorize\\api\constants"; |
@@ -636,8 +627,7 @@ discard block |
||
| 636 | 627 | } |
| 637 | 628 | |
| 638 | 629 | |
| 639 | - public function testItReplacesNamespaceConcatenatedStringConst() |
|
| 640 | - { |
|
| 630 | + public function testItReplacesNamespaceConcatenatedStringConst() { |
|
| 641 | 631 | |
| 642 | 632 | $originalNamespace = "net\\authorize\\api\\constants"; |
| 643 | 633 | $replacement = "Prefix\\net\\authorize\\api\\constants"; |
@@ -657,8 +647,7 @@ discard block |
||
| 657 | 647 | /** |
| 658 | 648 | * Another mpdf issue where the class "Mpdf" is in the namespace "Mpdf" and incorrect replacements are being made. |
| 659 | 649 | */ |
| 660 | - public function testClassnameNotConfusedWithNamespace() |
|
| 661 | - { |
|
| 650 | + public function testClassnameNotConfusedWithNamespace() { |
|
| 662 | 651 | |
| 663 | 652 | $contents = '$default_font_size = $mmsize * (Mpdf::SCALE);'; |
| 664 | 653 | $expected = $contents; |
@@ -671,8 +660,7 @@ discard block |
||
| 671 | 660 | $this->assertEquals($expected, $result); |
| 672 | 661 | } |
| 673 | 662 | |
| 674 | - public function testClassExtendsNamspacedClassIsPrefixed() |
|
| 675 | - { |
|
| 663 | + public function testClassExtendsNamspacedClassIsPrefixed() { |
|
| 676 | 664 | |
| 677 | 665 | $contents = 'class BarcodeException extends \Mpdf\MpdfException'; |
| 678 | 666 | $expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException'; |
@@ -690,8 +678,7 @@ discard block |
||
| 690 | 678 | * |
| 691 | 679 | * @see https://github.com/BrianHenryIE/strauss/issues/11 |
| 692 | 680 | */ |
| 693 | - public function testNewNamespacedClassIsPrefixed() |
|
| 694 | - { |
|
| 681 | + public function testNewNamespacedClassIsPrefixed() { |
|
| 695 | 682 | |
| 696 | 683 | $contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );'; |
| 697 | 684 | $expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );'; |
@@ -711,8 +698,7 @@ discard block |
||
| 711 | 698 | * |
| 712 | 699 | * @see https://github.com/BrianHenryIE/strauss/issues/11 |
| 713 | 700 | */ |
| 714 | - public function testStaticNamespacedClassIsPrefixed() |
|
| 715 | - { |
|
| 701 | + public function testStaticNamespacedClassIsPrefixed() { |
|
| 716 | 702 | |
| 717 | 703 | $contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container'; |
| 718 | 704 | $expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container'; |
@@ -730,8 +716,7 @@ discard block |
||
| 730 | 716 | * |
| 731 | 717 | * @see https://github.com/BrianHenryIE/strauss/issues/11 |
| 732 | 718 | */ |
| 733 | - public function testReturnedNamespacedClassIsPrefixed() |
|
| 734 | - { |
|
| 719 | + public function testReturnedNamespacedClassIsPrefixed() { |
|
| 735 | 720 | |
| 736 | 721 | $contents = 'return \Carbon_Fields\Carbon_Fields::resolve'; |
| 737 | 722 | $expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve'; |
@@ -749,8 +734,7 @@ discard block |
||
| 749 | 734 | * |
| 750 | 735 | * @see https://github.com/BrianHenryIE/strauss/issues/11 |
| 751 | 736 | */ |
| 752 | - public function testNamespacedStaticIsPrefixed() |
|
| 753 | - { |
|
| 737 | + public function testNamespacedStaticIsPrefixed() { |
|
| 754 | 738 | |
| 755 | 739 | $contents = " \\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
| 756 | 740 | $expected = " \\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
@@ -772,8 +756,7 @@ discard block |
||
| 772 | 756 | * |
| 773 | 757 | * @see https://github.com/BrianHenryIE/strauss/issues/15 |
| 774 | 758 | */ |
| 775 | - public function testDoNotReplaceInStringThatIsNotCode() |
|
| 776 | - { |
|
| 759 | + public function testDoNotReplaceInStringThatIsNotCode() { |
|
| 777 | 760 | $originalNamespace = "TrustedLogin"; |
| 778 | 761 | $replacement = "Prefix\\TrustedLogin"; |
| 779 | 762 | $contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
@@ -795,8 +778,7 @@ discard block |
||
| 795 | 778 | * @see https://github.com/BrianHenryIE/strauss/issues/19 |
| 796 | 779 | * |
| 797 | 780 | */ |
| 798 | - public function testDoNotReplaceInVariableNames() |
|
| 799 | - { |
|
| 781 | + public function testDoNotReplaceInVariableNames() { |
|
| 800 | 782 | $originalClassname = 'object'; |
| 801 | 783 | $classnamePrefix = 'Strauss_Issue19_'; |
| 802 | 784 | $contents = "public static function objclone(\$object) {"; |
@@ -18,16 +18,14 @@ discard block |
||
| 18 | 18 | * @package BrianHenryIE\Strauss\Tests\Unit |
| 19 | 19 | * @coversDefaultClass \BrianHenryIE\Strauss\Licenser |
| 20 | 20 | */ |
| 21 | -class LicenserTest extends TestCase |
|
| 22 | -{ |
|
| 21 | +class LicenserTest extends TestCase { |
|
| 23 | 22 | |
| 24 | 23 | |
| 25 | 24 | /** |
| 26 | 25 | * @covers ::findLicenseFiles |
| 27 | 26 | * |
| 28 | 27 | */ |
| 29 | - public function testFindLicenceFilesPathsAreRelative() |
|
| 30 | - { |
|
| 28 | + public function testFindLicenceFilesPathsAreRelative() { |
|
| 31 | 29 | $config = $this->createStub(StraussConfig::class); |
| 32 | 30 | $workingDir = __DIR__ . DIRECTORY_SEPARATOR; |
| 33 | 31 | |
@@ -79,8 +77,7 @@ discard block |
||
| 79 | 77 | * |
| 80 | 78 | * @covers ::addChangeDeclarationToPhpString |
| 81 | 79 | */ |
| 82 | - public function testAppendHeaderCommentInformationNoHeader() |
|
| 83 | - { |
|
| 80 | + public function testAppendHeaderCommentInformationNoHeader() { |
|
| 84 | 81 | |
| 85 | 82 | $author = 'BrianHenryIE'; |
| 86 | 83 | |
@@ -166,8 +163,7 @@ discard block |
||
| 166 | 163 | * |
| 167 | 164 | * @covers ::addChangeDeclarationToPhpString |
| 168 | 165 | */ |
| 169 | - public function testWithTwoCommentsBeforeFirstCode() |
|
| 170 | - { |
|
| 166 | + public function testWithTwoCommentsBeforeFirstCode() { |
|
| 171 | 167 | |
| 172 | 168 | $config = $this->createStub(StraussConfig::class); |
| 173 | 169 | $author = 'BrianHenryIE'; |
@@ -227,8 +223,7 @@ discard block |
||
| 227 | 223 | /** |
| 228 | 224 | * @covers ::addChangeDeclarationToPhpString |
| 229 | 225 | */ |
| 230 | - public function testUnusualHeaderCommentStyle() |
|
| 231 | - { |
|
| 226 | + public function testUnusualHeaderCommentStyle() { |
|
| 232 | 227 | |
| 233 | 228 | $config = $this->createStub(StraussConfig::class); |
| 234 | 229 | $author = 'BrianHenryIE'; |
@@ -272,8 +267,7 @@ discard block |
||
| 272 | 267 | /** |
| 273 | 268 | * @covers ::addChangeDeclarationToPhpString |
| 274 | 269 | */ |
| 275 | - public function testCommentWithLicenseWord() |
|
| 276 | - { |
|
| 270 | + public function testCommentWithLicenseWord() { |
|
| 277 | 271 | |
| 278 | 272 | $config = $this->createStub(StraussConfig::class); |
| 279 | 273 | $author = 'BrianHenryIE'; |
@@ -331,8 +325,7 @@ discard block |
||
| 331 | 325 | * |
| 332 | 326 | * Seems files loaded are treated different to strings passed. |
| 333 | 327 | */ |
| 334 | - public function testIncorrectlyMatching() |
|
| 335 | - { |
|
| 328 | + public function testIncorrectlyMatching() { |
|
| 336 | 329 | |
| 337 | 330 | $config = $this->createStub(StraussConfig::class); |
| 338 | 331 | $author = 'BrianHenryIE'; |
@@ -401,8 +394,7 @@ discard block |
||
| 401 | 394 | /** |
| 402 | 395 | * The licence was being inserted after every `<?php` in the file. |
| 403 | 396 | */ |
| 404 | - public function testLicenseDetailsOnlyInsertedOncePerFile() |
|
| 405 | - { |
|
| 397 | + public function testLicenseDetailsOnlyInsertedOncePerFile() { |
|
| 406 | 398 | |
| 407 | 399 | $config = $this->createStub(StraussConfig::class); |
| 408 | 400 | $author = 'BrianHenryIE'; |
@@ -8,8 +8,7 @@ discard block |
||
| 8 | 8 | use Symfony\Component\Console\Input\InputInterface; |
| 9 | 9 | use Symfony\Component\Console\Output\OutputInterface; |
| 10 | 10 | |
| 11 | -class ComposeTest extends TestCase |
|
| 12 | -{ |
|
| 11 | +class ComposeTest extends TestCase { |
|
| 13 | 12 | |
| 14 | 13 | /** |
| 15 | 14 | * When composer.json is absent, instead of failing with: |
@@ -29,8 +28,7 @@ discard block |
||
| 29 | 28 | ->method('write'); |
| 30 | 29 | |
| 31 | 30 | new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
| 32 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 33 | - { |
|
| 31 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) { |
|
| 34 | 32 | parent::__construct(); |
| 35 | 33 | |
| 36 | 34 | $this->execute($inputInterfaceMock, $outputInterfaceMock); |
@@ -61,8 +59,7 @@ discard block |
||
| 61 | 59 | ->method('write'); |
| 62 | 60 | |
| 63 | 61 | new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
| 64 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 65 | - { |
|
| 62 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) { |
|
| 66 | 63 | parent::__construct(); |
| 67 | 64 | |
| 68 | 65 | $this->execute($inputInterfaceMock, $outputInterfaceMock); |
@@ -94,8 +91,7 @@ discard block |
||
| 94 | 91 | ->method('write'); |
| 95 | 92 | |
| 96 | 93 | new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
| 97 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 98 | - { |
|
| 94 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) { |
|
| 99 | 95 | parent::__construct(); |
| 100 | 96 | |
| 101 | 97 | $this->execute($inputInterfaceMock, $outputInterfaceMock); |
@@ -127,8 +123,7 @@ discard block |
||
| 127 | 123 | ->method('write'); |
| 128 | 124 | |
| 129 | 125 | new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
| 130 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 131 | - { |
|
| 126 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) { |
|
| 132 | 127 | parent::__construct(); |
| 133 | 128 | |
| 134 | 129 | $this->execute($inputInterfaceMock, $outputInterfaceMock); |
@@ -159,8 +154,7 @@ discard block |
||
| 159 | 154 | ->method('write'); |
| 160 | 155 | |
| 161 | 156 | new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
| 162 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 163 | - { |
|
| 157 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) { |
|
| 164 | 158 | parent::__construct(); |
| 165 | 159 | |
| 166 | 160 | $this->execute($inputInterfaceMock, $outputInterfaceMock); |
@@ -193,8 +187,7 @@ discard block |
||
| 193 | 187 | ->method('write'); |
| 194 | 188 | |
| 195 | 189 | new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
| 196 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 197 | - { |
|
| 190 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) { |
|
| 198 | 191 | parent::__construct(); |
| 199 | 192 | |
| 200 | 193 | $this->execute($inputInterfaceMock, $outputInterfaceMock); |
@@ -6,14 +6,12 @@ |
||
| 6 | 6 | use BrianHenryIE\Strauss\Console\Commands\Compose; |
| 7 | 7 | use PHPUnit\Framework\TestCase; |
| 8 | 8 | |
| 9 | -class ApplicationTest extends TestCase |
|
| 10 | -{ |
|
| 9 | +class ApplicationTest extends TestCase { |
|
| 11 | 10 | |
| 12 | 11 | /** |
| 13 | 12 | * Test the Symfony\Component\Console\Application instance contains the Compose command. |
| 14 | 13 | */ |
| 15 | - public function testInstantiation() |
|
| 16 | - { |
|
| 14 | + public function testInstantiation() { |
|
| 17 | 15 | |
| 18 | 16 | $version = '1.0.0'; |
| 19 | 17 | |
@@ -16,8 +16,7 @@ discard block |
||
| 16 | 16 | * @package BrianHenryIE\Strauss\Tests\Integration\Util |
| 17 | 17 | * @coversNothing |
| 18 | 18 | */ |
| 19 | -class IntegrationTestCase extends TestCase |
|
| 20 | -{ |
|
| 19 | +class IntegrationTestCase extends TestCase { |
|
| 21 | 20 | protected $testsWorkingDir; |
| 22 | 21 | |
| 23 | 22 | public function setUp(): void |
@@ -49,8 +48,7 @@ discard block |
||
| 49 | 48 | $this->deleteDir($dir); |
| 50 | 49 | } |
| 51 | 50 | |
| 52 | - protected function deleteDir($dir) |
|
| 53 | - { |
|
| 51 | + protected function deleteDir($dir) { |
|
| 54 | 52 | if (!file_exists($dir)) { |
| 55 | 53 | return; |
| 56 | 54 | } |