Add linting rule against using useless parent calls in unit tests

monorepo/HydeStan/HydeStan.php

@@ -20,6 +20,7 @@ final class HydeStan
     private const TEST_FILE_ANALYSERS = [
         NoFixMeAnalyser::class,
         NoUsingAssertEqualsForScalarTypesTestAnalyser::class,
+        NoParentSetUpTearDownInUnitTestCaseAnalyser::class,
     ];
 
     private const LINE_ANALYSERS = [
@@ -382,3 +383,24 @@ public function run(string $file, int $lineNumber, string $line): void
         }
     }
 }
+
+class NoParentSetUpTearDownInUnitTestCaseAnalyser extends FileAnalyser
+{
+    public function run(string $file, string $contents): void
+    {
+        if (! str_contains($contents, 'extends UnitTestCase')) {
+            return;
+        }
+
+        $methods = ['setUp', 'tearDown'];
+
+        foreach ($methods as $method) {
+            AnalysisStatisticsContainer::analysedExpression();
+            if (str_contains($contents, "parent::$method()")) {
+                $lineNumber = substr_count(substr($contents, 0, strpos($contents, $method)), "\n") + 1;
+                $this->fail(sprintf("Found '%s' method in UnitTestCase at %s", "parent::$method()", fileLink($file, $lineNumber, false)));
+                HydeStan::addActionsMessage('error', $file, $lineNumber, "HydeStan: UnnecessaryParent{$method}MethodError", "{$method} method in UnitTestCase performs no operation and should be removed.");
+            }
+        }
+    }
+}

monorepo/HydeStan/includes/helpers.php

@@ -51,11 +51,14 @@ function check_str_contains_any(array $searches, string $line): bool
     return $strContainsAny;
 }
 
-function fileLink(string $file, ?int $line = null): string
+function fileLink(string $file, ?int $line = null, bool $substr = true): string
 {
     $path = (realpath(__DIR__.'/../../packages/framework/'.$file) ?: $file).($line ? ':'.$line : '');
-    $trim = strlen(getcwd()) + 2;
-    $path = substr($path, $trim);
+
+    if ($substr) {
+        $trim = strlen(getcwd()) + 2;
+        $path = substr($path, $trim);
+    }
 
     return str_replace('\\', '/', $path);
 }