skip property fetch (#624)

Author: TomasVotruba

rules-tests/PHPUnit120/Rector/MethodCall/ExplicitMockExpectsCallRector/Fixture/skip_property_as_can_be_defined_multiple_times.php.inc

@@ -0,0 +1,21 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\PHPUnit120\Rector\MethodCall\ExplicitMockExpectsCallRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipPropertyAsCanBeDefinedMultipleTimes extends TestCase
+{
+
+    private \PHPUnit\Framework\MockObject\MockObject $someClass;
+
+    protected function setUp(): void
+    {
+        $this->someClass = $this->createMock(\stdClass::class);
+    }
+
+    public function test(): void
+    {
+        $this->someClass->method('some');
+    }
+}

rules/PHPUnit120/Rector/MethodCall/ExplicitMockExpectsCallRector.php

@@ -7,7 +7,6 @@
 use PhpParser\Node;
 use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\MethodCall;
-use PhpParser\Node\Expr\PropertyFetch;
 use PhpParser\Node\Expr\Variable;
 use PHPStan\Type\ObjectType;
 use Rector\PHPStan\ScopeFetcher;
@@ -87,7 +86,7 @@ public function refactor(Node $node): Node|null
             return null;
         }
 
-        if (! $node->var instanceof Variable && ! $node->var instanceof PropertyFetch) {
+        if (! $node->var instanceof Variable) {
             return null;
         }