Add more tests

Author: kubawerlos

README.md

@@ -5,7 +5,7 @@
 [![Latest stable version](https://img.shields.io/packagist/v/kubawerlos/php-cs-fixer-custom-fixers.svg?label=current%20version)](https://packagist.org/packages/kubawerlos/php-cs-fixer-custom-fixers)
 [![PHP version](https://img.shields.io/packagist/php-v/kubawerlos/php-cs-fixer-custom-fixers.svg)](https://php.net)
 [![License](https://img.shields.io/github/license/kubawerlos/php-cs-fixer-custom-fixers.svg)](LICENSE)
-![Tests](https://img.shields.io/badge/tests-3777-brightgreen.svg)
+![Tests](https://img.shields.io/badge/tests-3781-brightgreen.svg)
 [![Downloads](https://img.shields.io/packagist/dt/kubawerlos/php-cs-fixer-custom-fixers.svg)](https://packagist.org/packages/kubawerlos/php-cs-fixer-custom-fixers)
 
 [![CI status](https://github.com/kubawerlos/php-cs-fixer-custom-fixers/actions/workflows/ci.yaml/badge.svg)](https://github.com/kubawerlos/php-cs-fixer-custom-fixers/actions/workflows/ci.yaml)

src/Fixer/PhpdocNoNamedArgumentsTagFixer.php

@@ -138,7 +138,7 @@ public function fix(\SplFileInfo $file, Tokens $tokens): void
 
             $content = $tokens[$docBlockIndex]->getContent();
 
-            $newContent = Preg::replace('/@no-named-arguments.*(\\R)/', \rtrim('@no-named-arguments ' . $this->description) . '$1', $content);
+            $newContent = Preg::replace('/@no-named-arguments.*\\R/', \rtrim('@no-named-arguments ' . $this->description) . $this->whitespacesConfig->getLineEnding(), $content);
 
             if ($newContent !== $content) {
                 $tokens[$docBlockIndex] = new Token([\T_DOC_COMMENT, $newContent]);

tests/Fixer/AbstractFixerTestCase.php

@@ -159,10 +159,14 @@ final protected static function getFixer(): FixerInterface
     /**
      * @param null|array<string, mixed> $configuration
      */
-    final protected function doTest(string $expected, ?string $input = null, ?array $configuration = null): void
+    final protected function doTest(string $expected, ?string $input = null, ?array $configuration = null, ?WhitespacesFixerConfig $whitespacesFixerConfig = null): void
     {
         $fixer = self::getFixer();
 
+        if ($whitespacesFixerConfig instanceof WhitespacesFixerConfig) {
+            $fixer->setWhitespacesConfig($whitespacesFixerConfig);
+        }
+
         if ($fixer instanceof ConfigurableFixerInterface) {
             $fixer->configure($configuration ?? []);
         }

tests/Fixer/PhpdocNoNamedArgumentsTagFixerTest.php

@@ -13,6 +13,7 @@
 
 use PhpCsFixer\ConfigurationException\InvalidFixerConfigurationException;
 use PhpCsFixer\Fixer\ConfigurableFixerInterface;
+use PhpCsFixer\WhitespacesFixerConfig;
 
 /**
  * @internal
@@ -52,9 +53,9 @@ public function testIsRisky(): void
      *
      * @dataProvider provideFixCases
      */
-    public function testFix(string $expected, ?string $input = null, array $configuration = []): void
+    public function testFix(string $expected, ?string $input = null, array $configuration = [], ?WhitespacesFixerConfig $whitespacesFixerConfig = null): void
     {
-        $this->doTest($expected, $input, $configuration);
+        $this->doTest($expected, $input, $configuration, $whitespacesFixerConfig);
     }
 
     /**
@@ -93,35 +94,88 @@ class Foo {}
                 PHP,
         ];
 
-        yield 'change the description' => [
+        yield 'create PHPDoc with description' => [
             <<<'PHP'
                 <?php
 
                 /**
-                 * @no-named-arguments Some description
+                 * @no-named-arguments The description
                  */
                 class Foo {}
                 PHP,
             <<<'PHP'
                 <?php
                 class Foo {}
                 PHP,
-            ['description' => 'Some description'],
+            ['description' => 'The description'],
+        ];
+
+        yield 'add description' => [
+            <<<'PHP'
+                <?php
+                /**
+                 * @no-named-arguments New description
+                 */
+                class Foo {}
+                PHP,
+            <<<'PHP'
+                <?php
+                /**
+                 * @no-named-arguments
+                 */
+                class Foo {}
+                PHP,
+            ['description' => 'New description'],
+        ];
+
+        yield 'change description' => [
+            <<<'PHP'
+                <?php
+                /**
+                 * @no-named-arguments New description
+                 */
+                class Foo {}
+                PHP,
+            <<<'PHP'
+                <?php
+                /**
+                 * @no-named-arguments Old description
+                 */
+                class Foo {}
+                PHP,
+            ['description' => 'New description'],
+        ];
+
+        yield 'remove description' => [
+            <<<'PHP'
+                <?php
+                /**
+                 * @no-named-arguments
+                 */
+                class Foo {}
+                PHP,
+            <<<'PHP'
+                <?php
+                /**
+                 * @no-named-arguments Description to remove
+                 */
+                class Foo {}
+                PHP,
         ];
 
         yield 'multiple classes' => [
             <<<'PHP'
                 <?php
 
                 /**
-                 * @no-named-arguments Some description
+                 * @no-named-arguments
                  */
                 class Foo {}
 
                 new class {};
 
                 /**
-                 * @no-named-arguments Some description
+                 * @no-named-arguments
                  */
                 class Bar {}
                 PHP,
@@ -133,7 +187,32 @@ class Foo {}
 
                 class Bar {}
                 PHP,
-            ['description' => 'Some description'],
+        ];
+
+        yield 'tabs and Windows Line endings' => [
+            \str_replace(
+                ['    ', "\n"],
+                ["\t", "\r\n"],
+                <<<'PHP'
+                    <?php
+
+                    /**
+                     * @no-named-arguments
+                     */
+                    class Foo {}
+                    PHP,
+            ),
+            \str_replace(
+                ['    ', "\n"],
+                ["\t", "\r\n"],
+                <<<'PHP'
+                    <?php
+
+                    class Foo {}
+                    PHP,
+            ),
+            [],
+            new WhitespacesFixerConfig("\t", "\r\n"),
         ];
     }
 }