123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428 |
- <?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\ControlStructure;
- use PhpCsFixer\Tests\Test\AbstractFixerTestCase;
- /**
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\ControlStructure\SwitchCaseSemicolonToColonFixer
- *
- * @extends AbstractFixerTestCase<\PhpCsFixer\Fixer\ControlStructure\SwitchCaseSemicolonToColonFixer>
- */
- final class SwitchCaseSemicolonToColonFixerTest extends AbstractFixerTestCase
- {
- /**
- * @dataProvider provideFixCases
- */
- public function testFix(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<int|string, array{string, string}>
- */
- public static function provideFixCases(): iterable
- {
- yield [
- '<?php
- switch (1) {
- case f(function () { return; }):
- break;
- }
- ',
- '<?php
- switch (1) {
- case f(function () { return; });
- break;
- }
- ',
- ];
- yield [
- '<?php
- switch ($a) {
- case 42:
- break;
- }
- ',
- '<?php
- switch ($a) {
- case 42;
- break;
- }
- ',
- ];
- yield [
- '<?php
- switch ($a) {
- case ["foo" => "bar"]:
- break;
- }
- ',
- '<?php
- switch ($a) {
- case ["foo" => "bar"];
- break;
- }
- ',
- ];
- yield [
- '<?php
- switch ($a) {
- case 42:
- break;
- case 1:
- switch ($a) {
- case 42:
- break;
- default :
- echo 1;
- }
- }',
- '<?php
- switch ($a) {
- case 42;
- break;
- case 1:
- switch ($a) {
- case 42;
- break;
- default ;
- echo 1;
- }
- }',
- ];
- yield [
- '<?php
- switch ($a) {
- case 42:;;// NoEmptyStatementFixer should clean this up (partly)
- break;
- }
- ',
- '<?php
- switch ($a) {
- case 42;;;// NoEmptyStatementFixer should clean this up (partly)
- break;
- }
- ',
- ];
- yield [
- '<?php
- switch ($a) {
- case $b ? "c" : "d" :
- break;
- }
- ',
- '<?php
- switch ($a) {
- case $b ? "c" : "d" ;
- break;
- }
- ',
- ];
- yield [
- '<?php
- switch ($a) {
- case $b ? "c" : "d": break;
- }
- ',
- '<?php
- switch ($a) {
- case $b ? "c" : "d"; break;
- }
- ',
- ];
- yield [
- '<?php
- switch($a) {
- case (int) $a < 1: {
- echo "leave ; alone";
- break;
- }
- case ($a < 2)/* test */ : {
- echo "fix 1";
- break;
- }
- case (3):{
- echo "fix 2";
- break;
- }
- case /**/(/**/ // test
- 4
- /**/)//
- /**/: {
- echo "fix 3";
- break;
- }
- case (((int)$b) + 4.1) : {
- echo "fix 4";
- break;
- }
- case ($b + 1) * 2 : {;;
- echo "leave alone";
- break;
- }
- }
- ',
- '<?php
- switch($a) {
- case (int) $a < 1; {
- echo "leave ; alone";
- break;
- }
- case ($a < 2)/* test */ ; {
- echo "fix 1";
- break;
- }
- case (3);{
- echo "fix 2";
- break;
- }
- case /**/(/**/ // test
- 4
- /**/)//
- /**/; {
- echo "fix 3";
- break;
- }
- case (((int)$b) + 4.1) ; {
- echo "fix 4";
- break;
- }
- case ($b + 1) * 2 ; {;;
- echo "leave alone";
- break;
- }
- }
- ',
- ];
- yield 'nested switch in switch case' => [
- '<?php
- switch (1) {
- case new class {public function A(){echo 1;switch(time()){case 1: echo 2;}}}:break;}
- ',
- '<?php
- switch (1) {
- case new class {public function A(){echo 1;switch(time()){case 1; echo 2;}}};break;}
- ',
- ];
- yield [
- '<?php
- switch (1) {
- case $b ? f(function () { return; }) : new class {public function A(){echo 1;}} :
- break;
- }
- ',
- '<?php
- switch (1) {
- case $b ? f(function () { return; }) : new class {public function A(){echo 1;}} ;
- break;
- }
- ',
- ];
- }
- /**
- * @dataProvider provideFixPre80Cases
- *
- * @requires PHP <8.0
- */
- public function testFixPre80(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string, string}>
- */
- public static function provideFixPre80Cases(): iterable
- {
- yield [
- '<?php
- switch ($a) {
- case $b ? "c" : "this" ? "is" : "ugly":
- break;
- }
- ',
- '<?php
- switch ($a) {
- case $b ? "c" : "this" ? "is" : "ugly";
- break;
- }
- ',
- ];
- }
- /**
- * @dataProvider provideFix80Cases
- *
- * @requires PHP 8
- */
- public function testFix80(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<string, array{0: string, 1?: string}>
- */
- public static function provideFix80Cases(): iterable
- {
- yield 'Simple match' => [
- '<?php
- echo match ($a) {
- default => "foo",
- };
- ',
- ];
- yield 'Match in switch' => [
- '<?php
- switch ($foo) {
- case "bar":
- echo match ($a) {
- default => "foo",
- };
- break;
- }
- ',
- ];
- yield 'Match in case value' => [
- '<?php
- switch ($foo) {
- case match ($bar) {
- default => "foo",
- }: echo "It works!";
- }
- ',
- '<?php
- switch ($foo) {
- case match ($bar) {
- default => "foo",
- }; echo "It works!";
- }
- ',
- ];
- }
- /**
- * @dataProvider provideFix81Cases
- *
- * @requires PHP 8.1
- */
- public function testFix81(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<string, array{string, string}>
- */
- public static function provideFix81Cases(): iterable
- {
- yield 'enums' => [
- '<?php
- enum Suit {
- case Hearts; // do not fix
- }
- enum UserStatus: string {
- case Pending = "P"; // do not fix
- public function label(): string {
- switch (foo()) {
- case 42: // do fix
- bar();
- $a = new class() {
- public function bar() {
- switch (foo()) {
- case 43: // do fix
- bar();
- }
- $expressionResult = match ($condition) {
- default => baz(),
- };
- }
- };
- $a->bar();
- break;
- }
- return "label";
- }
- }
- $expressionResult = match ($condition) {
- default => baz(),
- };
- ',
- '<?php
- enum Suit {
- case Hearts; // do not fix
- }
- enum UserStatus: string {
- case Pending = "P"; // do not fix
- public function label(): string {
- switch (foo()) {
- case 42; // do fix
- bar();
- $a = new class() {
- public function bar() {
- switch (foo()) {
- case 43; // do fix
- bar();
- }
- $expressionResult = match ($condition) {
- default => baz(),
- };
- }
- };
- $a->bar();
- break;
- }
- return "label";
- }
- }
- $expressionResult = match ($condition) {
- default => baz(),
- };
- ',
- ];
- }
- }
|