Fix RemoveNamedArgsInDataProviderRector so that it is not applied if no change is made

rules-tests/PHPUnit100/Rector/Class_/RemoveNamedArgsInDataProviderRector/Fixture/skip_provide_data_with_no_named_args.php.inc

@@ -0,0 +1,22 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\PHPUnit100\Rector\Class_\RemoveNamedArgsInDataProviderRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipProvideDataWithNoNamedArgs extends TestCase
+{
+    /**
+     * @dataProvider provideData()
+     */
+    public function test()
+    {
+    }
+
+    public static function provideData()
+    {
+        yield [100];
+    }
+}
+
+?>

rules/PHPUnit100/Rector/Class_/RemoveNamedArgsInDataProviderRector.php

@@ -5,6 +5,7 @@
 namespace Rector\PHPUnit\PHPUnit100\Rector\Class_;
 
 use PhpParser\Node;
+use PhpParser\Node\Expr;
 use PhpParser\Node\Expr\Array_;
 use PhpParser\Node\Expr\ArrayItem;
 use PhpParser\Node\Expr\Yield_;
@@ -99,11 +100,14 @@ public function refactor(Node $node): ?Node
             /** @var Expression $stmt */
             foreach ($dataProvider->getStmts() ?? [] as $stmt) {
                 $expr = $stmt->expr;
+                $arrayChanged = false;
                 if ($expr instanceof Yield_) {
-                    $this->handleStmt($expr->value);
-                    $hasChanged = true;
+                    $arrayChanged = $this->handleArray($expr->value);
                 } elseif ($expr instanceof Array_) {
-                    $this->handleStmt($expr);
+                    $arrayChanged = $this->handleArray($expr);
+                }
+
+                if ($arrayChanged) {
                     $hasChanged = true;
                 }
             }
@@ -116,16 +120,20 @@ public function refactor(Node $node): ?Node
         return null;
     }
 
-    private function handleStmt(Array_ $array): void
+    private function handleArray(Array_ $array): bool
     {
+        $hasChanged = false;
         foreach ($array->items as $item) {
             if (! $item instanceof ArrayItem) {
                 continue;
             }
 
-            if (! $item->key instanceof Int_) {
+            if (! $item->key instanceof Int_ && $item->key instanceof Expr) {
                 $item->key = null;
+                $hasChanged = true;
             }
         }
+
+        return $hasChanged;
     }
 }