123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303 |
- <?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\AbstractNoUselessElseFixer
- * @covers \PhpCsFixer\Fixer\ControlStructure\NoSuperfluousElseifFixer
- *
- * @extends AbstractFixerTestCase<\PhpCsFixer\Fixer\ControlStructure\NoSuperfluousElseifFixer>
- */
- final class NoSuperfluousElseifFixerTest extends AbstractFixerTestCase
- {
- /**
- * @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
- if ($some) { return 1; } if ($a == 6){ $test = false; } //',
- '<?php
- if ($some) { return 1; } elseif ($a == 6){ $test = false; } //',
- ];
- yield [
- '<?php
- if ($foo) {
- return 1;
- }
- if ($bar) {
- return 2;
- }
- if ($baz) {
- return 3;
- } else {
- return 4;
- }',
- '<?php
- if ($foo) {
- return 1;
- } elseif ($bar) {
- return 2;
- } else if ($baz) {
- return 3;
- } else {
- return 4;
- }',
- ];
- yield [
- '<?php
- if ($foo)
- return 1;
- if ($bar)
- echo \'bar\';
- else {
- return 3;
- }',
- '<?php
- if ($foo)
- return 1;
- elseif ($bar)
- echo \'bar\';
- else {
- return 3;
- }',
- ];
- yield [
- '<?php
- if ($foo)
- ?><?php
- elseif ($bar)
- ?><?php
- else {
- ?><?php
- }',
- ];
- yield [
- '<?php
- if ($foo) {
- ?><?php
- return;
- }
- if ($bar)
- ?><?php
- else {
- ?><?php
- }',
- '<?php
- if ($foo) {
- ?><?php
- return;
- } elseif ($bar)
- ?><?php
- else {
- ?><?php
- }',
- ];
- yield [
- '<?php
- while (1) {
- if (2) {
- if (3) {
- if (4) {
- die;
- }
- if (5) {
- exit;
- } else {#foo
- throw new \Exception();
- }
- '.'
- continue;
- }
- if (6) {
- return null;
- } else {
- return 1;
- }
- '.'
- break;
- }
- /* bar */if (7)
- return 2 + 3;
- else {# baz
- die(\'foo\');
- }
- }',
- '<?php
- while (1) {
- if (2) {
- if (3) {
- if (4) {
- die;
- } elseif (5) {
- exit;
- } else {#foo
- throw new \Exception();
- }
- '.'
- continue;
- } else if (6) {
- return null;
- } else {
- return 1;
- }
- '.'
- break;
- } else/* bar */if (7)
- return 2 + 3;
- else {# baz
- die(\'foo\');
- }
- }',
- ];
- yield [
- '<?php
- if ($a === false)
- {
- if ($v) { $ret = "foo"; }
- elseif($a)
- die;
- }
- elseif($a)
- $ret .= $value;
- return $ret;',
- ];
- yield [
- '<?php
- if ($a)
- echo 1;
- else if ($b)
- die;
- else {
- echo 2;
- }',
- ];
- yield [
- '<?php
- if ($a) {
- echo 1;
- } else if ($b)
- die;
- else {
- echo 2;
- }',
- ];
- yield [
- '<?php
- if ($a) {
- echo 1;
- } else if ($b) {
- die;
- } else {
- echo 2;
- }',
- ];
- yield [
- '<?php
- if ($foo) {
- return 1;
- }
- if ($bar) {
- return 2;
- }
- if ($baz) {
- throw new class extends Exception{};
- } else {
- return 4;
- }',
- '<?php
- if ($foo) {
- return 1;
- } elseif ($bar) {
- return 2;
- } else if ($baz) {
- throw new class extends Exception{};
- } else {
- return 4;
- }',
- ];
- }
- /**
- * @dataProvider provideFix80Cases
- *
- * @requires PHP 8.0
- */
- public function testFix80(string $expected): void
- {
- $this->doTest($expected);
- }
- /**
- * @return iterable<array{string}>
- */
- public static function provideFix80Cases(): iterable
- {
- yield [
- '<?php
- if ($foo) {
- $a = $bar ?? throw new \Exception();
- } elseif ($bar) {
- echo 1;
- }
- ',
- ];
- }
- }
|