Skip setup/teardown methods in no protected rule

Author: TomasVotruba

src/Rules/Explicit/NoProtectedClassStmtRule.php

@@ -84,7 +84,7 @@ private function shouldSkipClassMethod(ClassMethod|ClassConst|Property $classStm
 
         // PHPUnit test methods
         if (in_array($classStmt->name->toString(), [MethodName::SET_UP, MethodName::TEAR_DOWN])) {
-            return false;
+            return true;
         }
 
         return $this->isParentMethodOverride($classStmt, $scope);

tests/Rules/Explicit/NoProtectedClassStmtRule/Fixture/SkipPHPUnitTetsCase.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace Symplify\PHPStanRules\Tests\Rules\Explicit\NoProtectedClassStmtRule\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipPHPUnitTetsCase extends TestCase
+{
+    protected function setUp(): void
+    {
+        $value = 100;
+    }
+}

tests/Rules/Explicit/NoProtectedClassStmtRule/NoProtectedClassStmtRuleTest.php

@@ -24,6 +24,7 @@ public static function provideData(): Iterator
             [NoProtectedClassStmtRule::ERROR_MESSAGE, 7],
         ]];
 
+        yield [__DIR__ . '/Fixture/SkipPHPUnitTetsCase.php', []];
         yield [__DIR__ . '/Fixture/SkipParentRequired.php', []];
         yield [__DIR__ . '/Fixture/SkipAbstractWithProtected.php', []];
     }