[CodeQuality] Skip loose compare string on AssertEqualsToSameRector (#435)

Author: samsonasik

rules-tests/CodeQuality/Rector/MethodCall/AssertEqualsToSameRector/Fixture/skip_loose_compare_string.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\AssertEqualsToSameRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipLooseCompareString extends TestCase
+{
+    public function test()
+    {
+        $this->assertEquals(1, '1');
+    }
+}

rules/CodeQuality/Rector/MethodCall/AssertEqualsToSameRector.php

@@ -108,11 +108,11 @@ public function refactor(Node $node): ?Node
             $secondArgType = TypeCombinator::removeNull($this->nodeTypeResolver->getNativeType($args[1]->value));
 
             // loose comparison
-            if ($firstArgType instanceof IntegerType && $secondArgType instanceof FloatType) {
+            if ($firstArgType instanceof IntegerType && ($secondArgType instanceof FloatType || $secondArgType instanceof StringType)) {
                 return null;
             }
 
-            if ($firstArgType instanceof FloatType && $secondArgType instanceof IntegerType) {
+            if ($firstArgType instanceof FloatType && ($secondArgType instanceof IntegerType || $secondArgType instanceof StringType)) {
                 return null;
             }