PhpdocNoSuperfluousParamFixerTest - handle multiline array shapes (#1066)

Author: kubawerlos

CHANGELOG.md

@@ -1,5 +1,8 @@
 # CHANGELOG for PHP CS Fixer: custom fixers
 
+## v3.33.0
+- Update minimum PHP CS Fixer version to 3.86.0
+
 ## v3.32.0
 - Add FunctionParameterSeparationFixer
 - Update minimum PHP CS Fixer version to 3.85.0

composer.json

@@ -13,7 +13,7 @@
         "php": "^7.4 || ^8.0",
         "ext-filter": "*",
         "ext-tokenizer": "*",
-        "friendsofphp/php-cs-fixer": "^3.85"
+        "friendsofphp/php-cs-fixer": "^3.86"
     },
     "require-dev": {
         "phpunit/phpunit": "^9.6.22 || 10.5.45 || ^11.5.7"

tests/Fixer/PhpdocNoSuperfluousParamFixerTest.php

@@ -321,7 +321,7 @@ public function __construct(mixed $callback) {}
                  * @param array{value: string} $param2
                  * @param ARRAY{value: string} $param3
                  */
-                function foo(array $param1, $param2, ARRAY $param3) {}
+                function singleLineArrayShapes(array $param1, $param2, ARRAY $param3) {}
                 PHP,
             <<<'PHP'
                 <?php
@@ -331,7 +331,53 @@ function foo(array $param1, $param2, ARRAY $param3) {}
                  * @param array{value: string} $param404
                  * @param ARRAY{value: string} $param3
                  */
-                function foo(array $param1, $param2, ARRAY $param3) {}
+                function singleLineArrayShapes(array $param1, $param2, ARRAY $param3) {}
+                PHP,
+        ];
+
+        yield [
+            <<<'PHP'
+                <?php
+                /**
+                 */
+                function removeMultiLineArrayShapes() {}
+                PHP,
+            <<<'PHP'
+                <?php
+                /**
+                 * @param array{value: string} $param1
+                 * @param array{
+                 *     value: string
+                 * } $param2
+                 * @param array{value: string} $param3
+                 * @param array{
+                 *     value1: bool,
+                 *     value2: int,
+                 *     value3: string,
+                 * } $param4
+                 * @param array{value: string} $param5
+                 */
+                function removeMultiLineArrayShapes() {}
+                PHP,
+        ];
+
+        yield [
+            <<<'PHP'
+                <?php
+                /**
+                 * @param array{value: string} $param1
+                 * @param array{
+                 *     value: string
+                 * } $param2
+                 * @param array{value: string} $param3
+                 * @param array{
+                 *     value1: bool,
+                 *     value2: int,
+                 *     value3: string,
+                 * } $param4
+                 * @param array{value: string} $param5
+                 */
+                function doNotRemoveMultiLineArrayShapes($param1, $param2, $param3, $param4, $param5) {}
                 PHP,
         ];
     }