Pass ReflectionMethod object for test method to avoid overwriting the $className and $methodName variables in the loop

Author: sebastianbergmann

src/Metadata/Api/DataProvider.php

@@ -62,7 +62,7 @@ public function providedData(string $className, string $methodName): ?array
                 $this->triggerWarningForMixingOfDataProviderAndTestWith($testMethod);
             }
 
-            $data = $this->dataProvidedByMethods($className, $methodName, $dataProvider);
+            $data = $this->dataProvidedByMethods($testMethod, $dataProvider);
         } else {
             $data = $this->dataProvidedByMetadata($testWith);
         }
@@ -99,16 +99,13 @@ public function providedData(string $className, string $methodName): ?array
     }
 
     /**
-     * @param class-string<TestCase> $className
-     * @param non-empty-string       $methodName
-     *
      * @throws InvalidDataProviderException
      *
      * @return array<ProvidedData>
      */
-    private function dataProvidedByMethods(string $className, string $methodName, MetadataCollection $dataProvider): array
+    private function dataProvidedByMethods(ReflectionMethod $testMethod, MetadataCollection $dataProvider): array
     {
-        $testMethod    = new Event\Code\ClassMethod($className, $methodName);
+        $testMethod    = new Event\Code\ClassMethod($testMethod->getDeclaringClass()->getName(), $testMethod->getName());
         $methodsCalled = [];
         $result        = [];