Fix double assign

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/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;
                 }