[PhpUnitBridge] Skip internal classes in CoverageListenerTrait

sanmai · nicolas-grekas · commit a0dedb9aa69a · 2020-09-08T09:26:14.000+02:00
diff --git a/src/Symfony/Bridge/PhpUnit/Legacy/CoverageListenerTrait.php b/src/Symfony/Bridge/PhpUnit/Legacy/CoverageListenerTrait.php
@@ -107,6 +107,13 @@ private function addCoversForDocBlockInsideRegistry($test, $covers)
         $symbolAnnotations = new \ReflectionProperty($docBlock, 'symbolAnnotations');
         $symbolAnnotations->setAccessible(true);
 
+        // Exclude internal classes; PHPUnit 9.1+ is picky about tests covering, say, a \RuntimeException
+        $covers = array_filter($covers, function ($class) {
+            $reflector = new ReflectionClass($class);
+
+            return $reflector->isUserDefined();
+        });
+
         $symbolAnnotations->setValue($docBlock, array_replace($docBlock->symbolAnnotations(), [
             'covers' => $covers,
         ]));
