1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426142714281429143014311432143314341435143614371438143914401441144214431444144514461447144814491450145114521453145414551456145714581459146014611462146314641465146614671468146914701471147214731474147514761477147814791480148114821483148414851486148714881489149014911492149314941495149614971498149915001501150215031504150515061507150815091510151115121513151415151516151715181519152015211522152315241525152615271528152915301531153215331534153515361537153815391540154115421543154415451546154715481549155015511552155315541555155615571558155915601561156215631564156515661567156815691570157115721573157415751576157715781579158015811582158315841585158615871588158915901591159215931594159515961597159815991600160116021603160416051606160716081609161016111612161316141615161616171618161916201621162216231624162516261627162816291630163116321633163416351636163716381639164016411642164316441645164616471648164916501651165216531654165516561657165816591660166116621663166416651666166716681669167016711672167316741675167616771678167916801681 |
- <?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\ClassNotation;
- use PhpCsFixer\ConfigurationException\InvalidFixerConfigurationException;
- use PhpCsFixer\Fixer\ClassNotation\OrderedClassElementsFixer;
- use PhpCsFixer\Tests\Test\AbstractFixerTestCase;
- /**
- * @author Gregor Harlan <gharlan@web.de>
- *
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\ClassNotation\OrderedClassElementsFixer
- */
- final class OrderedClassElementsFixerTest extends AbstractFixerTestCase
- {
- /**
- * @dataProvider provideFixCases
- */
- public function testFix(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- public static function provideFixCases(): iterable
- {
- yield [
- <<<'EOT'
- <?php
- class Foo {}
- class Bar
- {
- }
- EOT
- ];
- yield [
- <<<'EOT'
- <?php
- class Foo { const C1 = 1; protected $abc = 'abc'; public function baz($y, $z) {} private function bar1($x) { return 1; } }
- EOT
- , <<<'EOT'
- <?php
- class Foo { private function bar1($x) { return 1; } protected $abc = 'abc'; const C1 = 1; public function baz($y, $z) {} }
- EOT
- ];
- yield [
- <<<'EOT'
- <?php
- interface FooInterface
- {
- /** const 1 */
- const CONST1 = 'const1';
- // foo
- // const 2
- const CONST2 = 'const2';
- public function xyz($x, $y, $z); // comment
- /**
- * @param array $a
- *
- * @return string
- */
- function abc(array &$a = null);
- public function def();
- }
- EOT
- , <<<'EOT'
- <?php
- interface FooInterface
- {
- public function xyz($x, $y, $z); // comment
- /** const 1 */
- const CONST1 = 'const1';
- /**
- * @param array $a
- *
- * @return string
- */
- function abc(array &$a = null);
- // foo
- // const 2
- const CONST2 = 'const2';
- public function def();
- }
- EOT
- ];
- yield [
- <<<'EOT'
- <?php
- abstract class Foo extends FooParent implements FooInterface1, FooInterface2
- {
- use Bar;
- use Baz {
- abc as private;
- }
- const C1 = 1;
- /* comment for C2 */
- const C2 = 2;
- public $fooPublic;
- // comment 3
- protected $fooProtected = array(1, 2);
- // comment 1
- private $fooPrivate;
- protected function __construct()
- {
- }
- public function __destruct() {}
- public function __clone() {}
- public static function setUpBeforeClass() {}
- public static function tearDownAfterclass() {
- } /* multiline
- comment */
- protected function setUp() {}
- protected function assertPreConditions() {}
- protected function assertPostConditions() {}
- protected function tearDown() {}
- abstract public function foo1($a, $b = 1);
- // foo2
- function foo2()
- {
- return $this->foo1(1);
- }
- public static function foo3(self $foo)
- {
- return $foo->foo2();
- } /* comment 1 */ /* comment 2 */
- // comment
- /**
- * Docblock
- */
- protected function foo4(\ArrayObject $object, array $array, $a = null)
- {
- bar();
- if (!$a) {
- $a = function ($x) {
- var_dump($x);
- };
- }
- }
- private function foo5()
- {
- } // end foo5
- }
- EOT
- , <<<'EOT'
- <?php
- abstract class Foo extends FooParent implements FooInterface1, FooInterface2
- {
- // comment 1
- private $fooPrivate;
- abstract public function foo1($a, $b = 1);
- protected function setUp() {}
- protected function tearDown() {}
- public function __clone() {}
- const C1 = 1;
- // foo2
- function foo2()
- {
- return $this->foo1(1);
- }
- public static function setUpBeforeClass() {}
- public function __destruct() {}
- use Bar;
- public static function foo3(self $foo)
- {
- return $foo->foo2();
- } /* comment 1 */ /* comment 2 */
- // comment 3
- protected $fooProtected = array(1, 2);
- public $fooPublic;
- /* comment for C2 */
- const C2 = 2;
- public static function tearDownAfterclass() {
- } /* multiline
- comment */
- protected function assertPostConditions() {}
- use Baz {
- abc as private;
- }
- protected function assertPreConditions() {}
- private function foo5()
- {
- } // end foo5
- protected function __construct()
- {
- }
- // comment
- /**
- * Docblock
- */
- protected function foo4(\ArrayObject $object, array $array, $a = null)
- {
- bar();
- if (!$a) {
- $a = function ($x) {
- var_dump($x);
- };
- }
- }
- }
- EOT
- ];
- yield [
- <<<'EOT'
- <?php
- class Foo
- {
- const C = 'C';
- public function abc() {}
- protected function xyz() {}
- }
- class Bar
- {
- const C = 1;
- public function foo($a) { return 1; }
- public function baz() {}
- }
- EOT
- , <<<'EOT'
- <?php
- class Foo
- {
- protected function xyz() {}
- const C = 'C';
- public function abc() {}
- }
- class Bar
- {
- public function foo($a) { return 1; }
- const C = 1;
- public function baz() {}
- }
- EOT
- ];
- yield [
- <<<'EOT'
- <?php
- trait FooTrait
- {
- use BarTrait;
- use BazTrait;
- protected function abc() {
- }
- }
- EOT
- , <<<'EOT'
- <?php
- trait FooTrait
- {
- protected function abc() {
- }
- use BarTrait;
- use BazTrait;
- }
- EOT
- ];
- yield [
- <<<'EOT'
- <?php
- /**
- * @see #4086
- */
- class ComplexStringVariableAndUseTrait
- {
- use FooTrait;
- public function sayHello($name)
- {
- echo "Hello, {$name}!";
- }
- }
- EOT
- ,
- <<<'EOT'
- <?php
- /**
- * @see #4086
- */
- class ComplexStringVariableAndUseTrait
- {
- public function sayHello($name)
- {
- echo "Hello, {$name}!";
- }
- use FooTrait;
- }
- EOT
- ];
- }
- /**
- * @param array<string, mixed> $configuration
- *
- * @dataProvider provideFixWithConfigurationCases
- */
- public function testFixWithConfiguration(array $configuration, string $expected, string $input): void
- {
- $this->fixer->configure(['order' => $configuration]);
- $this->doTest($expected, $input);
- }
- public static function provideFixWithConfigurationCases(): iterable
- {
- yield [
- ['use_trait', 'constant', 'property', 'construct', 'method', 'destruct'],
- <<<'EOT'
- <?php
- class Foo
- {
- use BarTrait;
- use BazTrait;
- const C1 = 1;
- const C2 = 2;
- protected static $protStatProp;
- public static $pubStatProp1;
- public $pubProp1;
- protected $protProp;
- var $pubProp2;
- private static $privStatProp;
- private $privProp;
- public static $pubStatProp2;
- public $pubProp3;
- protected function __construct() {}
- private static function privStatFunc() {}
- public function pubFunc1() {}
- public function __toString() {}
- protected function protFunc() {}
- function pubFunc2() {}
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- static function pubStatFunc2() {}
- private function privFunc() {}
- public static function pubStatFunc3() {}
- protected static function protStatFunc() {}
- public function __destruct() {}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- class Foo
- {
- private static function privStatFunc() {}
- protected static $protStatProp;
- public static $pubStatProp1;
- public function pubFunc1() {}
- use BarTrait;
- public $pubProp1;
- public function __toString() {}
- protected function protFunc() {}
- protected $protProp;
- function pubFunc2() {}
- public function __destruct() {}
- var $pubProp2;
- private static $privStatProp;
- use BazTrait;
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- private $privProp;
- const C1 = 1;
- static function pubStatFunc2() {}
- private function privFunc() {}
- public static $pubStatProp2;
- protected function __construct() {}
- const C2 = 2;
- public static function pubStatFunc3() {}
- public $pubProp3;
- protected static function protStatFunc() {}
- }
- EOT
- ,
- ];
- yield [
- ['public', 'protected', 'private'],
- <<<'EOT'
- <?php
- class Foo
- {
- public static $pubStatProp1;
- public function pubFunc1() {}
- public $pubProp1;
- public function __toString() {}
- function pubFunc2() {}
- public function __destruct() {}
- var $pubProp2;
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- const C1 = 1;
- static function pubStatFunc2() {}
- public static $pubStatProp2;
- const C2 = 2;
- public static function pubStatFunc3() {}
- public $pubProp3;
- protected static $protStatProp;
- protected function protFunc() {}
- protected $protProp;
- protected function __construct() {}
- protected static function protStatFunc() {}
- private static function privStatFunc() {}
- private static $privStatProp;
- private $privProp;
- private function privFunc() {}
- use BarTrait;
- use BazTrait;
- }
- EOT
- ,
- <<<'EOT'
- <?php
- class Foo
- {
- private static function privStatFunc() {}
- protected static $protStatProp;
- public static $pubStatProp1;
- public function pubFunc1() {}
- use BarTrait;
- public $pubProp1;
- public function __toString() {}
- protected function protFunc() {}
- protected $protProp;
- function pubFunc2() {}
- public function __destruct() {}
- var $pubProp2;
- private static $privStatProp;
- use BazTrait;
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- private $privProp;
- const C1 = 1;
- static function pubStatFunc2() {}
- private function privFunc() {}
- public static $pubStatProp2;
- protected function __construct() {}
- const C2 = 2;
- public static function pubStatFunc3() {}
- public $pubProp3;
- protected static function protStatFunc() {}
- }
- EOT
- ,
- ];
- yield [
- [
- 'use_trait',
- 'constant',
- 'property_public_static',
- 'property_protected_static',
- 'property_private_static',
- 'property_public',
- 'property_protected',
- 'property_private',
- 'construct',
- 'destruct',
- 'magic',
- 'method_public_static',
- 'method_protected_static',
- 'method_private_static',
- 'method_public',
- 'method_protected',
- 'method_private',
- ],
- <<<'EOT'
- <?php
- class Foo
- {
- use BarTrait;
- use BazTrait;
- const C1 = 1;
- const C2 = 2;
- public static $pubStatProp1;
- public static $pubStatProp2;
- protected static $protStatProp;
- private static $privStatProp;
- public $pubProp1;
- var $pubProp2;
- public $pubProp3;
- protected $protProp;
- private $privProp;
- protected function __construct() {}
- public function __destruct() {}
- public function __toString() {}
- public static function pubStatFunc1() {}
- static function pubStatFunc2() {}
- public static function pubStatFunc3() {}
- protected static function protStatFunc() {}
- private static function privStatFunc() {}
- public function pubFunc1() {}
- function pubFunc2() {}
- public function pubFunc3() {}
- protected function protFunc() {}
- private function privFunc() {}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- class Foo
- {
- private static function privStatFunc() {}
- protected static $protStatProp;
- public static $pubStatProp1;
- public function pubFunc1() {}
- use BarTrait;
- public $pubProp1;
- public function __toString() {}
- protected function protFunc() {}
- protected $protProp;
- function pubFunc2() {}
- public function __destruct() {}
- var $pubProp2;
- private static $privStatProp;
- use BazTrait;
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- private $privProp;
- const C1 = 1;
- static function pubStatFunc2() {}
- private function privFunc() {}
- public static $pubStatProp2;
- protected function __construct() {}
- const C2 = 2;
- public static function pubStatFunc3() {}
- public $pubProp3;
- protected static function protStatFunc() {}
- }
- EOT
- ,
- ];
- yield [
- ['use_trait', 'constant', 'property', 'construct', 'method', 'destruct'],
- <<<'EOT'
- <?php
- abstract class Foo
- {
- use BarTrait;
- use BazTrait;
- const C1 = 1;
- const C2 = 2;
- protected static $protStatProp;
- public static $pubStatProp1;
- public $pubProp1;
- protected $protProp;
- var $pubProp2;
- private static $privStatProp;
- private $privProp;
- public static $pubStatProp2;
- public $pubProp3;
- protected function __construct() {}
- abstract public function absPubFunc();
- private static function privStatFunc() {}
- public function pubFunc1() {}
- abstract protected function absProtFunc();
- public function __toString() {}
- protected function protFunc() {}
- function pubFunc2() {}
- abstract protected static function absProtStatFunc();
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- static function pubStatFunc2() {}
- private function privFunc() {}
- abstract public static function absPubStatFunc();
- public static function pubStatFunc3() {}
- protected static function protStatFunc() {}
- public function __destruct() {}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- abstract class Foo
- {
- abstract public function absPubFunc();
- private static function privStatFunc() {}
- protected static $protStatProp;
- public static $pubStatProp1;
- public function pubFunc1() {}
- abstract protected function absProtFunc();
- use BarTrait;
- public $pubProp1;
- public function __toString() {}
- protected function protFunc() {}
- protected $protProp;
- function pubFunc2() {}
- abstract protected static function absProtStatFunc();
- public function __destruct() {}
- var $pubProp2;
- private static $privStatProp;
- use BazTrait;
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- private $privProp;
- const C1 = 1;
- static function pubStatFunc2() {}
- private function privFunc() {}
- public static $pubStatProp2;
- abstract public static function absPubStatFunc();
- protected function __construct() {}
- const C2 = 2;
- public static function pubStatFunc3() {}
- public $pubProp3;
- protected static function protStatFunc() {}
- }
- EOT
- ,
- ];
- yield [
- ['public', 'protected', 'private'],
- <<<'EOT'
- <?php
- abstract class Foo
- {
- abstract public function absPubFunc();
- public static $pubStatProp1;
- public function pubFunc1() {}
- public $pubProp1;
- public function __toString() {}
- function pubFunc2() {}
- public function __destruct() {}
- var $pubProp2;
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- const C1 = 1;
- static function pubStatFunc2() {}
- public static $pubStatProp2;
- abstract public static function absPubStatFunc();
- const C2 = 2;
- public static function pubStatFunc3() {}
- public $pubProp3;
- protected static $protStatProp;
- abstract protected function absProtFunc();
- protected function protFunc() {}
- protected $protProp;
- abstract protected static function absProtStatFunc();
- protected function __construct() {}
- protected static function protStatFunc() {}
- private static function privStatFunc() {}
- private static $privStatProp;
- private $privProp;
- private function privFunc() {}
- use BarTrait;
- use BazTrait;
- }
- EOT
- ,
- <<<'EOT'
- <?php
- abstract class Foo
- {
- abstract public function absPubFunc();
- private static function privStatFunc() {}
- protected static $protStatProp;
- public static $pubStatProp1;
- public function pubFunc1() {}
- abstract protected function absProtFunc();
- use BarTrait;
- public $pubProp1;
- public function __toString() {}
- protected function protFunc() {}
- protected $protProp;
- function pubFunc2() {}
- abstract protected static function absProtStatFunc();
- public function __destruct() {}
- var $pubProp2;
- private static $privStatProp;
- use BazTrait;
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- private $privProp;
- const C1 = 1;
- static function pubStatFunc2() {}
- private function privFunc() {}
- public static $pubStatProp2;
- abstract public static function absPubStatFunc();
- protected function __construct() {}
- const C2 = 2;
- public static function pubStatFunc3() {}
- public $pubProp3;
- protected static function protStatFunc() {}
- }
- EOT
- ,
- ];
- yield [
- [
- 'use_trait',
- 'constant',
- 'property_public_static',
- 'property_protected_static',
- 'property_private_static',
- 'property_public',
- 'property_protected',
- 'property_private',
- 'construct',
- 'destruct',
- 'magic',
- 'method_public_static',
- 'method_public_abstract_static',
- 'method_protected_static',
- 'method_protected_abstract_static',
- 'method_private_static',
- 'method_public',
- 'method_public_abstract',
- 'method_protected',
- 'method_protected_abstract',
- 'method_private',
- ],
- <<<'EOT'
- <?php
- abstract class Foo
- {
- use BarTrait;
- use BazTrait;
- const C1 = 1;
- const C2 = 2;
- public static $pubStatProp1;
- public static $pubStatProp2;
- protected static $protStatProp;
- private static $privStatProp;
- public $pubProp1;
- var $pubProp2;
- public $pubProp3;
- protected $protProp;
- private $privProp;
- protected function __construct() {}
- public function __destruct() {}
- public function __toString() {}
- public static function pubStatFunc1() {}
- static function pubStatFunc2() {}
- public static function pubStatFunc3() {}
- abstract public static function absPubStatFunc();
- protected static function protStatFunc() {}
- abstract protected static function absProtStatFunc();
- private static function privStatFunc() {}
- public function pubFunc1() {}
- function pubFunc2() {}
- public function pubFunc3() {}
- abstract public function absPubFunc();
- protected function protFunc() {}
- abstract protected function absProtFunc();
- private function privFunc() {}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- abstract class Foo
- {
- abstract public function absPubFunc();
- private static function privStatFunc() {}
- protected static $protStatProp;
- public static $pubStatProp1;
- public function pubFunc1() {}
- abstract protected function absProtFunc();
- use BarTrait;
- public $pubProp1;
- public function __toString() {}
- protected function protFunc() {}
- protected $protProp;
- function pubFunc2() {}
- abstract protected static function absProtStatFunc();
- public function __destruct() {}
- var $pubProp2;
- private static $privStatProp;
- use BazTrait;
- public static function pubStatFunc1() {}
- public function pubFunc3() {}
- private $privProp;
- const C1 = 1;
- static function pubStatFunc2() {}
- private function privFunc() {}
- public static $pubStatProp2;
- abstract public static function absPubStatFunc();
- protected function __construct() {}
- const C2 = 2;
- public static function pubStatFunc3() {}
- public $pubProp3;
- protected static function protStatFunc() {}
- }
- EOT
- ,
- ];
- yield [
- [
- 'method_public',
- 'method_abstract',
- ],
- <<<'EOT'
- <?php
- abstract class Foo
- {
- public function test2(){}
- public abstract function test1();
- }
- EOT
- ,
- <<<'EOT'
- <?php
- abstract class Foo
- {
- public abstract function test1();
- public function test2(){}
- }
- EOT
- ,
- ];
- yield [
- [
- 'construct',
- 'method:__invoke',
- 'destruct',
- 'method_public_static',
- 'method:custom1',
- 'method_public_abstract_static',
- 'method_protected_static',
- 'method_protected_abstract_static',
- 'method_private_static',
- 'method_public',
- 'method:custom3',
- 'method_public_abstract',
- 'method_protected',
- 'method_protected_abstract',
- 'method_private',
- 'method:custom2',
- 'magic',
- ],
- <<<'EOT'
- <?php
- abstract class Foo
- {
- protected function __construct() {}
- public function __invoke() {}
- public function __destruct() {}
- public static function pubStatFunc1() {}
- static function pubStatFunc2() {}
- public static function pubStatFunc3() {}
- private function custom1() {}
- abstract public static function absPubStatFunc();
- protected static function protStatFunc() {}
- abstract protected static function absProtStatFunc();
- private static function privStatFunc() {}
- public function pubFunc1() {}
- function pubFunc2() {}
- public function pubFunc3() {}
- abstract public function custom3();
- abstract public function absPubFunc();
- protected function protFunc() {}
- abstract protected function absProtFunc();
- private function privFunc() {}
- protected static function custom2() {}
- public function __get($prop) {}
- public function __toString() {}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- abstract class Foo
- {
- public function __get($prop) {}
- private function custom1() {}
- abstract public function absPubFunc();
- private static function privStatFunc() {}
- public function pubFunc1() {}
- abstract protected function absProtFunc();
- public function __toString() {}
- protected function protFunc() {}
- function pubFunc2() {}
- abstract protected static function absProtStatFunc();
- public function __destruct() {}
- public static function pubStatFunc1() {}
- public function __invoke() {}
- public function pubFunc3() {}
- static function pubStatFunc2() {}
- private function privFunc() {}
- abstract public static function absPubStatFunc();
- protected function __construct() {}
- public static function pubStatFunc3() {}
- protected static function protStatFunc() {}
- abstract public function custom3();
- protected static function custom2() {}
- }
- EOT
- ,
- ];
- yield [
- [
- 'method:foo',
- 'method_public',
- 'method:bar',
- 'method:baz',
- 'magic',
- ],
- <<<'EOT'
- <?php
- abstract class Foo
- {
- public function pub() {}
- public function bar() {}
- public function __toString() {}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- abstract class Foo
- {
- public function __toString() {}
- public function pub() {}
- public function bar() {}
- }
- EOT
- ,
- ];
- }
- /**
- * @param array<string, mixed> $configuration
- *
- * @dataProvider provideFixWithSortingAlgorithmCases
- */
- public function testFixWithSortingAlgorithm(array $configuration, string $expected, string $input): void
- {
- $this->fixer->configure($configuration);
- $this->doTest($expected, $input);
- }
- public static function provideFixWithSortingAlgorithmCases(): iterable
- {
- yield [
- [
- 'order' => [
- 'property_public_static',
- 'method_public',
- 'method_private',
- ],
- 'sort_algorithm' => OrderedClassElementsFixer::SORT_ALPHA,
- ],
- <<<'EOT'
- <?php
- class Example
- {
- public static $pubStatProp1;
- public static $pubStatProp2;
- public function A(){}
- public function B1(){}
- public function B2(){}
- public function C(){}
- public function C1(){}
- public function D(){}
- private function E(){}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- class Example
- {
- public function D(){}
- public static $pubStatProp2;
- public function B1(){}
- public function B2(){}
- private function E(){}
- public static $pubStatProp1;
- public function A(){}
- public function C(){}
- public function C1(){}
- }
- EOT
- ,
- ];
- yield [
- [
- 'order' => [
- 'use_trait',
- 'constant',
- 'property_public_static',
- 'property_protected_static',
- 'property_private_static',
- 'property_public',
- 'property_protected',
- 'property_private',
- 'construct',
- 'destruct',
- 'magic',
- 'method_public_static',
- 'method_protected_static',
- 'method_private_static',
- 'method_public',
- 'method_protected',
- 'method_private',
- ],
- 'sort_algorithm' => OrderedClassElementsFixer::SORT_ALPHA,
- ],
- <<<'EOT'
- <?php
- class Foo
- {
- use BarTrait;
- use BazTrait;
- const C1 = 1;
- const C2 = 2;
- public static $pubStatProp1;
- public static $pubStatProp2;
- protected static $protStatProp;
- private static $privStatProp;
- public $pubProp1;
- var $pubProp2;
- public $pubProp3;
- protected $protProp;
- private $privProp;
- protected function __construct() {}
- public function __destruct() {}
- public function __magicA() {}
- public function __magicB() {}
- public function __toString() {}
- public static function pubStatFunc1() {}
- static function pubStatFunc2() {}
- public static function pubStatFunc3() {
- return $this->privFunc();
- }
- protected static function protStatFunc() {}
- private static function privStatFunc() {}
- public function pubFunc1() {}
- function pubFunc2() {}
- public function pubFunc3(int $b, int $c) {
- $a = $b*$c;
- return $a % 4;
- }
- protected function protFunc() {}
- private function privFunc() {}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- class Foo
- {
- private static function privStatFunc() {}
- protected static $protStatProp;
- use BazTrait;
- public static $pubStatProp2;
- public $pubProp3;
- use BarTrait;
- public function __toString() {}
- protected function protFunc() {}
- protected $protProp;
- function pubFunc2() {}
- public $pubProp1;
- public function __destruct() {}
- var $pubProp2;
- public function __magicB() {}
- const C2 = 2;
- public static $pubStatProp1;
- public function __magicA() {}
- private static $privStatProp;
- static function pubStatFunc2() {}
- public function pubFunc3(int $b, int $c) {
- $a = $b*$c;
- return $a % 4;
- }
- private $privProp;
- const C1 = 1;
- public static function pubStatFunc3() {
- return $this->privFunc();
- }
- public function pubFunc1() {}
- public static function pubStatFunc1() {}
- private function privFunc() {}
- protected function __construct() {}
- protected static function protStatFunc() {}
- }
- EOT
- ,
- ];
- yield [
- [
- 'order' => [
- 'use_trait',
- 'constant',
- 'property_public_static',
- 'property_protected_static',
- 'property_private_static',
- 'property_public',
- 'property_protected',
- 'property_private',
- 'construct',
- 'destruct',
- 'magic',
- 'method_public_static',
- 'method_public_abstract_static',
- 'method_protected_static',
- 'method_protected_abstract_static',
- 'method_private_static',
- 'method_public',
- 'method_public_abstract',
- 'method_protected',
- 'method_protected_abstract',
- 'method_private',
- ],
- 'sort_algorithm' => OrderedClassElementsFixer::SORT_ALPHA,
- ],
- <<<'EOT'
- <?php
- abstract class Foo
- {
- use BarTrait;
- use BazTrait;
- const C1 = 1;
- const C2 = 2;
- public static $pubStatProp1;
- public static $pubStatProp2;
- protected static $protStatProp;
- private static $privStatProp;
- public $pubProp1;
- var $pubProp2;
- public $pubProp3;
- protected $protProp;
- private $privProp;
- protected function __construct() {}
- public function __destruct() {}
- public function __magicA() {}
- public function __magicB() {}
- public function __toString() {}
- public static function pubStatFunc1() {}
- static function pubStatFunc2() {}
- public static function pubStatFunc3() {
- return $this->privFunc();
- }
- abstract public static function absPubStatFunc1();
- protected static function protStatFunc() {}
- abstract protected static function absProtStatFunc();
- private static function privStatFunc() {}
- public function pubFunc1() {}
- function pubFunc2() {}
- public function pubFunc3(int $b, int $c) {
- $a = $b*$c;
- return $a % 4;
- }
- abstract public function absPubFunc();
- protected function protFunc() {}
- abstract protected function absProtFunc();
- private function privFunc() {}
- }
- EOT
- ,
- <<<'EOT'
- <?php
- abstract class Foo
- {
- private static function privStatFunc() {}
- protected static $protStatProp;
- use BazTrait;
- abstract public function absPubFunc();
- public static $pubStatProp2;
- public $pubProp3;
- use BarTrait;
- public function __toString() {}
- protected function protFunc() {}
- protected $protProp;
- function pubFunc2() {}
- public $pubProp1;
- abstract protected static function absProtStatFunc();
- public function __destruct() {}
- var $pubProp2;
- public function __magicB() {}
- const C2 = 2;
- public static $pubStatProp1;
- public function __magicA() {}
- private static $privStatProp;
- static function pubStatFunc2() {}
- public function pubFunc3(int $b, int $c) {
- $a = $b*$c;
- return $a % 4;
- }
- private $privProp;
- const C1 = 1;
- abstract protected function absProtFunc();
- public static function pubStatFunc3() {
- return $this->privFunc();
- }
- public function pubFunc1() {}
- public static function pubStatFunc1() {}
- private function privFunc() {}
- protected function __construct() {}
- protected static function protStatFunc() {}
- abstract public static function absPubStatFunc1();
- }
- EOT
- ,
- ];
- yield [
- [],
- <<<'EOT'
- <?php
- class Foo
- {
- const C2 = 2;
- public const C1 = 1;
- public const C3 = 3;
- protected const C4 = 4;
- private const C5 = 5;
- }
- EOT
- , <<<'EOT'
- <?php
- class Foo
- {
- private const C5 = 5;
- const C2 = 2;
- public const C1 = 1;
- protected const C4 = 4;
- public const C3 = 3;
- }
- EOT
- ];
- yield [
- ['sort_algorithm' => OrderedClassElementsFixer::SORT_ALPHA],
- <<<'EOT'
- <?php
- class Foo
- {
- public const C1 = 1;
- const C2 = 2;
- public const C3b = 3;
- protected const C4a = 4;
- private const C5 = 5;
- }
- EOT
- , <<<'EOT'
- <?php
- class Foo
- {
- private const C5 = 5;
- const C2 = 2;
- public const C1 = 1;
- protected const C4a = 4;
- public const C3b = 3;
- }
- EOT
- ];
- yield [
- ['sort_algorithm' => OrderedClassElementsFixer::SORT_ALPHA],
- <<<'EOT'
- <?php
- class Foo
- {
- const A_ = 1;
- const AA = 2;
- const Ab = 3;
- }
- EOT
- , <<<'EOT'
- <?php
- class Foo
- {
- const AA = 2;
- const Ab = 3;
- const A_ = 1;
- }
- EOT
- ];
- yield [
- ['sort_algorithm' => OrderedClassElementsFixer::SORT_ALPHA, 'case_sensitive' => true],
- <<<'EOT'
- <?php
- class Foo
- {
- const AA = 2;
- const A_ = 1;
- const Ab = 3;
- }
- EOT
- , <<<'EOT'
- <?php
- class Foo
- {
- const AA = 2;
- const Ab = 3;
- const A_ = 1;
- }
- EOT
- ];
- }
- /**
- * @param array<string, mixed> $configuration
- *
- * @dataProvider provideFix74Cases
- */
- public function testFix74(string $expected, ?string $input = null, array $configuration = []): void
- {
- $this->fixer->configure($configuration);
- $this->doTest($expected, $input);
- }
- public static function provideFix74Cases(): iterable
- {
- yield [
- '<?php
- class Foo {
- public iterable $baz;
- var ? Foo\Bar $qux;
- protected string $bar;
- private ?int $foo;
- }',
- '<?php
- class Foo {
- private ?int $foo;
- protected string $bar;
- public iterable $baz;
- var ? Foo\Bar $qux;
- }',
- ];
- yield [
- '<?php
- class Foo {
- public string $bar;
- public array $baz;
- public ?int $foo;
- public ? Foo\Bar $qux;
- }',
- '<?php
- class Foo {
- public array $baz;
- public ? Foo\Bar $qux;
- public string $bar;
- public ?int $foo;
- }',
- [
- 'sort_algorithm' => OrderedClassElementsFixer::SORT_ALPHA,
- ],
- ];
- }
- public function testWrongConfig(): void
- {
- $this->expectException(InvalidFixerConfigurationException::class);
- $this->expectExceptionMessageMatches('/^\[ordered_class_elements\] Invalid configuration: The option "order" .*\.$/');
- $this->fixer->configure(['order' => ['foo']]);
- }
- /**
- * @dataProvider provideWithConfigWithNoCandidateCases
- */
- public function testWithConfigWithNoCandidate(string $methodName1, string $methodName2): void
- {
- $template = '<?php
- class TestClass
- {
- public function %s(){}
- public function %s(){}
- }';
- $this->fixer->configure(['order' => ['use_trait'], 'sort_algorithm' => OrderedClassElementsFixer::SORT_ALPHA]);
- $this->doTest(
- sprintf($template, $methodName2, $methodName1),
- sprintf($template, $methodName1, $methodName2)
- );
- }
- public static function provideWithConfigWithNoCandidateCases(): iterable
- {
- yield ['z', '__construct'];
- yield ['z', '__destruct'];
- yield ['z', '__sleep'];
- yield ['z', 'abc'];
- }
- /**
- * @dataProvider provideFix80Cases
- *
- * @requires PHP 8.0
- */
- public function testFix80(string $expected, string $input): void
- {
- $this->doTest($expected, $input);
- }
- public static function provideFix80Cases(): iterable
- {
- yield [
- '<?php
- trait TestTrait
- {
- abstract static public function abstractStaticPublic();
- abstract private function abstractPrivate();
- abstract static private function abstractStaticPrivate();
- }
- ',
- '<?php
- trait TestTrait
- {
- abstract private function abstractPrivate();
- abstract static private function abstractStaticPrivate();
- abstract static public function abstractStaticPublic();
- }
- ',
- ];
- yield [
- '<?php class Foo
- {
- #[PublicBarAttribute0]
- public int $bar = 1;
- #[PublicAttribute0]
- #[PublicAttribute1]
- #[PublicAttribute2]
- public function fooPublic()
- {
- }
- #[PrivateAttribute0]
- private function fooPrivate()
- {
- }
- }',
- '<?php class Foo
- {
- #[PrivateAttribute0]
- private function fooPrivate()
- {
- }
- #[PublicBarAttribute0]
- public int $bar = 1;
- #[PublicAttribute0]
- #[PublicAttribute1]
- #[PublicAttribute2]
- public function fooPublic()
- {
- }
- }',
- ];
- }
- /**
- * @param array<string, mixed> $configuration
- *
- * @dataProvider provideFix81Cases
- *
- * @requires PHP 8.1
- */
- public function testFix81(string $expected, ?string $input = null, array $configuration = []): void
- {
- $this->fixer->configure($configuration);
- $this->doTest($expected, $input);
- }
- public static function provideFix81Cases(): iterable
- {
- yield [
- '<?php
- class A
- {
- readonly public string $publicProp0;
- public readonly string $publicProp1;
- public string $pubProp2;
- protected readonly string $protectedProp0;
- readonly protected string $protectedProp1;
- readonly private string $privateProp0;
- private readonly string $privateProp1;
- }
- ',
- '<?php
- class A
- {
- public string $pubProp2;
- readonly public string $publicProp0;
- public readonly string $publicProp1;
- private readonly string $privateProp1;
- readonly private string $privateProp0;
- protected readonly string $protectedProp0;
- readonly protected string $protectedProp1;
- }
- ',
- ['order' => ['property_public_readonly', 'property_public', 'property_protected_readonly', 'property_private_readonly'], 'sort_algorithm' => 'alpha'],
- ];
- yield [
- '<?php
- enum A: int
- {
- case Foo = 1;
- case Bar = 2;
- private const C1 = 1;
- function qux() {
- switch (true) {
- case 1: break;
- }
- }
- }
- ',
- '<?php
- enum A: int
- {
- private const C1 = 1;
- case Foo = 1;
- function qux() {
- switch (true) {
- case 1: break;
- }
- }
- case Bar = 2;
- }
- ',
- ];
- }
- /**
- * @dataProvider provideFix82Cases
- *
- * @requires PHP 8.2
- */
- public function testFix82(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- public static function provideFix82Cases(): iterable
- {
- yield [
- '<?php trait Foo { const C1 = 1; protected $abc = "abc"; }',
- '<?php trait Foo { protected $abc = "abc"; const C1 = 1; }',
- ];
- }
- }
|