diff --git a/rules-tests/PHPUnit90/Rector/MethodCall/SpecificAssertContainsWithoutIdentityRector/Fixture/skip_no_second_arg.php.inc b/rules-tests/PHPUnit90/Rector/MethodCall/SpecificAssertContainsWithoutIdentityRector/Fixture/skip_no_second_arg.php.inc new file mode 100644 index 00000000..367af616 --- /dev/null +++ b/rules-tests/PHPUnit90/Rector/MethodCall/SpecificAssertContainsWithoutIdentityRector/Fixture/skip_no_second_arg.php.inc @@ -0,0 +1,15 @@ +transport()->queue()->assertContains(new stdClass()); + } +} \ No newline at end of file diff --git a/rules/PHPUnit90/Rector/MethodCall/SpecificAssertContainsWithoutIdentityRector.php b/rules/PHPUnit90/Rector/MethodCall/SpecificAssertContainsWithoutIdentityRector.php index 0e6cf40d..66a220cc 100644 --- a/rules/PHPUnit90/Rector/MethodCall/SpecificAssertContainsWithoutIdentityRector.php +++ b/rules/PHPUnit90/Rector/MethodCall/SpecificAssertContainsWithoutIdentityRector.php @@ -92,6 +92,10 @@ public function refactor(Node $node): ?Node return null; } + if (count($node->getArgs()) < 2) { + return null; + } + // when second argument is string: do nothing $secondArgType = $this->getType($node->getArgs()[1]->value); if ($secondArgType instanceof StringType) {