note

Author: TomasVotruba

rules/PHPUnit120/Rector/Class_/AllowMockObjectsWithoutExpectationsAttributeRector.php

@@ -14,7 +14,6 @@
 use PHPStan\Reflection\ReflectionProvider;
 use Rector\Doctrine\NodeAnalyzer\AttributeFinder;
 use Rector\PhpParser\Node\BetterNodeFinder;
-use Rector\PhpParser\NodeFinder\PropertyFetchFinder;
 use Rector\PHPUnit\Enum\PHPUnitAttribute;
 use Rector\PHPUnit\Enum\PHPUnitClassName;
 use Rector\PHPUnit\NodeAnalyzer\TestsNodeAnalyzer;
@@ -97,7 +96,7 @@ public function refactor(Node $node): ?Class_
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition(
-            'Add #[AllowMockObjectsWithoutExpectations] attribute to PHPUnit test classes with mock properties used in multiple methods',
+            'Add #[AllowMockObjectsWithoutExpectations] attribute to PHPUnit test classes with mock properties used in multiple methods but one, to avoid irrelevant notices in tests run',
             [
                 new CodeSample(
                     <<<'CODE_SAMPLE'
@@ -202,12 +201,14 @@ private function shouldSkipClass(Class_ $class): bool
         return ! $setupClassMethod instanceof ClassMethod;
     }
 
-    private function isClassMethodUsingMethodCallOnPropertyNamed(ClassMethod $classMethod, string $mockObjectPropertyName): bool
-    {
+    private function isClassMethodUsingMethodCallOnPropertyNamed(
+        ClassMethod $classMethod,
+        string $mockObjectPropertyName
+    ): bool {
         /** @var Node\Expr\MethodCall[] $methodCalls */
         $methodCalls = $this->betterNodeFinder->findInstancesOfScoped([$classMethod], [Node\Expr\MethodCall::class]);
         foreach ($methodCalls as $methodCall) {
-            if (!$methodCall->var instanceof Node\Expr\PropertyFetch) {
+            if (! $methodCall->var instanceof Node\Expr\PropertyFetch) {
                 continue;
             }