fix

Author: staabm

src/Rules/PHPUnit/DataProviderDataRule.php

@@ -16,16 +16,12 @@
 class DataProviderDataRule implements Rule
 {
 
-	private ReflectionProvider $reflectionProvider;
-
 	private TestMethodsHelper $testMethodsHelper;
 
 	public function __construct(
-		ReflectionProvider $reflectionProvider,
 		TestMethodsHelper $testMethodsHelper
 	)
 	{
-		$this->reflectionProvider = $reflectionProvider;
 		$this->testMethodsHelper = $testMethodsHelper;
 	}
 
@@ -80,10 +76,14 @@ public function processNode(Node $node, Scope $scope): array
 
 		foreach ($node->expr->items as $item) {
 			if (!$item->value instanceof Node\Expr\Array_) {
-				return [];
+				continue;
 			}
 
 			$args = $this->arrayItemsToArgs($item->value);
+			if ($args === null) {
+				continue;
+			}
+
 			$var = new Node\Expr\New_(new Node\Name($classReflection->getName()));
 			$scope->invokeNodeCallback(new Node\Expr\MethodCall(
 				$var,

tests/PHPStan/Rules/CompositeRule.php

@@ -6,7 +6,10 @@
 use PHPStan\Analyser\Scope;
 use function get_class;
 
-class CompositeRule implements Rule
+/**
+ * @implements Rule<Node>
+ */
+final class CompositeRule implements Rule
 {
 
 	private DirectRegistry $registry;

tests/Rules/PHPUnit/DataProviderDataRuleTest.php

@@ -18,7 +18,7 @@
 use PHPStan\Type\FileTypeMapper;
 
 /**
- * @extends RuleTestCase<DataProviderDataRule>
+ * @extends RuleTestCase<CompositeRule>
  */
 class DataProviderDataRuleTest extends RuleTestCase
 {
@@ -27,12 +27,8 @@ protected function getRule(): Rule
 	{
 		$reflectionProvider = $this->createReflectionProvider();
 
-		$ruleLevelHelper = new RuleLevelHelper($reflectionProvider, true, false, true, true, false, false, true);
-
-
 		return new CompositeRule(new DirectRegistry([
 			new DataProviderDataRule(
-				$reflectionProvider,
 				new TestMethodsHelper($reflectionProvider, self::getContainer()->getByType(FileTypeMapper::class), self::getContainer()->getService('defaultAnalysisParser'))
 			),
 			self::getContainer()->getByType(CallMethodsRule::class)