123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184 |
- <?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;
- /**
- * @author Marc Aubé
- *
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\Whitespace\NoSpacesInsideParenthesisFixer
- *
- * @extends AbstractFixerTestCase<\PhpCsFixer\Fixer\Whitespace\NoSpacesInsideParenthesisFixer>
- */
- final class NoSpacesInsideParenthesisFixerTest extends AbstractFixerTestCase
- {
- /**
- * @dataProvider provideFixCases
- */
- public function testFix(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- public function testLeaveNewLinesAlone(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- private function bar()
- {
- if (foo(
- 'foo' ,
- 'bar' ,
- [1, 2, 3],
- 'baz' // a comment just to mix things up
- )) {
- return 1;
- };
- }
- }
- EOF;
- $this->doTest($expected);
- }
- /**
- * @return iterable<array{0: string, 1?: string}>
- */
- public static function provideFixCases(): iterable
- {
- 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
- );',
- ];
- }
- /**
- * @dataProvider provideFix80Cases
- *
- * @requires PHP 8.0
- */
- public function testFix80(string $expected, string $input): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string, string}>
- */
- public static function provideFix80Cases(): iterable
- {
- yield [
- '<?php function foo(mixed $a){}',
- '<?php function foo( mixed $a ){}',
- ];
- }
- /**
- * @dataProvider provideFix81Cases
- *
- * @requires PHP 8.1
- */
- public function testFix81(string $expected, string $input): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<string, array{string, string}>
- */
- public static function provideFix81Cases(): iterable
- {
- yield 'first callable class' => [
- '<?php $a = strlen(...);',
- '<?php $a = strlen( ... );',
- ];
- }
- }
|