Is used in closure (#614)

Author: TomasVotruba

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

@@ -0,0 +1,20 @@
+<?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 SkipIfUse extends TestCase
+{
+    public function test()
+    {
+        $someMock = $this->createMock(AnotherClass::class);
+
+        $someClosure = function() use ($someMock) {
+            // some logic here
+
+            return $someMock;
+        };
+    }
+}

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

@@ -6,6 +6,7 @@
 
 use PhpParser\Node;
 use PhpParser\Node\Arg;
+use PhpParser\Node\ClosureUse;
 use PhpParser\Node\Expr\Assign;
 use PhpParser\Node\Expr\CallLike;
 use PhpParser\Node\Expr\MethodCall;
@@ -120,6 +121,10 @@ public function refactor(Node $node): ?Node
                 continue;
             }
 
+            if ($this->isUsedInClosure($node, $variableName)) {
+                continue;
+            }
+
             // 1. remove initial assign
             $variablesToMethodCalls = [];
 
@@ -232,4 +237,18 @@ private function findCallLikes(ClassMethod|Foreach_|Stmt $node): array
 
         return array_filter($callLikes, fn (CallLike $callLike): bool => ! $callLike->isFirstClassCallable());
     }
+
+    private function isUsedInClosure(ClassMethod|Foreach_ $stmtsAware, string $variableName): bool
+    {
+        /** @var Node\ClosureUse[] $uses */
+        $uses = $this->betterNodeFinder->findInstancesOf([$stmtsAware], [ClosureUse::class]);
+
+        foreach ($uses as $use) {
+            if ($this->isName($use->var, $variableName)) {
+                return true;
+            }
+        }
+
+        return false;
+    }
 }