123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478 |
- <?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\Phpdoc;
- use PhpCsFixer\Tests\Test\AbstractFixerTestCase;
- /**
- * @author Graham Campbell <hello@gjcampbell.co.uk>
- *
- * @internal
- *
- * @covers \PhpCsFixer\Fixer\Phpdoc\NoBlankLinesAfterPhpdocFixer
- *
- * @extends AbstractFixerTestCase<\PhpCsFixer\Fixer\Phpdoc\NoBlankLinesAfterPhpdocFixer>
- */
- final class NoBlankLinesAfterPhpdocFixerTest extends AbstractFixerTestCase
- {
- public function testSimpleExampleIsNotChanged(): void
- {
- $input = <<<'EOF'
- <?php
- /**
- * This is the bar class.
- */
- class Bar
- {
- /**
- * @return void
- */
- public function foo()
- {
- //
- }
- }
- EOF;
- $this->doTest($input);
- }
- public function testComplexExampleIsNotChanged(): void
- {
- $input = <<<'EOF'
- <?php
- /**
- * This is the hello function.
- * Yeh, this layout should be allowed.
- * We're fixing lines following a docblock.
- */
- function hello($foo) {}
- /**
- * This is the bar class.
- */
- final class Bar
- {
- /**
- * @return void
- */
- public static function foo()
- {
- //
- }
- /**
- * @return void
- */
- static private function bar123() {}
- /*
- * This T_COMMENT should not be moved
- *
- * Only T_DOC_COMMENT should be moved
- */
- final protected
- // mixin' it up a bit
- function baz() {
- }
- /*
- * This T_COMMENT should not be moved
- *
- * Only T_DOC_COMMENT should be moved
- */
- public function cool() {}
- /**
- * This is the first docblock
- *
- * Not removing blank line here.
- * No element is being documented
- */
- /**
- * Another docblock
- */
- public function silly() {}
- }
- EOF;
- $this->doTest($input);
- }
- public function testCommentsAreNotChanged(): void
- {
- $input = <<<'EOF'
- <?php
- /*
- * This file is part of xyz.
- *
- * License etc...
- */
- namespace Foo\Bar;
- EOF;
- $this->doTest($input);
- }
- public function testLineBeforeDeclareIsNotRemoved(): void
- {
- $expected = <<<'EOF'
- <?php
- /**
- * This is some license header.
- */
- declare(strict_types=1);
- EOF;
- $this->doTest($expected);
- }
- public function testLineBeforeUseStatementIsNotRemoved(): void
- {
- $expected = <<<'EOF'
- <?php
- /**
- * This is some license header.
- */
- use Foo\Bar;
- EOF;
- $this->doTest($expected);
- }
- /**
- * @dataProvider provideLineBeforeIncludeOrRequireIsNotRemovedCases
- */
- public function testLineBeforeIncludeOrRequireIsNotRemoved(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string}>
- */
- public static function provideLineBeforeIncludeOrRequireIsNotRemovedCases(): iterable
- {
- yield [
- <<<'EOF'
- <?php
- /**
- * This describes what my script does.
- */
- include 'vendor/autoload.php';
- EOF,
- ];
- yield [
- <<<'EOF'
- <?php
- /**
- * This describes what my script does.
- */
- include_once 'vendor/autoload.php';
- EOF,
- ];
- yield [
- <<<'EOF'
- <?php
- /**
- * This describes what my script does.
- */
- require 'vendor/autoload.php';
- EOF,
- ];
- yield [
- <<<'EOF'
- <?php
- /**
- * This describes what my script does.
- */
- require_once 'vendor/autoload.php';
- EOF,
- ];
- }
- public function testLineWithSpacesIsRemovedWhenNextTokenIsIndented(): void
- {
- $this->doTest(
- '<?php
- /**
- * PHPDoc with a line with space
- */
- class Foo {}',
- '<?php
- /**
- * PHPDoc with a line with space
- */
- '.'
- class Foo {}'
- );
- }
- public function testLineWithSpacesIsRemovedWhenNextTokenIsNotIndented(): void
- {
- $this->doTest(
- '<?php
- /**
- * PHPDoc with a line with space
- */
- class Foo {}',
- '<?php
- /**
- * PHPDoc with a line with space
- */
- '.'
- class Foo {}'
- );
- }
- public function testFixesSimpleClass(): void
- {
- $expected = <<<'EOF'
- <?php
- /**
- * This is the bar class.
- */
- class Bar {}
- EOF;
- $input = <<<'EOF'
- <?php
- /**
- * This is the bar class.
- */
- class Bar {}
- EOF;
- $this->doTest($expected, $input);
- }
- public function testFixesIndentedClass(): void
- {
- $expected = <<<'EOF'
- <?php
- /**
- *
- */
- class Foo {
- private $a;
- }
- EOF;
- $input = <<<'EOF'
- <?php
- /**
- *
- */
- class Foo {
- private $a;
- }
- EOF;
- $this->doTest($expected, $input);
- }
- public function testFixesOthers(): void
- {
- $expected = <<<'EOF'
- <?php
- /**
- * Constant!
- */
- const test = 'constant';
- /**
- * Foo!
- */
- $foo = 123;
- EOF;
- $input = <<<'EOF'
- <?php
- /**
- * Constant!
- */
- const test = 'constant';
- /**
- * Foo!
- */
- $foo = 123;
- EOF;
- $this->doTest($expected, $input);
- }
- public function testWhitespaceInDocBlockAboveNamespaceIsNotTouched(): void
- {
- $expected = <<<'EOF'
- <?php
- /**
- * This is a file-level docblock.
- */
- namespace Foo\Bar\Baz;
- EOF;
- $this->doTest($expected);
- }
- public function testFixesWindowsStyle(): void
- {
- $expected = "<?php\r\n /** * Constant! */\n \$foo = 123;";
- $input = "<?php\r\n /** * Constant! */\r\n\r\n\r\n \$foo = 123;";
- $this->doTest($expected, $input);
- }
- /**
- * Empty line between typehinting docs and return statement should be preserved.
- *
- * @dataProvider provideInlineTypehintingDocsBeforeFlowBreakCases
- */
- public function testInlineTypehintingDocsBeforeFlowBreak(string $expected, ?string $input = null): void
- {
- $this->doTest($expected, $input);
- }
- /**
- * @return iterable<array{string}>
- */
- public static function provideInlineTypehintingDocsBeforeFlowBreakCases(): iterable
- {
- yield [
- <<<'EOF'
- <?php
- function parseTag($tag)
- {
- $tagClass = get_class($tag);
- if ('phpDocumentor\Reflection\DocBlock\Tag\VarTag' === $tagClass) {
- /** @var DocBlock\Tag\VarTag $tag */
- return $tag->getDescription();
- }
- }
- EOF,
- ];
- yield [
- <<<'EOF'
- <?php
- function parseTag($tag)
- {
- $tagClass = get_class($tag);
- if ('phpDocumentor\Reflection\DocBlock\Tag\VarTag' === $tagClass) {
- /** @var DocBlock\Tag\VarTag $tag */
- throw new Exception($tag->getDescription());
- }
- }
- EOF,
- ];
- yield [
- <<<'EOF'
- <?php
- function parseTag($tag)
- {
- $tagClass = get_class($tag);
- if ('phpDocumentor\Reflection\DocBlock\Tag\VarTag' === $tagClass) {
- /** @var DocBlock\Tag\VarTag $tag */
- goto FOO;
- }
- FOO:
- }
- EOF,
- ];
- yield [
- <<<'EOF'
- <?php
- function parseTag($tag)
- {
- while (true) {
- $tagClass = get_class($tag);
- if ('phpDocumentor\Reflection\DocBlock\Tag\VarTag' === $tagClass) {
- /** @var DocBlock\Tag\VarTag $tag */
- continue;
- }
- }
- }
- EOF,
- ];
- yield [
- <<<'EOF'
- <?php
- function parseTag($tag)
- {
- while (true) {
- $tagClass = get_class($tag);
- if ('phpDocumentor\Reflection\DocBlock\Tag\VarTag' === $tagClass) {
- /** @var DocBlock\Tag\VarTag $tag */
- break;
- }
- }
- }
- EOF,
- ];
- }
- }
|