[CodeQuality] Skip on non-mock object on DecorateWillReturnMapWithExpectsMockRector (#655)

* [CodeQuality] Skip on non-mock object on DecorateWillReturnMapWithExpectsMockRector

* fix

Author: samsonasik

rules-tests/CodeQuality/Rector/Expression/DecorateWillReturnMapWithExpectsMockRector/Fixture/skip_on_create_stub.php.inc

@@ -0,0 +1,24 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Expression\DecorateWillReturnMapWithExpectsMockRector\Fixture;
+
+use PHPUnit\Framework\MockObject\Stub;
+use Rector\PHPUnit\Tests\CodeQuality\Rector\Expression\DecorateWillReturnMapWithExpectsMockRector\Source\SomeInterfaceToStub;
+
+final class SkipOnCreateStub extends \PHPUnit\Framework\TestCase
+{
+    private SomeInterfaceToStub&Stub $service;
+
+    protected function setUp(): void
+    {
+        $this->service = $this->createStub(SomeInterfaceToStub::class);
+    }
+
+    public function test()
+    {
+        $this->service->method('execute')->willReturnMap(['asdf', 'adsfasdf']);
+        $this->service->execute();
+    }
+}

rules-tests/CodeQuality/Rector/Expression/DecorateWillReturnMapWithExpectsMockRector/Source/SomeInterfaceToStub.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Expression\DecorateWillReturnMapWithExpectsMockRector\Source;
+
+interface SomeInterfaceToStub
+{
+    public function execute(): string;
+}

rules/CodeQuality/Rector/Expression/DecorateWillReturnMapWithExpectsMockRector.php

@@ -12,7 +12,9 @@
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Scalar\Int_;
 use PhpParser\Node\Stmt\Expression;
+use PHPStan\Type\ObjectType;
 use Rector\PHPStan\ScopeFetcher;
+use Rector\PHPUnit\Enum\PHPUnitClassName;
 use Rector\Rector\AbstractRector;
 use Rector\ValueObject\MethodName;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -108,6 +110,10 @@ public function refactor(Node $node)
             return null;
         }
 
+        if (! $this->isObjectType($topmostCall->var, new ObjectType(PHPUnitClassName::MOCK_OBJECT))) {
+            return null;
+        }
+
         if ($methodCall->isFirstClassCallable()) {
             return null;
         }