PhpdocTo Property/Return/Param Fixer - allow fixing mixed on PHP >= 8

PHP-CS-Fixer · Apr 6, 2022 · 9d3b094 · 9d3b094
1 parent aa805a8
commit 9d3b094
Show file tree

Hide file tree

Showing 6 changed files with 89 additions and 17 deletions.
diff --git a/src/Fixer/FunctionNotation/PhpdocToParamTypeFixer.php b/src/Fixer/FunctionNotation/PhpdocToParamTypeFixer.php
@@ -39,7 +39,6 @@ final class PhpdocToParamTypeFixer extends AbstractPhpdocToTypeDeclarationFixer
      * @var array<string, true>
      */
     private const SKIPPED_TYPES = [
-        'mixed' => true,
         'resource' => true,
         'static' => true,
         'void' => true,

diff --git a/src/Fixer/FunctionNotation/PhpdocToPropertyTypeFixer.php b/src/Fixer/FunctionNotation/PhpdocToPropertyTypeFixer.php
@@ -29,7 +29,6 @@ final class PhpdocToPropertyTypeFixer extends AbstractPhpdocToTypeDeclarationFix
      * @var array<string, true>
      */
     private array $skippedTypes = [
-        'mixed' => true,
         'resource' => true,
         'null' => true,
     ];

diff --git a/src/Fixer/FunctionNotation/PhpdocToReturnTypeFixer.php b/src/Fixer/FunctionNotation/PhpdocToReturnTypeFixer.php
@@ -42,7 +42,6 @@ final class PhpdocToReturnTypeFixer extends AbstractPhpdocToTypeDeclarationFixer
      * @var array<string, true>
      */
     private array $skippedTypes = [
-        'mixed' => true,
         'resource' => true,
         'null' => true,
     ];

diff --git a/tests/Fixer/FunctionNotation/PhpdocToParamTypeFixerTest.php b/tests/Fixer/FunctionNotation/PhpdocToParamTypeFixerTest.php
@@ -28,20 +28,26 @@ final class PhpdocToParamTypeFixerTest extends AbstractFixerTestCase
     /**
      * @dataProvider provideFixCases
      */
-    public function testFix(string $expected, ?string $input = null, ?int $versionSpecificFix = null, array $config = null): void
-    {
+    public function testFix(
+        string $expected,
+        ?string $input = null,
+        ?int $availableAboveVersion = null,
+        array $config = [],
+        ?int $skipFromVersion = null
+    ): void {
+        if (null !== $skipFromVersion && \PHP_VERSION_ID >= $skipFromVersion) {
+            static::markTestSkipped(sprintf('Only available up to version %d', $skipFromVersion));
+        }
+
         if (
             null !== $input
-            && (null !== $versionSpecificFix && \PHP_VERSION_ID < $versionSpecificFix)
+            && (null !== $availableAboveVersion && \PHP_VERSION_ID < $availableAboveVersion)
         ) {
             $expected = $input;
             $input = null;
         }
 
-        if (null !== $config) {
-            $this->fixer->configure($config);
-        }
-
+        $this->fixer->configure($config);
         $this->doTest($expected, $input);
     }
 
@@ -175,12 +181,29 @@ function my_foo(string $bar, int $baz, float $tab) {}',
                     */
                     function my_foo($bar, $baz, $tab) {}',
             ],
-            'non-root class with mixed type of param' => [
+            'non-root class with mixed type of param for php < 8' => [
+                '<?php
+                    /**
+                    * @param mixed $bar
+                    */
+                    function my_foo($bar) {}',
+                null,
+                null,
+                [],
+                80000,
+            ],
+            'non-root class with mixed type of param for php >= 8' => [
+                '<?php
+                    /**
+                    * @param mixed $bar
+                    */
+                    function my_foo(mixed $bar) {}',
                 '<?php
                     /**
                     * @param mixed $bar
                     */
                     function my_foo($bar) {}',
+                80000,
             ],
             'non-root namespaced class' => [
                 '<?php /** @param My\Bar $foo */ function my_foo(My\Bar $foo) {}',

diff --git a/tests/Fixer/FunctionNotation/PhpdocToPropertyTypeFixerTest.php b/tests/Fixer/FunctionNotation/PhpdocToPropertyTypeFixerTest.php
@@ -26,9 +26,21 @@ final class PhpdocToPropertyTypeFixerTest extends AbstractFixerTestCase
     /**
      * @dataProvider provideFixCases
      */
-    public function testFix(string $expected, ?string $input = null, array $config = []): void
-    {
-        if (null !== $input && \PHP_VERSION_ID < 70400) {
+    public function testFix(
+        string $expected,
+        ?string $input = null,
+        ?int $availableAboveVersion = null,
+        array $config = [],
+        ?int $skipFromVersion = null
+    ): void {
+        if (null !== $skipFromVersion && \PHP_VERSION_ID >= $skipFromVersion) {
+            static::markTestSkipped(sprintf('Only available up to version %d', $skipFromVersion));
+        }
+
+        if (
+            null !== $input
+            && (null !== $availableAboveVersion && \PHP_VERSION_ID < $availableAboveVersion)
+        ) {
             $expected = $input;
             $input = null;
         }
@@ -116,6 +128,7 @@ class Foo {
             'do not fix scalar types when configured as such' => [
                 '<?php class Foo { /** @var int */ private $foo; }',
                 null,
+                null,
                 ['scalar_types' => false],
             ],
             'array native type' => [
@@ -138,6 +151,15 @@ class Foo {
             ],
             'skip mixed special type' => [
                 '<?php class Foo { /** @var mixed */ private $foo; }',
+                null,
+                null,
+                [],
+                80000,
+            ],
+            'fix mixed special type' => [
+                '<?php class Foo { /** @var mixed */ private mixed $foo; }',
+                '<?php class Foo { /** @var mixed */ private $foo; }',
+                80000,
             ],
             'null alone cannot be a property type' => [
                 '<?php class Foo { /** @var null */ private $foo; }',

diff --git a/tests/Fixer/FunctionNotation/PhpdocToReturnTypeFixerTest.php b/tests/Fixer/FunctionNotation/PhpdocToReturnTypeFixerTest.php
@@ -28,11 +28,20 @@ final class PhpdocToReturnTypeFixerTest extends AbstractFixerTestCase
     /**
      * @dataProvider provideFixCases
      */
-    public function testFix(string $expected, ?string $input = null, ?int $versionSpecificFix = null, array $config = []): void
-    {
+    public function testFix(
+        string $expected,
+        ?string $input = null,
+        ?int $availableAboveVersion = null,
+        array $config = [],
+        ?int $skipFromVersion = null
+    ): void {
+        if (null !== $skipFromVersion && \PHP_VERSION_ID >= $skipFromVersion) {
+            static::markTestSkipped(sprintf('Only available up to version %d', $skipFromVersion));
+        }
+
         if (
             null !== $input
-            && (null !== $versionSpecificFix && \PHP_VERSION_ID < $versionSpecificFix)
+            && (null !== $availableAboveVersion && \PHP_VERSION_ID < $availableAboveVersion)
         ) {
             $expected = $input;
             $input = null;
@@ -330,6 +339,27 @@ function bar() {}
                     function bar() {}
                 ',
             ],
+            'skip mixed type' => [
+                '<?php
+                    /** @return mixed */
+                    function bar() {}
+                ',
+                null,
+                null,
+                [],
+                80000,
+            ],
+            'fix skip mixed type' => [
+                '<?php
+                    /** @return mixed */
+                    function bar(): mixed {}
+                ',
+                '<?php
+                    /** @return mixed */
+                    function bar() {}
+                ',
+                80000,
+            ],
         ];
     }