[CodeQuality] Skip first class callable on DecorateWillReturnMapWithExpectsMockRector (#654)

* [CodeQuality] Skip first class callable on DecorateWillReturnMapWithExpectsMockRector

* Fix

* Fix

Author: samsonasik

rules-tests/CodeQuality/Rector/Expression/DecorateWillReturnMapWithExpectsMockRector/Fixture/skip_first_class_callable.php.inc

@@ -0,0 +1,16 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Expression\DecorateWillReturnMapWithExpectsMockRector\Fixture;
+
+final class SkipFirstClassCallable extends \PHPUnit\Framework\TestCase
+{
+    public function test()
+    {
+        $someMock = $this->createMock(\stdClass::class);
+        $someMock->method('some')->willReturnMap(...);
+    }
+}
+
+?>

rules/CodeQuality/Rector/Expression/DecorateWillReturnMapWithExpectsMockRector.php

@@ -108,8 +108,16 @@ public function refactor(Node $node)
             return null;
         }
 
+        if ($methodCall->isFirstClassCallable()) {
+            return null;
+        }
+
         // count values in will map arg
-        $willReturnMapArg = $methodCall->getArgs()[0];
+        $willReturnMapArg = $methodCall->getArgs()[0] ?? null;
+        if (! $willReturnMapArg instanceof Arg) {
+            return null;
+        }
+
         if (! $willReturnMapArg->value instanceof Array_) {
             return null;
         }