123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534 |
- <?php
- declare(strict_types=1);
- /*
- * This file is part of PHP CS Fixer.
- *
- * (c) Fabien Potencier <fabien@symfony.com>
- * Dariusz Rumiński <dariusz.ruminski@gmail.com>
- *
- * This source file is subject to the MIT license that is bundled
- * with this source code in the file LICENSE.
- */
- namespace PhpCsFixer\Tests\Fixer\Basic;
- use PhpCsFixer\Tests\Test\AbstractFixerTestCase;
- /**
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\Basic\NoTrailingCommaInSinglelineFixer
- *
- * @extends AbstractFixerTestCase<\PhpCsFixer\Fixer\Basic\NoTrailingCommaInSinglelineFixer>
- *
- * @phpstan-import-type _AutogeneratedInputConfiguration from \PhpCsFixer\Fixer\Basic\NoTrailingCommaInSinglelineFixer
- */
- final class NoTrailingCommaInSinglelineFixerTest extends AbstractFixerTestCase
- {
- /**
- * @param _AutogeneratedInputConfiguration $configuration
- *
- * @dataProvider provideFixCases
- */
- public function testFix(string $expected, ?string $input = null, array $configuration = []): void
- {
- $this->fixer->configure($configuration);
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string, null|string, 2?: array{elements?: list<string>}}>
- */
- public static function provideFixCases(): iterable
- {
- yield [
- '<?php [$x, $y] = $a;',
- '<?php [$x, $y,] = $a;',
- ];
- yield 'group_import' => [
- '<?php use a\{ClassA, ClassB};',
- '<?php use a\{ClassA, ClassB,};',
- ];
- yield 'lots of nested' => [
- '<?php $a = [1,[1,[1,[1,[1,[1,[1,[1]],[1,[1,[1,[1,[1,[1,[1,[1]]]]]]]]]]]]]];',
- '<?php $a = [1,[1,[1,[1,[1,[1,[1,[1,],],[1,[1,[1,[1,[1,[1,[1,[1,],],],],],],],],],],],],],];',
- ];
- yield 'simple var' => [
- '<?php $a(1);',
- '<?php $a(1,);',
- ['elements' => ['arguments']],
- ];
- yield '&' => [
- '<?php $a = &foo($a);',
- '<?php $a = &foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield 'open' => [
- '<?php foo($a);',
- '<?php foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield '=' => [
- '<?php $b = foo($a);',
- '<?php $b = foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield '.' => [
- '<?php $c = $b . foo($a);',
- '<?php $c = $b . foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield '(' => [
- '<?php (foo($a/* 1X */ /* 2 */ ));',
- '<?php (foo($a /* 1X */ , /* 2 */ ));',
- ['elements' => ['arguments']],
- ];
- yield '\\' => [
- '<?php \foo($a);',
- '<?php \foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield 'A\\' => [
- '<?php A\foo($a);',
- '<?php A\foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield '\A\\' => [
- '<?php \A\foo($a);',
- '<?php \A\foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield ';' => [
- '<?php ; foo($a);',
- '<?php ; foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield '}' => [
- '<?php if ($a) { echo 1;} foo($a);',
- '<?php if ($a) { echo 1;} foo($a,);',
- ['elements' => ['arguments']],
- ];
- yield 'test method call' => [
- '<?php $o->abc($a);',
- '<?php $o->abc($a,);',
- ['elements' => ['arguments']],
- ];
- yield 'nested call' => [
- '<?php $o->abc($a,foo(1));',
- '<?php $o->abc($a,foo(1,));',
- ['elements' => ['arguments']],
- ];
- yield 'wrapped' => [
- '<?php echo (new Process())->getOutput(1);',
- '<?php echo (new Process())->getOutput(1,);',
- ['elements' => ['arguments']],
- ];
- yield 'dynamic function and method calls' => [
- '<?php $b->$a(1); $c("");',
- '<?php $b->$a(1,); $c("",);',
- ['elements' => ['arguments']],
- ];
- yield 'static function call' => [
- '<?php
- unset($foo->bar);
- $b = isset($foo->bar);
- ',
- '<?php
- unset($foo->bar,);
- $b = isset($foo->bar,);
- ',
- ['elements' => ['arguments']],
- ];
- yield 'unset' => [
- '<?php A::foo(1);',
- '<?php A::foo(1,);',
- ['elements' => ['arguments']],
- ];
- yield 'anonymous_class construction' => [
- '<?php new class(1, 2) {};',
- '<?php new class(1, 2,) {};',
- ['elements' => ['arguments']],
- ];
- yield 'array/property access call' => [
- '<?php
- $a = [
- "e" => static function(int $a): void{ echo $a;},
- "d" => [
- [2 => static function(int $a): void{ echo $a;}]
- ]
- ];
- $a["e"](1);
- $a["d"][0][2](1);
- $z = new class { public static function b(int $a): void {echo $a; }};
- $z::b(1);
- ${$e}(1);
- $$e(2);
- $f(0)(1);
- $g["e"](1); // foo',
- '<?php
- $a = [
- "e" => static function(int $a): void{ echo $a;},
- "d" => [
- [2 => static function(int $a): void{ echo $a;}]
- ]
- ];
- $a["e"](1,);
- $a["d"][0][2](1,);
- $z = new class { public static function b(int $a): void {echo $a; }};
- $z::b(1,);
- ${$e}(1,);
- $$e(2,);
- $f(0,)(1,);
- $g["e"](1,); // foo',
- ['elements' => ['arguments']],
- ];
- yield 'do not fix' => [
- '<?php
- function someFunction ($p1){}
- function & foo($a,$b): array { return []; }
- foo (
- 1,
- 2,
- );
- $a = new class (
- $a,
- ) {};
- isset($a, $b);
- unset($a,$b);
- list($a,$b) = $a;
- $a = [1,2,3,];
- $a = array(1,2,3,);
- function foo1(string $param = null ): void
- {
- }
- ;',
- null,
- ['elements' => ['arguments']],
- ];
- yield [
- '<?php $x = array();',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = array("foo");',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = array("foo");',
- '<?php $x = array("foo", );',
- ['elements' => ['array']],
- ];
- yield [
- "<?php \$x = array(\n'foo', \n);",
- null,
- ['elements' => ['array']],
- ];
- yield [
- "<?php \$x = array('foo', \n);",
- null,
- ['elements' => ['array']],
- ];
- yield [
- "<?php \$x = array(array('foo'), \n);",
- "<?php \$x = array(array('foo',), \n);",
- ['elements' => ['array']],
- ];
- yield [
- "<?php \$x = array(array('foo',\n), \n);",
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- $test = array("foo", <<<TWIG
- foo
- TWIG
- , $twig, );',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- $test = array(
- "foo", <<<TWIG
- foo
- TWIG
- , $twig, );',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- $test = array("foo", <<<\'TWIG\'
- foo
- TWIG
- , $twig, );',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- $test = array(
- "foo", <<<\'TWIG\'
- foo
- TWIG
- , $twig, );',
- null,
- ['elements' => ['array']],
- ];
- // Short syntax
- yield [
- '<?php $x = array([]);',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = [[]];',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = ["foo"];',
- '<?php $x = ["foo",];',
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = bar(["foo"]);',
- '<?php $x = bar(["foo",]);',
- ['elements' => ['array']],
- ];
- yield [
- "<?php \$x = bar([['foo'],\n]);",
- null,
- ['elements' => ['array']],
- ];
- yield [
- "<?php \$x = ['foo', \n];",
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = array([]);',
- '<?php $x = array([],);',
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = [[]];',
- '<?php $x = [[],];',
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = [$y[""]];',
- '<?php $x = [$y[""],];',
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- $test = ["foo", <<<TWIG
- foo
- TWIG
- , $twig, ];',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- $test = [
- "foo", <<<TWIG
- foo
- TWIG
- , $twig, ];',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- $test = ["foo", <<<\'TWIG\'
- foo
- TWIG
- , $twig, ];',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- $test = [
- "foo", <<<\'TWIG\'
- foo
- TWIG
- , $twig, ];',
- null,
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = array(...$foo);',
- '<?php $x = array(...$foo, );',
- ['elements' => ['array']],
- ];
- yield [
- '<?php $x = [...$foo];',
- '<?php $x = [...$foo, ];',
- ['elements' => ['array']],
- ];
- yield [
- '<?php
- list($a1, $b) = foo();
- list($a2, , $c, $d) = foo();
- list($a3, , $c) = foo();
- list($a4) = foo();
- list($a5 , $b) = foo();
- list($a6, /* $b */, $c) = foo();
- ',
- '<?php
- list($a1, $b) = foo();
- list($a2, , $c, $d, ) = foo();
- list($a3, , $c, , ) = foo();
- list($a4, , , , , ) = foo();
- list($a5 , $b , ) = foo();
- list($a6, /* $b */, $c, ) = foo();
- ',
- ['elements' => ['array_destructuring']],
- ];
- yield [
- '<?php
- list(
- $a#
- ,#
- #
- ) = $a;',
- null,
- ['elements' => ['array_destructuring']],
- ];
- yield [
- '<?php
- [$a7, $b] = foo();
- [$a8, , $c, $d] = foo();
- [$a9, , $c] = foo();
- [$a10] = foo();
- [$a11 , $b] = foo();
- [$a12, /* $b */, $c] = foo();
- ',
- '<?php
- [$a7, $b] = foo();
- [$a8, , $c, $d, ] = foo();
- [$a9, , $c, , ] = foo();
- [$a10, , , , , ] = foo();
- [$a11 , $b , ] = foo();
- [$a12, /* $b */, $c, ] = foo();
- ',
- ['elements' => ['array_destructuring']],
- ];
- }
- /**
- * @dataProvider provideFix80Cases
- *
- * @requires PHP 8.0
- */
- public function testFix80(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string}>
- */
- public static function provideFix80Cases(): iterable
- {
- yield [
- '<?php function foo(
- #[MyAttr(1, 2,)] Type $myParam,
- ) {}
- $foo1b = function() use ($bar, ) {};
- ',
- ];
- }
- /**
- * @dataProvider provideFix81Cases
- *
- * @requires PHP 8.1
- */
- public function testFix81(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string, string}>
- */
- public static function provideFix81Cases(): iterable
- {
- yield [
- '<?php $object?->method(1); strlen(...);',
- '<?php $object?->method(1,); strlen(...);',
- ];
- }
- }
|