From f452fac4d5bf9cac39b827728d19dbb4f96ff4ea Mon Sep 17 00:00:00 2001
From: Tomas Votruba <tomas.vot@gmail.com>
Date: Sun, 19 Jan 2025 18:34:36 +0000
Subject: [PATCH] Add non-testcase assert call in FlipAssertRector

---
 .../AssertEqualsToSameRector/Fixture/skip.php | 19 -----------
 .../cover_external_static_call.php.inc        | 33 +++++++++++++++++++
 src/Enum/PHPUnitClassName.php                 | 18 ++++++++++
 src/NodeAnalyzer/TestsNodeAnalyzer.php        | 20 +++++++++--
 4 files changed, 68 insertions(+), 22 deletions(-)
 delete mode 100644 rules-tests/CodeQuality/Rector/MethodCall/AssertEqualsToSameRector/Fixture/skip.php
 create mode 100644 rules-tests/CodeQuality/Rector/MethodCall/FlipAssertRector/Fixture/cover_external_static_call.php.inc
 create mode 100644 src/Enum/PHPUnitClassName.php

diff --git a/rules-tests/CodeQuality/Rector/MethodCall/AssertEqualsToSameRector/Fixture/skip.php b/rules-tests/CodeQuality/Rector/MethodCall/AssertEqualsToSameRector/Fixture/skip.php
deleted file mode 100644
index 1a7b0b71..00000000
--- a/rules-tests/CodeQuality/Rector/MethodCall/AssertEqualsToSameRector/Fixture/skip.php
+++ /dev/null
@@ -1,19 +0,0 @@
-<?php
-
-namespace Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\AssertEqualsToSameRector\Fixture;
-
-use PHPUnit\Framework\TestCase;
-
-final class MySkipTest extends TestCase
-{
-    public function test()
-    {
-        $null = null;
-        $expectedNull = null;
-        $this->assertEquals($expectedNull, $null);
-
-        $bool = true;
-        $expectedBool = true;
-        $this->assertEquals($expectedBool, $bool);
-    }
-}
diff --git a/rules-tests/CodeQuality/Rector/MethodCall/FlipAssertRector/Fixture/cover_external_static_call.php.inc b/rules-tests/CodeQuality/Rector/MethodCall/FlipAssertRector/Fixture/cover_external_static_call.php.inc
new file mode 100644
index 00000000..b46bac6e
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/MethodCall/FlipAssertRector/Fixture/cover_external_static_call.php.inc
@@ -0,0 +1,33 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\FlipAssertRector\Fixture;
+
+use PHPUnit\Framework\Assert;
+
+final class CovertExternalStaticCall
+{
+    public function test()
+    {
+        $result = '...';
+        Assert::assertStringContainsString($result, 'expected');
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\FlipAssertRector\Fixture;
+
+use PHPUnit\Framework\Assert;
+
+final class CovertExternalStaticCall
+{
+    public function test()
+    {
+        $result = '...';
+        Assert::assertStringContainsString('expected', $result);
+    }
+}
+
+?>
diff --git a/src/Enum/PHPUnitClassName.php b/src/Enum/PHPUnitClassName.php
new file mode 100644
index 00000000..51708c6a
--- /dev/null
+++ b/src/Enum/PHPUnitClassName.php
@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\PHPUnit\Enum;
+
+final class PHPUnitClassName
+{
+    /**
+     * @var string
+     */
+    public const TEST_CASE = 'PHPUnit\Framework\TestCase';
+
+    /**
+     * @var string
+     */
+    public const ASSERT = 'PHPUnit\Framework\Assert';
+}
diff --git a/src/NodeAnalyzer/TestsNodeAnalyzer.php b/src/NodeAnalyzer/TestsNodeAnalyzer.php
index 2c48b3e8..a36d1e6e 100644
--- a/src/NodeAnalyzer/TestsNodeAnalyzer.php
+++ b/src/NodeAnalyzer/TestsNodeAnalyzer.php
@@ -13,6 +13,7 @@
 use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfoFactory;
 use Rector\NodeNameResolver\NodeNameResolver;
 use Rector\NodeTypeResolver\NodeTypeResolver;
+use Rector\PHPUnit\Enum\PHPUnitClassName;
 use Rector\Reflection\ReflectionResolver;
 
 final readonly class TestsNodeAnalyzer
@@ -96,10 +97,23 @@ public function isPHPUnitMethodCallNames(Node $node, array $names): bool
 
     public function isPHPUnitTestCaseCall(Node $node): bool
     {
-        if (! $this->isInTestClass($node)) {
-            return false;
+        if ($node instanceof MethodCall) {
+            return $this->isInTestClass($node);
+        }
+
+        if ($node instanceof StaticCall) {
+            $classType = $this->nodeTypeResolver->getType($node->class);
+            if ($classType instanceof ObjectType) {
+                if ($classType->isInstanceOf(PHPUnitClassName::TEST_CASE)->yes()) {
+                    return true;
+                }
+
+                if ($classType->isInstanceOf(PHPUnitClassName::ASSERT)->yes()) {
+                    return true;
+                }
+            }
         }
 
-        return $node instanceof MethodCall || $node instanceof StaticCall;
+        return false;
     }
 }