123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675 |
- <?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\Semicolon;
- use PhpCsFixer\Tests\Test\AbstractFixerTestCase;
- /**
- * @author Dariusz Rumiński <dariusz.ruminski@gmail.com>
- *
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\Semicolon\NoEmptyStatementFixer
- *
- * @extends AbstractFixerTestCase<\PhpCsFixer\Fixer\Semicolon\NoEmptyStatementFixer>
- */
- final class NoEmptyStatementFixerTest extends AbstractFixerTestCase
- {
- /**
- * @dataProvider provideNoEmptyStatementsCases
- */
- public function testNoEmptyStatements(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{0: string, 1?: string}>
- */
- public static function provideNoEmptyStatementsCases(): iterable
- {
- yield [
- '<?php
- abstract class TestClass0 extends Test IMPLEMENTS TestInterface, TestInterface2
- {
- }
- ',
- '<?php
- abstract class TestClass0 extends Test IMPLEMENTS TestInterface, TestInterface2
- {
- };
- ',
- ];
- yield [
- '<?php
- abstract class TestClass1 EXTENDS Test implements TestInterface
- {
- }
- ',
- '<?php
- abstract class TestClass1 EXTENDS Test implements TestInterface
- {
- };
- ',
- ];
- yield [
- '<?php
- CLASS TestClass2 extends Test
- {
- }
- ',
- '<?php
- CLASS TestClass2 extends Test
- {
- };
- ',
- ];
- yield [
- '<?php
- class TestClass3 implements TestInterface1
- {
- }
- ',
- '<?php
- class TestClass3 implements TestInterface1
- {
- };
- ',
- ];
- yield [
- '<?php
- class TestClass4
- {
- }
- ',
- '<?php
- class TestClass4
- {
- };
- ',
- ];
- yield [
- '<?php
- interface TestInterface1
- {
- }
- ',
- '<?php
- interface TestInterface1
- {
- };
- ',
- ];
- yield [
- '<?php
- interface TestExtendingInterface extends TestInterface2, TestInterface3 {
- }
- ',
- '<?php
- interface TestExtendingInterface extends TestInterface2, TestInterface3 {
- };
- ',
- ];
- yield [
- '<?php
- namespace Two {
- $a = 1; {
- }
- }
- ',
- '<?php
- namespace Two {;;
- $a = 1; {
- };
- }
- ',
- ];
- yield [
- '<?php
- {
- '.'
- }
- echo 1;
- ',
- '<?php
- {
- ;
- };
- echo 1;
- ',
- ];
- yield [
- '<?php
- while($time < $a)
- ;
- echo "done waiting.";
- $b = \Test;
- ',
- ];
- yield [
- '<?php
- if($a>1){
- }
- ',
- '<?php
- if($a>1){
- };
- ',
- ];
- yield [
- '<?php
- if($a>1) {
- } else {
- }
- ',
- '<?php
- if($a>1) {
- } else {
- };
- ',
- ];
- yield [
- '<?php
- try{
- }catch (\Exception $e) {
- }
- ',
- '<?php
- try{
- }catch (\Exception $e) {
- };
- ',
- ];
- yield [
- '<?php ',
- '<?php ;',
- ];
- yield [
- '<?php
- function foo()
- {
- // a
- }
- ',
- '<?php
- function foo()
- {
- ; // a
- }
- ',
- ];
- yield [
- '<?php function foo(){}',
- '<?php function foo(){;;}',
- ];
- yield [
- '<?php class Test{}',
- '<?php class Test{};',
- ];
- yield [
- '<?php
- for(;;) {
- }
- ',
- '<?php
- for(;;) {
- };
- ',
- ];
- yield [
- '<?php
- foreach($a as $b) {
- }
- foreach($a as $b => $c) {
- }
- ',
- '<?php
- foreach($a as $b) {
- };
- foreach($a as $b => $c) {
- };
- ',
- ];
- yield [
- '<?php
- while($a > 1){
- }
- do {
- } while($a>1); // 1
- ',
- '<?php
- while($a > 1){
- };
- do {
- } while($a>1); 1; // 1
- ',
- ];
- yield [
- '<?php
- switch($a) {
- default : {echo 1;}
- }
- ',
- '<?php
- switch($a) {
- default : {echo 1;}
- };
- ',
- ];
- yield [
- '<?php
- function test($a, $b) {
- }
- ',
- '<?php
- function test($a, $b) {
- };
- ',
- ];
- yield [
- '<?php
- function foo($n)
- {
- '.'
- $a = function(){};
- $b = function() use ($a) {};
- ++${"a"};
- switch(fooBar()) {
- case 5;{
- }
- }
- return $n->{$o};
- }
- ',
- '<?php
- function foo($n)
- {
- ;
- $a = function(){};
- $b = function() use ($a) {};
- ++${"a"};
- switch(fooBar()) {
- case 5;{
- }
- };
- return $n->{$o};
- };
- ',
- ];
- yield [
- '<?php
- declare(ticks=1) {
- // entire script here
- }
- declare(ticks=1);
- ',
- '<?php
- declare(ticks=1) {
- // entire script here
- };
- declare(ticks=1);
- ',
- ];
- yield [
- '<?php
- namespace A\B\C;
- use D;
- ',
- '<?php
- namespace A\B\C;;;;
- use D;;;;
- ',
- ];
- yield [
- '<?php
- namespace A\B\C;
- use D;
- ',
- '<?php
- namespace A\B\C;
- use D;;;;
- ',
- ];
- yield [
- '<?php
- namespace A\B\C;use D;
- ',
- '<?php
- namespace A\B\C;;use D;
- ',
- ];
- yield [
- '<?php
- trait TestTrait
- {
- }
- ',
- '<?php
- trait TestTrait
- {
- };
- ',
- ];
- yield [
- '<?php
- try {
- throw new \Exception("Foo.");
- } catch (\Exception $e){
- //
- } finally {
- } '.'
- ',
- '<?php
- try {
- throw new \Exception("Foo.");
- } catch (\Exception $e){
- //
- } finally {
- } ;
- ',
- ];
- foreach (['break', 'continue'] as $ops) {
- yield [
- \sprintf('<?php while(true) {%s ;}', $ops),
- \sprintf('<?php while(true) {%s 1;}', $ops),
- ];
- }
- foreach (['1', '1.0', '"foo"', '$foo'] as $noop) {
- yield [
- '<?php echo "foo"; ',
- \sprintf('<?php echo "foo"; %s ;', $noop),
- ];
- }
- yield [
- '<?php /* 1 */ /* 2 */ /* 3 */ ',
- '<?php /* 1 */ ; /* 2 */ 1 /* 3 */ ;',
- ];
- yield [
- '<?php
- while(true) {while(true) {break 2;}}
- while(true) {continue;}
- ',
- ];
- yield [
- '<?php if ($foo1) {} ',
- '<?php if ($foo1) {} 1;',
- ];
- yield [
- '<?php if ($foo2) {}',
- '<?php if ($foo2) {1;}',
- ];
- }
- /**
- * @dataProvider provideFixCases
- */
- public function testFix(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{0: string, 1?: string}>
- */
- public static function provideFixCases(): iterable
- {
- yield [
- '<?php
- use function Functional\map;
- $a = new class {
- public function log($msg)
- {
- }
- };
- ',
- ];
- yield [
- '<?php
- use function Functional\map;
- $a = new class extends A {
- };
- ',
- ];
- yield [
- '<?php
- use function Functional\map;
- $a = new class implements B {
- };
- ',
- ];
- yield [
- '<?php
- use function Functional\map;
- $a = new class extends A implements B {
- };
- ',
- ];
- yield [
- '<?php
- $a = new class extends \A implements B\C {
- };
- ',
- ];
- yield [
- '<?php {{}}',
- '<?php {{}};',
- ];
- yield [
- '<?php
- namespace A\B\C {
- }
- ',
- '<?php
- namespace A\B\C {
- };
- ',
- ];
- yield [
- '<?php
- namespace A{
- }
- ',
- '<?php
- namespace A{
- };
- ',
- ];
- yield [
- '<?php
- namespace{
- }
- ',
- '<?php
- namespace{
- };
- ',
- ];
- }
- /**
- * @dataProvider provideCasesWithShortOpenTagCases
- */
- public function testCasesWithShortOpenTag(string $expected, ?string $input = null): void
- {
- if (!\ini_get('short_open_tag')) {
- self::markTestSkipped('No short tag tests possible.');
- }
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string, string}>
- */
- public static function provideCasesWithShortOpenTagCases(): iterable
- {
- yield [
- '<? ',
- '<? ;',
- ];
- }
- /**
- * @dataProvider provideFixMultipleSemicolonsCases
- */
- public function testFixMultipleSemicolons(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{0: string, 1?: string}>
- */
- public static function provideFixMultipleSemicolonsCases(): iterable
- {
- yield [
- '<?php $foo = 2 ; //
- '.'
- ',
- '<?php $foo = 2 ; //
- ;
- ',
- ];
- yield [
- '<?php $foo = 3; /**/ ',
- '<?php $foo = 3; /**/; ;',
- ];
- yield [
- '<?php $foo = 1;',
- '<?php $foo = 1;;;',
- ];
- yield [
- '<?php $foo = 4; ',
- '<?php $foo = 4;; ;;',
- ];
- yield [
- '<?php $foo = 5;
- ',
- '<?php $foo = 5;;
- ;
- ;',
- ];
- yield [
- '<?php $foo = 6; ',
- '<?php $foo = 6;; ',
- ];
- yield [
- '<?php for ($i = 7; ; ++$i) {}',
- ];
- yield [
- '<?php
- switch($a){
- case 8;
- echo 9;
- }
- ',
- '<?php
- switch($a){
- case 8;;
- echo 9;
- }
- ',
- ];
- }
- /**
- * @dataProvider provideFix81Cases
- *
- * @requires PHP 8.1
- */
- public function testFix81(string $expected, string $input): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string, string}>
- */
- public static function provideFix81Cases(): iterable
- {
- yield [
- '<?php enum Foo{}',
- '<?php enum Foo{};',
- ];
- }
- }
|