NarrowPrivateClassMethodParamTypeRule: Skip variadics (#66)

* NarrowPrivateClassMethodParamTypeRule: Skip variadics

* remove unused import

Author: staabm

src/Rules/NarrowPrivateClassMethodParamTypeRule.php

@@ -110,6 +110,10 @@ private function validateArgVsParamTypes(array $args, MethodCall $methodCall, Sc
                 continue;
             }
 
+            if ($param->variadic) {
+                return [];
+            }
+
             $paramRuleError = $this->validateParam($param, $position, $arg->value, $scope);
             if (! $paramRuleError instanceof RuleError) {
                 continue;

tests/Rules/NarrowPrivateClassMethodParamTypeRule/Fixture/SkipVariadics.php

@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\TypePerfect\Tests\Rules\NarrowPrivateClassMethodParamTypeRule\Fixture;
+
+use PhpParser\Node;
+
+class SkipVariadics
+{
+    public function run(Node $node, Node $node2, Node $node3)
+    {
+        $arr = [$node, $node2, $node3];
+        $this->takesVariadic(...$arr);
+    }
+
+    private function takesVariadic(Node ...$node)
+    {
+
+    }
+}

tests/Rules/NarrowPrivateClassMethodParamTypeRule/NarrowPrivateClassMethodParamTypeRuleTest.php

@@ -50,6 +50,7 @@ public static function provideData(): Iterator
         yield [__DIR__ . '/Fixture/DoubleShot.php', [[$argErrorMessage, 15], [$paramErrorMessage, 15]]];
         yield [__DIR__ . '/Fixture/SkipGenericType.php', []];
         yield [__DIR__ . '/Fixture/SkipAbstractBase.php', []];
+        yield [__DIR__ . '/Fixture/SkipVariadics.php', []];
     }
 
     /**