123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005 |
- <?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\Whitespace;
- use PhpCsFixer\Tests\Test\AbstractFixerTestCase;
- use PhpCsFixer\WhitespacesFixerConfig;
- /**
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\Whitespace\ArrayIndentationFixer
- *
- * @extends AbstractFixerTestCase<\PhpCsFixer\Fixer\Whitespace\ArrayIndentationFixer>
- */
- final class ArrayIndentationFixerTest extends AbstractFixerTestCase
- {
- /**
- * @dataProvider provideFixCases
- */
- public function testFix(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<int|string, array{0: string, 1?: string}>
- */
- public static function provideFixCases(): iterable
- {
- yield from self::withLongArraySyntaxCases([
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- 'foo',
- 'bar' => 'baz',
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'foo',
- 'bar' => 'baz',
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- 'foo',
- 'bar' => 'baz',
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'foo',
- 'bar' => 'baz',
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- ['bar', 'baz'],
- [
- 'bar',
- 'baz'
- ],
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- ['bar', 'baz'],
- [
- 'bar',
- 'baz'
- ],
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- ['foo',
- 'bar',
- ['foo',
- 'bar',
- ['foo',
- 'bar',
- 'baz']],
- 'baz'],
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- ['foo',
- 'bar',
- ['foo',
- 'bar',
- ['foo',
- 'bar',
- 'baz']],
- 'baz'],
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- class Foo
- {
- public $foo = [
- ['bar', 'baz'],
- [
- 'bar',
- 'baz'
- ],
- ];
- public function bar()
- {
- return [
- ['bar', 'baz'],
- [
- 'bar',
- 'baz'
- ],
- ];
- }
- }
- EXPECTED,
- <<<'INPUT'
- <?php
- class Foo
- {
- public $foo = [
- ['bar', 'baz'],
- [
- 'bar',
- 'baz'
- ],
- ];
- public function bar()
- {
- return [
- ['bar', 'baz'],
- [
- 'bar',
- 'baz'
- ],
- ];
- }
- }
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- 'foo' => foo(
- 1,
- 2
- ),
- 'bar' => bar(
- 1,
- 2
- ),
- 'baz' => baz(
- 1,
- 2
- ),
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'foo' => foo(
- 1,
- 2
- ),
- 'bar' => bar(
- 1,
- 2
- ),
- 'baz' => baz(
- 1,
- 2
- ),
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- 'foo' => ['bar' => [
- 'baz',
- ]],
- 'qux',
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'foo' => ['bar' => [
- 'baz',
- ]],
- 'qux',
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- 'foo' => [
- (new Foo())
- ->foo(),
- ],
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'foo' => [
- (new Foo())
- ->foo(),
- ],
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- [new Foo(
- )],
- [new Foo(
- )],
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- [new Foo(
- )],
- [new Foo(
- )],
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = new Foo([
- (new Bar())
- ->foo([
- 'foo',
- 'foo',
- ])
- ->bar(['bar', 'bar'])
- ->baz([
- 'baz',
- 'baz',
- ])
- ,
- ]);
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = new Foo([
- (new Bar())
- ->foo([
- 'foo',
- 'foo',
- ])
- ->bar(['bar', 'bar'])
- ->baz([
- 'baz',
- 'baz',
- ])
- ,
- ]);
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- class Foo
- {
- public function bar()
- {
- $foo = [
- 'foo',
- 'foo',
- ];
- $bar = [
- 'bar',
- 'bar',
- ];
- }
- }
- EXPECTED,
- <<<'INPUT'
- <?php
- class Foo
- {
- public function bar()
- {
- $foo = [
- 'foo',
- 'foo',
- ];
- $bar = [
- 'bar',
- 'bar',
- ];
- }
- }
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- class Foo
- {
- public function bar()
- {
- return new Bar([
- (new Baz())
- ->qux([function ($a) {
- foreach ($a as $b) {
- if ($b) {
- throw new Exception(sprintf(
- 'Oops: %s',
- $b
- ));
- }
- }
- }]),
- ]);
- }
- }
- EXPECTED,
- <<<'INPUT'
- <?php
- class Foo
- {
- public function bar()
- {
- return new Bar([
- (new Baz())
- ->qux([function ($a) {
- foreach ($a as $b) {
- if ($b) {
- throw new Exception(sprintf(
- 'Oops: %s',
- $b
- ));
- }
- }
- }]),
- ]);
- }
- }
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- 'Foo'.
- foo()
- .bar()
- ,
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'Foo'.
- foo()
- .bar()
- ,
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- [new \stdClass()],
- 'foo',
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- [new \stdClass()],
- 'foo',
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- $bar
- ? 'bar'
- : 'foo'
- ,
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- $bar
- ? 'bar'
- : 'foo'
- ,
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- $bar ?
- 'bar' :
- 'foo'
- ,
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- $bar ?
- 'bar' :
- 'foo'
- ,
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- [
- 'foo',
- ], [
- 'bar',
- ],
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- [
- 'foo',
- ], [
- 'bar',
- ],
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- 'foo', // comment
- 'bar',
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'foo', // comment
- 'bar',
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [[[
- 'foo',
- 'bar',
- ],
- ],
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [[[
- 'foo',
- 'bar',
- ],
- ],
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- [
- [
- 'foo',
- 'bar',
- ]]];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- [
- [
- 'foo',
- 'bar',
- ]]];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- [
- [[
- [[[
- 'foo',
- 'bar',
- ]
- ]]
- ]
- ]
- ]];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- [
- [[
- [[[
- 'foo',
- 'bar',
- ]
- ]]
- ]
- ]
- ]];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [[
- [
- [[[],[[
- [[[
- 'foo',
- 'bar',
- ],[[],[]]]
- ]],[
- 'baz',
- ]]
- ],[]],[]]
- ]
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [[
- [
- [[[],[[
- [[[
- 'foo',
- 'bar',
- ],[[],[]]]
- ]],[
- 'baz',
- ]]
- ],[]],[]]
- ]
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php if ($foo): ?>
- <?php foo([
- 'bar',
- 'baz',
- ]) ?>
- <?php endif ?>
- EXPECTED,
- <<<'INPUT'
- <?php if ($foo): ?>
- <?php foo([
- 'bar',
- 'baz',
- ]) ?>
- <?php endif ?>
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <div>
- <a
- class="link"
- href="<?= Url::to([
- '/site/page',
- 'id' => 123,
- ]); ?>"
- >
- Link text
- </a>
- </div>
- EXPECTED,
- <<<'INPUT'
- <div>
- <a
- class="link"
- href="<?= Url::to([
- '/site/page',
- 'id' => 123,
- ]); ?>"
- >
- Link text
- </a>
- </div>
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $arr = [
- 'a' => 'b',
- // 'c' => 'd',
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $arr = [
- 'a' => 'b',
- // 'c' => 'd',
- ];
- INPUT,
- ],
- [
- '<?php
- '.'
- $foo = [
- "foo",
- "bar",
- ];',
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- 'foo' =>
- 'Some'
- .' long'
- .' string',
- 'bar' =>
- 'Another'
- .' long'
- .' string'
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'foo' =>
- 'Some'
- .' long'
- .' string',
- 'bar' =>
- 'Another'
- .' long'
- .' string'
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- $test
- ? [
- 123,
- ]
- : [
- 321,
- ],
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- $test
- ? [
- 123,
- ]
- : [
- 321,
- ],
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [[
- new Foo(
- 'foo'
- ),
- ]];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [[
- new Foo(
- 'foo'
- ),
- ]];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $array = [
- 'foo' => [
- 'bar' => [
- 'baz',
- ],
- ], // <- this one
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $array = [
- 'foo' => [
- 'bar' => [
- 'baz',
- ],
- ], // <- this one
- ];
- INPUT,
- ],
- [
- <<<'EXPECTED'
- <?php
- $foo = [
- ...$foo,
- ...$bar,
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- ...$foo,
- ...$bar,
- ];
- INPUT,
- ],
- ]);
- yield 'array destructuring' => [
- <<<'EXPECTED'
- <?php
- [
- $foo,
- $bar,
- $baz
- ] = $arr;
- EXPECTED,
- <<<'INPUT'
- <?php
- [
- $foo,
- $bar,
- $baz
- ] = $arr;
- INPUT,
- ];
- yield 'array destructuring using list' => [
- <<<'EXPECTED'
- <?php
- list(
- $foo,
- $bar,
- $baz
- ) = $arr;
- EXPECTED,
- <<<'INPUT'
- <?php
- list(
- $foo,
- $bar,
- $baz
- ) = $arr;
- INPUT,
- ];
- }
- /**
- * @dataProvider provideWithWhitespacesConfigCases
- */
- public function testWithWhitespacesConfig(string $expected, ?string $input = null): void
- {
- $this->fixer->setWhitespacesConfig(new WhitespacesFixerConfig("\t"));
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<int|string, array{0: string, 1?: string}>
- */
- public static function provideWithWhitespacesConfigCases(): iterable
- {
- yield from self::withLongArraySyntaxCases([
- [
- <<<EXPECTED
- <?php
- \$foo = [
- \t'foo',
- \t'bar' => 'baz',
- ];
- EXPECTED,
- <<<'INPUT'
- <?php
- $foo = [
- 'foo',
- 'bar' => 'baz',
- ];
- INPUT,
- ],
- [
- <<<EXPECTED
- <?php
- \$foo = [
- \t'foo',
- \t'bar' => 'baz',
- ];
- EXPECTED,
- <<<INPUT
- <?php
- \$foo = [
- \t\t\t'foo',
- \t\t'bar' => 'baz',
- ];
- INPUT,
- ],
- ]);
- yield 'array destructuring' => [
- <<<EXPECTED
- <?php
- [
- \t\$foo,
- \t\$bar,
- \t\$baz
- ] = \$arr;
- EXPECTED,
- <<<'INPUT'
- <?php
- [
- $foo,
- $bar,
- $baz
- ] = $arr;
- INPUT,
- ];
- yield 'array destructuring using list' => [
- <<<EXPECTED
- <?php
- list(
- \t\$foo,
- \t\$bar,
- \t\$baz
- ) = \$arr;
- EXPECTED,
- <<<'INPUT'
- <?php
- list(
- $foo,
- $bar,
- $baz
- ) = $arr;
- INPUT,
- ];
- }
- /**
- * @dataProvider provideFix80Cases
- *
- * @requires PHP 8.0
- */
- public function testFix80(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<string, array{string, string}>
- */
- public static function provideFix80Cases(): iterable
- {
- yield 'attribute' => [
- '<?php
- class Foo {
- #[SimpleAttribute]
- #[ComplexAttribute(
- foo: true,
- bar: [
- 1,
- 2,
- 3,
- ]
- )]
- public function bar()
- {
- }
- }',
- '<?php
- class Foo {
- #[SimpleAttribute]
- #[ComplexAttribute(
- foo: true,
- bar: [
- 1,
- 2,
- 3,
- ]
- )]
- public function bar()
- {
- }
- }',
- ];
- }
- /**
- * @param list<array{0: string, 1?: string}> $cases
- *
- * @return list<array{0: string, 1?: string}>
- */
- private static function withLongArraySyntaxCases(array $cases): array
- {
- $longSyntaxCases = [];
- foreach ($cases as $case) {
- $case[0] = self::toLongArraySyntax($case[0]);
- if (isset($case[1])) {
- $case[1] = self::toLongArraySyntax($case[1]);
- }
- $longSyntaxCases[] = $case;
- }
- return [...$cases, ...$longSyntaxCases];
- }
- private static function toLongArraySyntax(string $php): string
- {
- return strtr($php, [
- '[' => 'array(',
- ']' => ')',
- ]);
- }
- }
|