1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261 |
- <?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\Tests\Test\AbstractFixerTestCase;
- /**
- * @author Matteo Beccati <matteo@beccati.com>
- *
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\ClassNotation\NoPhp4ConstructorFixer
- */
- final class NoPhp4ConstructorFixerTest extends AbstractFixerTestCase
- {
- /**
- * @dataProvider provideFix70Cases
- * @requires PHP 7.0
- */
- public function testFix70(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- public function provideFix70Cases()
- {
- return [
- [
- '<?php $a = new class {};',
- ],
- [
- '<?php $a = new class {}?>',
- ],
- [
- '<?php
- $a = new Foo() <=> 1;
- $a = new Foo <=> 1;
- $a = new class() {};
- $a = new class() implements Foo{};
- $a = new class() /**/ extends Bar1{};
- $a = new class() extends Bar2 implements Foo{};
- $a = new class() extends Bar3 implements Foo, Foo2{};
- $a = new class() {};
- $a = new class {};
- $a = new class implements Foo{};
- $a = new class /**/ extends Bar1{};
- $a = new class extends Bar2 implements Foo{};
- $a = new class extends Bar3 implements Foo, Foo2{};
- $a = new class {}?>
- ',
- ],
- ];
- }
- /**
- * @dataProvider provideSimpleCases
- */
- public function testSimpleClass(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- public function provideSimpleCases()
- {
- return [
- [
- <<<'EOF'
- <?php
- class Foo
- {
- public function __construct($bar)
- {
- var_dump(1);
- }
- }
- EOF
- ,
- <<<'EOF'
- <?php
- class Foo
- {
- public function Foo($bar)
- {
- var_dump(1);
- }
- }
- EOF
- ],
- [
- <<<'EOF'
- <?php
- class Foo
- {
- public#
- function#
- __construct#
- (#
- $bar#
- )#
- {}
- }
- EOF
- ,
- <<<'EOF'
- <?php
- class Foo
- {
- public#
- function#
- Foo#
- (#
- $bar#
- )#
- {}
- }
- EOF
- ],
- ];
- }
- public function testNamespaces(): void
- {
- $expected = <<<'EOF'
- <?php
- namespace Baz\Qux;
- class Foo
- {
- public function __construct($bar)
- {
- var_dump(1);
- }
- public function Foo($bar)
- {
- var_dump(2);
- }
- }
- EOF;
- $this->doTest($expected);
- }
- public function testNamespaces2(): void
- {
- $expected = <<<'EOF'
- <?php
- namespace Baz\Qux
- {
- class Foo
- {
- public function __construct($bar)
- {
- var_dump(1);
- }
- public function Foo($bar)
- {
- var_dump(2);
- }
- }
- class Bar
- {
- public function Bar()
- {
- var_dump(3);
- }
- }
- }
- EOF;
- $this->doTest($expected);
- }
- public function testNamespaceGlobal(): void
- {
- $expected = <<<'EOF'
- <?php
- namespace {
- class Foo
- {
- function __construct($bar)
- {
- var_dump(1);
- }
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- namespace {
- class Foo
- {
- function Foo($bar)
- {
- var_dump(1);
- }
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testPhp5Only(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- function __construct($bar)
- {
- var_dump(1);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected);
- }
- public function testPhp4Only(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * Constructor
- */
- function __construct($bar)
- {
- var_dump(1);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * Constructor
- */
- function foO($bar)
- {
- var_dump(1);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testBothTheRightWay1(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * Constructor
- */
- public function __construct()
- {
- var_dump(1);
- }
- public function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * Constructor
- */
- public function __construct()
- {
- var_dump(1);
- }
- /**
- * PHP-4 Constructor
- */
- function Foo()
- {
- // Call PHP5!
- $this->__construct();
- }
- public function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testBothTheRightWay2(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * Constructor
- */
- public function __construct($bar)
- {
- var_dump(1);
- }
- public function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * Constructor
- */
- public function __construct($bar)
- {
- var_dump(1);
- }
- /**
- * PHP-4 Constructor
- */
- function Foo($bar)
- {
- // Call PHP5!
- $this->__construct($bar);
- }
- public function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testBothTheRightWay3(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * Constructor
- */
- public function __construct($bar = 1, $baz = null)
- {
- var_dump(1);
- }
- public function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * Constructor
- */
- public function __construct($bar = 1, $baz = null)
- {
- var_dump(1);
- }
- /**
- * PHP-4 Constructor
- */
- function Foo($bar = 1, $baz = null)
- {
- // Call PHP5!
- $this->__construct($bar, $baz);
- }
- public function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testBothTheOtherWayAround(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * PHP-4 Constructor.
- *
- * This is the real constructor. It's the one that most likely contains any meaningful info in the docblock.
- */
- private function __construct($bar)
- {
- var_dump(1);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo
- {
- /**
- * PHP-5 Constructor.
- *
- * This docblock is removed, along with the entire wrapper method.
- */
- protected function __construct($bar)
- {
- // Call The Real Constructor, not the hippy fake one!
- $this->Foo($bar);
- }
- /**
- * PHP-4 Constructor.
- *
- * This is the real constructor. It's the one that most likely contains any meaningful info in the docblock.
- */
- private function Foo($bar)
- {
- var_dump(1);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testPhp4Parent(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo extends FooParEnt
- {
- /**
- * Constructor
- */
- function __construct($bar)
- {
- parent::__construct(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo extends FooParEnt
- {
- /**
- * Constructor
- */
- function Foo($bar)
- {
- parent::FooPaRent(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testPhp4ParentInit(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function __construct($bar)
- {
- parent::init(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function Foo($bar)
- {
- parent::init(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testMixedParent(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function __construcT($bar)
- {
- parent::__construct(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function __construcT($bar)
- {
- parent::FooParenT(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testMixedParent2(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function __construcT($bar)
- {
- parent::__construct(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function __construcT($bar)
- {
- $this->FooParenT(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testParentOther(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function __construct($bar)
- {
- parent::__construct(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function Foo($bar)
- {
- $this->FooParent(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testParentOther2(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function __construct($bar)
- {
- parent::__construct(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo extends FooParent
- {
- /**
- * Constructor
- */
- function Foo($bar)
- {
- FooParent::FooParent(1);
- var_dump(9);
- }
- function bar()
- {
- var_dump(3);
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testClassWithAnonymous(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo {
- private $bar;
- public function __construct()
- {
- $this->bar = function () {};
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo {
- private $bar;
- public function Foo()
- {
- $this->bar = function () {};
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testClassWithComments(): void
- {
- $expected = <<<'EOF'
- <?php
- class /* test */
- // another
- Foo {
- public function /* test */ __construct($param) {
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class /* test */
- // another
- Foo {
- public function /* test */ Foo($param) {
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testAlphaBeta(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- public function Foo()
- {
- echo 'alpha';
- }
- public function __construct()
- {
- echo 'beta';
- }
- }
- EOF;
- $this->doTest($expected);
- }
- public function testAlphaBetaTrick1(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- public function Foo()
- {
- // This is not $this->__construct()
- echo 'alpha';
- }
- public function __construct()
- {
- echo 'beta';
- }
- }
- EOF;
- $this->doTest($expected);
- }
- public function testAlphaBetaTrick2(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- public function Foo()
- {
- echo 'alpha';
- }
- public function __construct()
- {
- // This is not $this->Foo()
- echo 'beta';
- }
- }
- EOF;
- $this->doTest($expected);
- }
- public function testAlphaBetaTrick3(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- public function Foo()
- {
- echo 'alpha';
- /* yeah, ok let's construct it anyway */
- $this->__construct();
- }
- public function __construct()
- {
- echo 'beta';
- }
- }
- EOF;
- $this->doTest($expected);
- }
- public function testAlphaBetaTrick4WithAnotherClass(): void
- {
- $expected = <<<'EOF'
- <?php
- class Foo
- {
- public function Foo()
- {
- echo 'alpha';
- }
- public function __construct()
- {
- $this->Foo();
- // Do something more!
- echo 'beta';
- }
- }
- Class Bar
- {
- function __construct()
- {
- $this->foo = 1;
- }
- }
- EOF;
- $input = <<<'EOF'
- <?php
- class Foo
- {
- public function Foo()
- {
- echo 'alpha';
- }
- public function __construct()
- {
- $this->Foo();
- // Do something more!
- echo 'beta';
- }
- }
- Class Bar
- {
- function bar()
- {
- $this->foo = 1;
- }
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testAbstract(): void
- {
- $expected = <<<'EOF'
- <?php
- abstract class Foo
- {
- abstract function Foo();
- }
- EOF;
- $this->doTest($expected);
- }
- public function testAbstractTrick(): void
- {
- $expected = <<<'EOF'
- <?php
- abstract class Foo
- {
- abstract public function Foo();
- public function bar()
- {
- // This is messed up, I know
- $this->__construct();
- }
- public function __construct()
- {
- $this->baz = 1;
- }
- }
- EOF;
- $this->doTest($expected);
- }
- public function testParentMultipleClasses(): void
- {
- $expected = <<<'EOF'
- <?php
- class Class1 extends Parent1
- {
- function __construct($foo)
- {
- parent::__construct();
- echo "something";
- }
- }
- class Class2 extends Parent2
- {
- function __construct($foo)
- {
- echo "something";
- }
- }
- ?>
- EOF;
- $input = <<<'EOF'
- <?php
- class Class1 extends Parent1
- {
- function __construct($foo)
- {
- $this->Parent1();
- echo "something";
- }
- }
- class Class2 extends Parent2
- {
- function __construct($foo)
- {
- echo "something";
- }
- }
- ?>
- EOF;
- $this->doTest($expected, $input);
- }
- public function testInfiniteRecursion(): void
- {
- $expected = <<<'EOF'
- <?php
- class Parent1
- {
- function __construct()
- {
- echo "foobar";
- }
- }
- class Class1 extends Parent1
- {
- function __construct($foo)
- {
- parent::__construct();
- echo "something";
- }
- }
- ?>
- EOF;
- $input = <<<'EOF'
- <?php
- class Parent1
- {
- function __construct()
- {
- echo "foobar";
- }
- }
- class Class1 extends Parent1
- {
- function Class1($foo)
- {
- $this->__construct();
- echo "something";
- }
- }
- ?>
- EOF;
- $this->doTest($expected, $input);
- }
- /**
- * @dataProvider provideFixPhp80Cases
- * @requires PHP 8.0
- */
- public function testFixPhp80(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- public function provideFixPhp80Cases()
- {
- yield [
- <<<'EOF'
- <?php
- class Foo
- {
- public function __construct($bar,)
- {
- var_dump(1);
- }
- }
- EOF
- ,
- <<<'EOF'
- <?php
- class Foo
- {
- public function Foo($bar,)
- {
- var_dump(1);
- }
- }
- EOF
- ];
- yield [
- '<?php
- class Foo
- {
- public function __construct()
- {
- }
- }',
- '<?php
- class Foo
- {
- public function Foo()
- {
- }
- }',
- ];
- yield [
- '<?php
- class Foo
- {
- public function __construct()
- {
- $this?->__construct();
- }
- }',
- '<?php
- class Foo
- {
- public function Foo()
- {
- $this?->__construct();
- }
- }',
- ];
- yield [
- '<?php
- class Foo extends Bar
- {
- public function __construct()
- {
- parent::__construct();
- }
- }',
- '<?php
- class Foo extends Bar
- {
- public function Foo()
- {
- $this?->Bar();
- }
- }',
- ];
- yield [
- '<?php
- class Foo
- {
- /**
- * Constructor
- */
- public function __construct($bar = 1, $baz = null)
- {
- var_dump(1);
- }
- }
- ',
- '<?php
- class Foo
- {
- /**
- * Constructor
- */
- public function __construct($bar = 1, $baz = null)
- {
- var_dump(1);
- }
- /**
- * PHP-4 Constructor
- */
- function Foo($bar = 1, $baz = null)
- {
- $this?->__construct($bar, $baz);
- }
- }
- ',
- ];
- }
- }
|