exclude test case setup from override attribute, as not helpful

Author: TomasVotruba

rules-tests/Php83/Rector/ClassMethod/AddOverrideAttributeToOverriddenMethodsRector/Fixture/skip_setup_phpunit_as_clutter.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Tests\Php83\Rector\ClassMethod\AddOverrideAttributeToOverriddenMethodsRector\Fixture;
+
+use Rector\Tests\Php83\Rector\ClassMethod\AddOverrideAttributeToOverriddenMethodsRector\Source\SomeAbstractTest;
+
+final class SkipSetupPHPUnitAsClutter extends SomeAbstractTest
+{
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $someValue = 1;
+    }
+}

rules-tests/Php83/Rector/ClassMethod/AddOverrideAttributeToOverriddenMethodsRector/Source/SomeAbstractTest.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\Php83\Rector\ClassMethod\AddOverrideAttributeToOverriddenMethodsRector\Source;
+
+use PHPUnit\Framework\TestCase;
+
+abstract class SomeAbstractTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        $value = 1000;
+    }
+}

rules/DeadCode/NodeAnalyzer/ParentClassAnalyzer.php

@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\DeadCode\NodeAnalyzer;
+
+use PhpParser\Node\Stmt\ClassMethod;
+use Rector\NodeNameResolver\NodeNameResolver;
+
+final class ParentClassAnalyzer
+{
+    public function __construct(
+        private readonly NodeNameResolver $nodeNameResolver,
+    ) {
+
+    }
+
+    public function hasParentCall(ClassMethod $classMethod): bool
+    {
+        if ($classMethod->isAbstract()) {
+            return false;
+        }
+
+        if ($classMethod->isPrivate()) {
+            return false;
+        }
+
+        $classMethodName = $classMethod->name->name;
+
+        foreach ($classMethod->stmts as $stmt) {
+            if (! $stmt instanceof \PhpParser\Node\Stmt\Expression) {
+                continue;
+            }
+
+            $expr = $stmt->expr;
+            if (! $expr instanceof \PhpParser\Node\Expr\StaticCall) {
+                continue;
+            }
+
+            if (! $this->nodeNameResolver->isName($expr->class, 'parent')) {
+                continue;
+            }
+
+            if ($this->nodeNameResolver->isName($expr->name, $classMethodName)) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+}

rules/Php83/Rector/ClassMethod/AddOverrideAttributeToOverriddenMethodsRector.php

@@ -19,6 +19,7 @@
 use PHPStan\Reflection\ClassReflection;
 use PHPStan\Reflection\ReflectionProvider;
 use Rector\Contract\Rector\ConfigurableRectorInterface;
+use Rector\DeadCode\NodeAnalyzer\ParentClassAnalyzer;
 use Rector\NodeAnalyzer\ClassAnalyzer;
 use Rector\Php80\NodeAnalyzer\PhpAttributeAnalyzer;
 use Rector\PhpParser\AstResolver;
@@ -54,6 +55,7 @@ public function __construct(
         private readonly PhpAttributeAnalyzer $phpAttributeAnalyzer,
         private readonly AstResolver $astResolver,
         private readonly ValueResolver $valueResolver,
+        private readonly ParentClassAnalyzer $parentClassAnalyzer,
     ) {
     }
 
@@ -232,12 +234,24 @@ private function shouldSkipClassMethod(ClassMethod $classMethod): bool
             return true;
         }
 
+        // nothing to override
         if ($classMethod->isPrivate()) {
             return true;
         }
 
         // ignore if it already uses the attribute
-        return $this->phpAttributeAnalyzer->hasPhpAttribute($classMethod, self::OVERRIDE_CLASS);
+        if ($this->phpAttributeAnalyzer->hasPhpAttribute($classMethod, self::OVERRIDE_CLASS)) {
+            return true;
+        }
+
+        // skip test setup method override, as rather clutters the code than helps
+        if ($this->isName($classMethod, 'setUp')) {
+            if ($this->parentClassAnalyzer->hasParentCall($classMethod)) {
+                return true;
+            }
+        }
+
+        return false;
     }
 
     private function shouldSkipParentClassMethod(ClassReflection $parentClassReflection, ClassMethod $classMethod): bool