123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463 |
- <?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\ConfigurationException\InvalidFixerConfigurationException;
- use PhpCsFixer\Tests\Test\AbstractFixerTestCase;
- /**
- * @author Marc Aubé
- *
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\Whitespace\SpacesInsideParenthesesFixer
- *
- * @extends AbstractFixerTestCase<\PhpCsFixer\Fixer\Whitespace\SpacesInsideParenthesesFixer>
- *
- * @phpstan-import-type _AutogeneratedInputConfiguration from \PhpCsFixer\Fixer\Whitespace\SpacesInsideParenthesesFixer
- */
- final class SpacesInsideParenthesesFixerTest extends AbstractFixerTestCase
- {
- public function testInvalidConfigMissingKey(): void
- {
- $this->expectException(InvalidFixerConfigurationException::class);
- $this->expectExceptionMessageMatches('#^\[spaces_inside_parentheses\] Invalid configuration: The option "a" does not exist\. Defined options are: "space"\.$#');
- $this->fixer->configure(['a' => 1]);
- }
- public function testInvalidConfigValue(): void
- {
- $this->expectException(InvalidFixerConfigurationException::class);
- $this->expectExceptionMessageMatches('#^\[spaces_inside_parentheses\] Invalid configuration: The option "space" with value "double" is invalid\. Accepted values are: "none", "single"\.$#');
- $this->fixer->configure(['space' => 'double']); // @phpstan-ignore-line
- }
- /**
- * @dataProvider provideDefaultFixCases
- */
- public function testDefaultFix(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @dataProvider provideSpacesFixCases
- */
- public function testSpacesFix(string $expected, ?string $input = null): void
- {
- $this->fixer->configure(['space' => 'single']);
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{0: string, 1?: string}>
- */
- public static function provideDefaultFixCases(): iterable
- {
- // default leaves new lines alone
- yield [
- "<?php
- class Foo
- {
- private function bar()
- {
- if (foo(
- 'foo' ,
- 'bar' ,
- [1, 2, 3],
- 'baz' // a comment just to mix things up
- )) {
- return 1;
- };
- }
- }
- ",
- ];
- yield [
- '<?php foo();',
- '<?php foo( );',
- ];
- yield [
- '<?php
- if (true) {
- // if body
- }',
- '<?php
- if ( true ) {
- // if body
- }',
- ];
- yield [
- '<?php
- if (true) {
- // if body
- }',
- '<?php
- if ( true ) {
- // if body
- }',
- ];
- yield [
- '<?php
- function foo($bar, $baz)
- {
- // function body
- }',
- '<?php
- function foo( $bar, $baz )
- {
- // function body
- }',
- ];
- yield [
- '<?php
- $foo->bar($arg1, $arg2);',
- '<?php
- $foo->bar( $arg1, $arg2 );',
- ];
- yield [
- '<?php
- $var = array( 1, 2, 3 );
- ',
- ];
- yield [
- '<?php
- $var = [ 1, 2, 3 ];
- ',
- ];
- // list call with trailing comma - need to leave alone
- yield [
- '<?php list($path, $mode, ) = foo();',
- ];
- yield [
- '<?php list($path, $mode,) = foo();',
- ];
- yield [
- '<?php
- $a = $b->test( // do not remove space
- $e // between `(` and `)`
- // and this comment
- );',
- ];
- yield [
- '<?php
- function hello($value) {
- // code...
- }',
- '<?php
- function hello( $value ) {
- // code...
- }',
- ];
- yield [
- '<?php
- $code = function ($hello, $there) use ($ami, $tumi) {
- // code
- };
- ',
- '<?php
- $code = function ( $hello, $there ) use ( $ami, $tumi ) {
- // code
- };
- ',
- ];
- yield [
- '<?php
- for ($i = 0; $i < 42; $i++) {
- // code...
- }
- ',
- '<?php
- for ( $i = 0; $i < 42; $i++ ) {
- // code...
- }
- ',
- ];
- yield [
- '<?php
- explode($a, $b);
- ',
- '<?php
- explode( $a, $b );
- ',
- ];
- yield [
- '<?php
- if ($something) {
- // code
- }
- ',
- '<?php
- if ( $something ) {
- // code
- }
- ',
- ];
- yield [
- '<?php
- multiply((2 + 3) * 4);
- ',
- '<?php
- multiply( ( 2 + 3 ) * 4 );
- ',
- ];
- yield [
- '<?php $x = (new Foo())->bar();',
- '<?php $x = ( new Foo() )->bar();',
- ];
- yield [
- '<?php $x = (new Foo)->bar;',
- '<?php $x = ( new Foo )->bar;',
- ];
- }
- /**
- * @return iterable<array{0: string, 1?: string}>
- */
- public static function provideSpacesFixCases(): iterable
- {
- // Leaves new lines alone
- yield [
- "<?php
- class Foo
- {
- private function bar()
- {
- if ( foo(
- 'foo' ,
- 'bar' ,
- [1, 2, 3],
- 'baz' // a comment just to mix things up
- ) ) {
- return 1;
- };
- }
- }",
- ];
- yield [
- '<?php foo();',
- '<?php foo( );',
- ];
- yield [
- '<?php
- if ( true ) {
- // if body
- }',
- '<?php
- if (true) {
- // if body
- }',
- ];
- yield [
- '<?php
- if ( true ) {
- // if body
- }',
- '<?php
- if ( true ) {
- // if body
- }',
- ];
- yield [
- '<?php
- function foo( $bar, $baz )
- {
- // function body
- }',
- '<?php
- function foo($bar, $baz)
- {
- // function body
- }',
- ];
- yield [
- '<?php
- $foo->bar( $arg1, $arg2 );',
- '<?php
- $foo->bar( $arg1, $arg2 );',
- ];
- yield [
- '<?php
- $var = array( 1, 2, 3 );
- ',
- '<?php
- $var = array(1, 2, 3);
- ',
- ];
- yield [
- '<?php
- $var = [ 1, 2, 3 ];
- ',
- ];
- yield [
- '<?php list( $path, $mode, ) = foo();',
- '<?php list($path, $mode,) = foo();',
- ];
- yield [
- '<?php
- $a = $b->test( // do not remove space
- $e // between `(` and `)`
- // and this comment
- );',
- ];
- yield [
- '<?php
- function hello( $value ) {
- // code...
- }',
- '<?php
- function hello($value) {
- // code...
- }',
- ];
- yield [
- '<?php
- $code = function ( $hello, $there ) use ( $ami, $tumi ) {
- // code
- };
- ',
- '<?php
- $code = function ($hello, $there) use ($ami, $tumi) {
- // code
- };
- ',
- ];
- yield [
- '<?php
- for ( $i = 0; $i < 42; $i++ ) {
- // code...
- }
- ',
- '<?php
- for ($i = 0; $i < 42; $i++) {
- // code...
- }
- ',
- ];
- yield [
- '<?php
- explode( $a, $b );
- ',
- '<?php
- explode($a, $b);
- ',
- ];
- yield [
- '<?php
- if ( $something ) {
- // code
- }
- ',
- '<?php
- if ( $something ) {
- // code
- }
- ',
- ];
- yield [
- '<?php
- multiply( ( 2 + 3 ) * 4 );
- ',
- '<?php
- multiply((2 + 3) * 4);
- ',
- ];
- yield [
- '<?php $x = ( new Foo() )->bar();',
- '<?php $x = (new Foo())->bar();',
- ];
- }
- /**
- * @dataProvider provideDefaultFix80Cases
- *
- * @requires PHP 8.0
- */
- public function testDefaultFix80(string $expected, string $input): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<string, array{0: string, 1?: string}>
- */
- public static function provideDefaultFix80Cases(): iterable
- {
- yield 'mixed argument' => [
- '<?php function foo(mixed $a){}',
- '<?php function foo( mixed $a ){}',
- ];
- }
- /**
- * @dataProvider provideSpacesFix80Cases
- *
- * @requires PHP 8.0
- */
- public function testSpacesFix80(string $expected, string $input): void
- {
- $this->fixer->configure(['space' => 'single']);
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<string, array{0: string, 1?: string}>
- */
- public static function provideSpacesFix80Cases(): iterable
- {
- yield 'mixed argument' => [
- '<?php function foo( mixed $a ){}',
- '<?php function foo(mixed $a){}',
- ];
- }
- }
|