123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781 |
- <?php
- /*
- * 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;
- use PhpCsFixer\WhitespacesFixerConfig;
- /**
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\ClassNotation\MethodSeparationFixer
- */
- final class MethodSeparationFixerTest extends AbstractFixerTestCase
- {
- /**
- * @param string $expected
- * @param null|string $input
- *
- * @dataProvider provideFixClassesCases
- */
- public function testFixClasses($expected, $input = null)
- {
- $this->doTest($expected, $input);
- }
- public function provideFixClassesCases()
- {
- $cases = [];
- $cases[] = ['<?php
- class SomeClass1
- {
- // This comment
- // is multiline.
- public function echoA()
- {
- echo "a";
- }
- }
- '];
- $cases[] = [
- '<?php
- class SomeClass2
- {
- // This comment
- /* is multiline. */
- public function echoA()
- {
- echo "a";
- }
- }
- ',
- '<?php
- class SomeClass2
- {
- // This comment
- /* is multiline. */public function echoA()
- {
- echo "a";
- }
- }
- ',
- ];
- $cases[] = [
- '<?php
- class SomeClass3
- {
- // This comment
- // is multiline.
- public function echoA()
- {
- echo "a";
- }
- }
- ', ];
- $cases[] = [
- '<?php
- class SomeClass1
- {
- private $a; //
- public function methodA()
- {
- }
- private $b;
- //
- public function methodB()
- {
- }
- // C
- public function methodC()
- {
- }
- // D
- public function methodD()
- {
- }
- /* E */
- public function methodE()
- {
- }
- /* F */
- public function methodF()
- {
- }
- }
- ',
- '<?php
- class SomeClass1
- {
- private $a; //
- public function methodA()
- {
- }
- private $b;
- //
- public function methodB()
- {
- }
- // C
- public function methodC()
- {
- }
- // D
- public function methodD()
- {
- }
- /* E */
- public function methodE()
- {
- }
- /* F */
- public function methodF()
- {
- }
- }
- ', ];
- $cases[] = ['<?php
- class SomeClass
- {
- // comment
- public function echoA()
- {
- echo "a";
- }
- }
- '];
- $cases[] = ['<?php
- class SomeClass
- {
- // This comment
- // is multiline.
- public function echoA()
- {
- echo "a";
- }
- }
- '];
- $cases[] = [
- '<?php
- class SomeClass
- {
- // comment
- public function echoA()
- {
- echo "a";
- }
- }
- ',
- '<?php
- class SomeClass
- {
- // comment
- public function echoA()
- {
- echo "a";
- }
- }
- ',
- ];
- $cases[] = [
- '<?php
- class SomeClass
- {
- /* comment */
- public function echoB()
- {
- echo "a";
- }
- }
- ',
- '<?php
- class SomeClass
- {
- /* comment */public function echoB()
- {
- echo "a";
- }
- }
- ',
- ];
- $cases[] = [
- '<?php
- class SomeClass
- {
- /* comment */
- public function echoC()
- {
- echo "a";
- }
- }
- ',
- '<?php
- class SomeClass
- {
- /* comment */ public function echoC()
- {
- echo "a";
- }
- }
- ',
- ];
- $cases[] = [
- '<?php
- abstract class MethodTest2
- {
- public function method045()
- {
- $files = null;
- if (!empty($files)) {
- $this->filter(
- function (\SplFileInfo $file) use ($files) {
- return !in_array($file->getRelativePathname(), $files, true);
- }
- );
- }
- }
- private $a;
- public static function method145()
- {
- }
- abstract protected function method245();
- // comment
- final private function method345()
- {
- }
- }
- function test1(){ echo 1;}
- function test2(){ echo 2;}',
- '<?php
- abstract class MethodTest2
- {
- public function method045()
- {
- $files = null;
- if (!empty($files)) {
- $this->filter(
- function (\SplFileInfo $file) use ($files) {
- return !in_array($file->getRelativePathname(), $files, true);
- }
- );
- }
- }
- private $a;
- public static function method145()
- {
- }
- abstract protected function method245();
- // comment
- final private function method345()
- {
- }
- }
- function test1(){ echo 1;}
- function test2(){ echo 2;}',
- ];
- $cases[] = [
- '<?php
- /*
- * This file is part of the PHP CS utility.
- *
- * (c) Fabien Potencier <fabien@symfony.com>
- *
- * This source file is subject to the MIT license that is bundled
- * with this source code in the file LICENSE.
- */
- namespace PhpCsFixer\Linter;
- /**
- * Dummy linter. No linting is performed. No error is raised.
- *
- * @author Dariusz Rumiński <dariusz.ruminski@gmail.com>
- *
- * @internal
- */
- final class NullLinter implements LinterInterface
- {
- /**
- * {@inheritdoc}
- */
- public function lintFile($path)
- {
- unset($path);
- }
- /**
- * {@inheritdoc}
- */
- public function lintSource($source)
- {
- unset($source);
- }
- }
- ',
- ];
- // do not touch anonymous functions (since PHP doesn't allow
- // for class attributes being functions :(, we only have to test
- // those used within methods)
- $cases[] = [
- '<?php
- class MethodTestAnonymous
- {
- public function method444a()
- {
- $text = "hello";
- $example = function ($arg) use ($message) {
- var_dump($arg . " " . $message);
- };
- $example($text);
- $example = function($arg) use ($message) {
- var_dump($arg . " " . $message);
- };
- $example = function /*test*/ ($arg) use ($message) {
- var_dump($arg . " " . $message);
- };
- }
- }',
- ];
- $cases[] = [
- '<?php
- class MethodTest1
- {
- private $c; //
- public function method444a()
- {
- }
- /**
- *
- */
- public function method444b()
- {
- }
- //
- public function method444c()
- {
- }
- private $a;
- public function method444d()
- {
- }
- private $b;
- //
- public function method444e()
- {
- }
- public function method444f()
- {
- }
- private $d; //
- public function method444f1()
- {
- }
- /**/
- public function method444g()
- {
- }
- }',
- '<?php
- class MethodTest1
- {
- private $c; //
- public function method444a()
- {
- }
- /**
- *
- */
- public function method444b()
- {
- }
- //
- public function method444c()
- {
- }
- private $a;
- public function method444d()
- {
- }
- private $b;
- //
- public function method444e()
- {
- }
- public function method444f()
- {
- }
- private $d; //
- public function method444f1()
- {
- }
- /**/
- public function method444g()
- {
- }
- }',
- ];
- // spaces between methods
- $cases[] = [
- '<?php
- abstract class MethodTest3
- {
- public function method021()
- {
- }
- public static function method121()
- {
- }
- abstract protected function method221(); '.'
- final private function method321a()
- {
- }
- }',
- '<?php
- abstract class MethodTest3
- {
- public function method021()
- {
- }
- public static function method121()
- {
- }
- abstract protected function method221();
- '.'
- final private function method321a()
- {
- }
- }', ];
- // don't change correct code
- $cases[] = [
- '<?php
- class SmallHelperException extends \Exception
- {
- public function getId111()
- {
- return 1;
- }
- public function getMessage111()
- {
- return \'message\';
- }
- }
- class MethodTest123124124
- {
- public function method111a(){}
- public function method211a(){}
- }',
- ];
- // do not touch function out of class scope
- $cases[] = [
- '<?php
- function test0() {
- }
- class MethodTest4
- {
- public function method122b()
- {
- }
- public function method222b()
- {
- }
- }
- function test() {
- }
- function test2() {
- }
- ',
- ];
- return $cases;
- }
- /**
- * @param string $expected
- * @param null|string $input
- *
- * @dataProvider provideFixTraitsCases
- */
- public function testFixTraits($expected, $input = null)
- {
- $this->doTest($expected, $input);
- }
- public function provideFixTraitsCases()
- {
- $cases = [];
- // do not touch well formatted traits
- $cases[] = [
- '<?php
- trait OkTrait
- {
- function getReturnTypeOk()
- {
- }
- /**
- *
- */
- function getReturnDescriptionOk()
- {
- }
- }',
- ];
- $cases[] = [
- '<?php
- trait ezcReflectionReturnInfo {
- public $x = 1;
- protected function getA(){echo 1;}
- function getB(){echo 2;}
- protected function getC(){echo 3;}
- /** Description */
- function getD(){echo 4;}
- protected function getE(){echo 3;}
- private $a;
- function getF(){echo 4;}
- }',
- '<?php
- trait ezcReflectionReturnInfo {
- public $x = 1;
- protected function getA(){echo 1;}function getB(){echo 2;}
- protected function getC(){echo 3;}/** Description */function getD(){echo 4;}
- protected function getE(){echo 3;}private $a;function getF(){echo 4;}
- }',
- ];
- $cases[] = [
- '<?php
- trait SomeReturnInfo {
- function getReturnType()
- {
- }
- function getReturnDescription()
- {
- }
- function getReturnDescription2()
- {
- }
- abstract public function getWorld();
- }',
- '<?php
- trait SomeReturnInfo {
- function getReturnType()
- {
- }
- function getReturnDescription()
- {
- } function getReturnDescription2()
- {
- }
- abstract public function getWorld();
- }',
- ];
- return $cases;
- }
- /**
- * @param string $expected
- * @param null|string $input
- *
- * @dataProvider provideFixInterfaceCases
- */
- public function testFixInterface($expected, $input = null)
- {
- $this->doTest($expected, $input);
- }
- public function provideFixInterfaceCases()
- {
- $cases = [];
- $cases[] = [
- '<?php
- interface TestInterface
- {
- public function testInterfaceMethod4();
- public function testInterfaceMethod5();
- /**
- * {@link}
- */ '.'
- public function testInterfaceMethod6();
- public function testInterfaceMethod7();
- public function testInterfaceMethod8();
- }',
- '<?php
- interface TestInterface
- { public function testInterfaceMethod4();
- public function testInterfaceMethod5();
- /**
- * {@link}
- */ '.'
- public function testInterfaceMethod6();
- public function testInterfaceMethod7(); public function testInterfaceMethod8();
- }',
- ];
- // do not touch well formatted interfaces
- $cases[] = [
- '<?php
- interface TestInterfaceOK
- {
- public function testMethod1();
- public function testMethod2();
- }',
- ];
- // method after trait use
- $cases[] = [
- '<?php
- trait ezcReflectionReturnInfo {
- function getReturnDescription() {}
- }
- class ezcReflectionMethod extends ReflectionMethod {
- use ezcReflectionReturnInfo;
- function afterUseTrait(){}
- function afterUseTrait2(){}
- }',
- '<?php
- trait ezcReflectionReturnInfo {
- function getReturnDescription() {}
- }
- class ezcReflectionMethod extends ReflectionMethod {
- use ezcReflectionReturnInfo;function afterUseTrait(){}function afterUseTrait2(){}
- }',
- ];
- return $cases;
- }
- /**
- * @param string $expected
- * @param null|string $input
- *
- * @dataProvider provideMessyWhitespacesCases
- */
- public function testMessyWhitespaces($expected, $input = null)
- {
- $this->fixer->setWhitespacesConfig(new WhitespacesFixerConfig("\t", "\r\n"));
- $this->doTest($expected, $input);
- }
- public function provideMessyWhitespacesCases()
- {
- return [
- [
- "<?php\r\nclass SomeClass\r\n{\r\n // comment\n\n public function echoA()\r\n {\r\n echo 'a';\r\n }\r\n}\r\n",
- "<?php\r\nclass SomeClass\r\n{\r\n // comment\n\n\n public function echoA()\r\n {\r\n echo 'a';\r\n }\r\n}\r\n",
- ],
- [
- "<?php\r\nclass SomeClass\r\n{\r\n // comment\r\n\r\n public function echoA()\r\n {\r\n echo 'a';\r\n }\r\n}\r\n",
- "<?php\r\nclass SomeClass\r\n{\r\n // comment\r\n\r\n\r\n public function echoA()\r\n {\r\n echo 'a';\r\n }\r\n}\r\n",
- ],
- ];
- }
- }
|