Ignore inherited PHPUnit suite factories

Author: chubes4

packages/runtime-playground/src/phpunit-command-handlers.ts

@@ -352,8 +352,11 @@ function ${functionName}(array $argv) {
 
 function phpunitClassHasTestsPhp(functionName: string): string {
   return `function ${functionName}(ReflectionClass $class): bool {
-    if ($class->hasMethod('suite') && $class->getMethod('suite')->isStatic()) {
-        return true;
+    if ($class->hasMethod('suite')) {
+        $suite_method = $class->getMethod('suite');
+        if ($suite_method->isStatic() && $suite_method->getDeclaringClass()->getName() === $class->getName()) {
+            return true;
+        }
     }
     foreach ((new PHPUnit\\Util\\Reflection())->publicMethodsInTestClass($class) as $method) {
         if (PHPUnit\\Util\\Test::isTestMethod($method)) {

tests/phpunit-project-autoload.test.ts

@@ -325,10 +325,13 @@ function assertDiscoveredTestExecutes(source: string, stagePrefix: "pg" | "core_
     : `public function __construct($name) { $this->name = $name; }`
 
   writeFileSync(testFile, `<?php
-class ConcreteHelper extends PHPUnit\\Framework\\TestCase {
+abstract class ProjectTestCase extends PHPUnit\\Framework\\TestCase {
+    public static function suite($class) {}
+}
+class ConcreteHelper extends ProjectTestCase {
     public function helperMethod(): void {}
 }
-class DiscoveredTest extends PHPUnit\\Framework\\TestCase {
+class DiscoveredTest extends ProjectTestCase {
     public function testDiscovered(): void {}
     public function run(): void {
         file_put_contents(getenv('EXECUTION_MARKER'), 'executed');