Fix double assign

* [PHPUnit 12] Skip first class callable on CreateStubOverCreateMockArgRector

* fix

Author: samsonasik

rules-tests/CodeQuality/Rector/ClassMethod/BareCreateMockAssignToDirectUseRector/Fixture/hop_around_in_call.php.inc

@@ -0,0 +1,35 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\ClassMethod\BareCreateMockAssignToDirectUseRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+use Rector\PHPUnit\Tests\CodeQuality\Rector\ClassMethod\BareCreateMockAssignToDirectUseRector\Source\AnotherClass;
+
+final class HopAroundInCall extends TestCase
+{
+    public function test()
+    {
+        $someMock = $this->createMock(AnotherClass::class);
+
+        $anotherClass = new AnotherClass(1, 2, 3, $someMock);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\ClassMethod\BareCreateMockAssignToDirectUseRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+use Rector\PHPUnit\Tests\CodeQuality\Rector\ClassMethod\BareCreateMockAssignToDirectUseRector\Source\AnotherClass;
+
+final class HopAroundInCall extends TestCase
+{
+    public function test()
+    {
+        $anotherClass = new AnotherClass(1, 2, 3, $this->createMock(AnotherClass::class));
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/ClassMethod/BareCreateMockAssignToDirectUseRector/Fixture/multiple_uses_as_no_connection.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\ClassMethod\BareCreateMockAssignToDirectUseRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class MultipleUsesAsNoConnection extends TestCase
+{
+    public function test()
+    {
+        $someMock = $this->createMock(\Rector\PHPUnit\Tests\CodeQuality\Rector\ClassMethod\BareCreateMockAssignToDirectUseRector\Source\AnotherClass::class);
+
+        $this->useMock($someMock);
+        $this->useMockAgain($someMock);
+    }
+
+    private function useMock($someMock)
+    {
+    }
+
+    private function useMockAgain($someMock)
+    {
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/ClassMethod/BareCreateMockAssignToDirectUseRector/Source/AnotherClass.php

@@ -6,4 +6,8 @@
 
 final class AnotherClass
 {
+    public function __construct(...$various)
+    {
+
+    }
 }

rules-tests/PHPUnit120/Rector/CallLike/CreateStubOverCreateMockArgRector/Fixture/skip_first_class_callable.php.inc

@@ -0,0 +1,14 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\PHPUnit120\Rector\CallLike\CreateStubOverCreateMockArgRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+use Utils\PHPStan\Tests\Rule\PHPUnit\StubOverMockArgRule\Source\AnyClass;
+
+class SkipFirstClassCallable extends TestCase
+{
+    public function testThat()
+    {
+        AnyClass::staticRun(...);
+    }
+}

rules/CodeQuality/Rector/ClassMethod/BareCreateMockAssignToDirectUseRector.php

@@ -128,23 +128,20 @@ public function refactor(Node $node): ?Node
                     $assign = $stmt->expr;
 
                     $instanceArg = $this->assignedMocksCollector->matchCreateMockArgAssignedToVariable($assign);
-                    if (! $instanceArg instanceof Arg) {
+                    if ($instanceArg instanceof Arg && $assign->var instanceof Variable && $this->isName(
+                        $assign->var,
+                        $variableName
+                    )) {
+                        // 1. remove assign
+                        unset($node->stmts[$key]);
+                        $hasChanged = true;
+                        $variablesToMethodCalls[$variableName] = $assign->expr;
                         continue;
                     }
+                }
 
-                    if (! $assign->var instanceof Variable) {
-                        continue;
-                    }
-
-                    if (! $this->isName($assign->var, $variableName)) {
-                        continue;
-                    }
-
-                    // 1. remove assign
-                    unset($node->stmts[$key]);
-                    $hasChanged = true;
-
-                    $variablesToMethodCalls[$variableName] = $assign->expr;
+                // nothing to processy yet
+                if ($variablesToMethodCalls === []) {
                     continue;
                 }
 

rules/PHPUnit120/Rector/CallLike/CreateStubOverCreateMockArgRector.php

@@ -105,6 +105,10 @@ public function refactor(Node $node): MethodCall|StaticCall|New_|ArrayItem|null
 
         $hasChanges = false;
 
+        if ($node->isFirstClassCallable()) {
+            return null;
+        }
+
         foreach ($node->getArgs() as $arg) {
             if (! $arg->value instanceof MethodCall) {
                 continue;