[PHPUnit90] Handle crash on different object without second arg on SpecificAssertContainsWithoutIdentityRector (#474)

* [PHPUnit90] Handle crash on different object without second arg on SpecificAssertContainsWithoutIdentityRector

* [PHPUnit90] Handle crash on different object without second arg on SpecificAssertContainsWithoutIdentityRector

* [ci-review] Rector Rectify

* [PHPUnit90] Handle crash on different object without second arg on SpecificAssertContainsWithoutIdentityRector

* [PHPUnit90] Handle crash on different object without second arg on SpecificAssertContainsWithoutIdentityRector

* Fix

* [ci-review] Rector Rectify

* Fix

---------

Co-authored-by: GitHub Action <[email protected]>

Author: samsonasik

rules-tests/PHPUnit90/Rector/MethodCall/SpecificAssertContainsWithoutIdentityRector/Fixture/skip_no_second_arg.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\PHPUnit90\Rector\MethodCall\SpecificAssertContainsWithoutIdentityRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+use stdClass;
+
+final class SkipNoSecondArg extends TestCase
+{
+    public function test()
+    {
+        $objects = [ new stdClass(), new stdClass(), new stdClass() ];
+        $this->transport()->queue()->assertContains(new stdClass());
+    }
+}

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) {