Merge branch '10.5' into 11.3

Author: sebastianbergmann

ChangeLog-11.3.md

@@ -7,6 +7,7 @@ All notable changes of the PHPUnit 11.3 release series are documented in this fi
 ### Fixed
 
 * [#5950](https://github.com/sebastianbergmann/phpunit/pull/5950): TestDox text should not be `trim()`med when it contains `$` character
+* The attribute parser will no longer try to instantiate attribute classes that do not exist
 
 ## [11.3.5] - 2024-09-13
 

src/Metadata/Parser/AnnotationParser.php

@@ -11,6 +11,7 @@
 
 use function array_merge;
 use function assert;
+use function class_exists;
 use function count;
 use function explode;
 use function method_exists;
@@ -60,6 +61,8 @@ final class AnnotationParser implements Parser
      */
     public function forClass(string $className): MetadataCollection
     {
+        assert(class_exists($className));
+
         $result = [];
 
         foreach (AnnotationRegistry::getInstance()->forClassName($className)->symbolAnnotations() as $annotation => $values) {
@@ -206,6 +209,9 @@ public function forClass(string $className): MetadataCollection
      */
     public function forMethod(string $className, string $methodName): MetadataCollection
     {
+        assert(class_exists($className));
+        assert(method_exists($className, $methodName));
+
         $result = [];
 
         foreach (AnnotationRegistry::getInstance()->forMethod($className, $methodName)->symbolAnnotations() as $annotation => $values) {

src/Metadata/Parser/AttributeParser.php

@@ -11,7 +11,9 @@
 
 use const JSON_THROW_ON_ERROR;
 use function assert;
+use function class_exists;
 use function json_decode;
+use function method_exists;
 use function sprintf;
 use function str_starts_with;
 use function strtolower;
@@ -91,13 +93,19 @@
      */
     public function forClass(string $className): MetadataCollection
     {
+        assert(class_exists($className));
+
         $result = [];
 
         foreach ((new ReflectionClass($className))->getAttributes() as $attribute) {
             if (!str_starts_with($attribute->getName(), 'PHPUnit\\Framework\\Attributes\\')) {
                 continue;
             }
 
+            if (!class_exists($attribute->getName())) {
+                continue;
+            }
+
             $attributeInstance = $attribute->newInstance();
 
             switch ($attribute->getName()) {
@@ -369,13 +377,20 @@ public function forClass(string $className): MetadataCollection
      */
     public function forMethod(string $className, string $methodName): MetadataCollection
     {
+        assert(class_exists($className));
+        assert(method_exists($className, $methodName));
+
         $result = [];
 
         foreach ((new ReflectionMethod($className, $methodName))->getAttributes() as $attribute) {
             if (!str_starts_with($attribute->getName(), 'PHPUnit\\Framework\\Attributes\\')) {
                 continue;
             }
 
+            if (!class_exists($attribute->getName())) {
+                continue;
+            }
+
             $attributeInstance = $attribute->newInstance();
 
             switch ($attribute->getName()) {

src/Metadata/Parser/CachingParser.php

@@ -9,6 +9,9 @@
  */
 namespace PHPUnit\Metadata\Parser;
 
+use function assert;
+use function class_exists;
+use function method_exists;
 use PHPUnit\Metadata\MetadataCollection;
 
 /**
@@ -45,6 +48,8 @@ public function __construct(Parser $reader)
      */
     public function forClass(string $className): MetadataCollection
     {
+        assert(class_exists($className));
+
         if (isset($this->classCache[$className])) {
             return $this->classCache[$className];
         }
@@ -60,6 +65,9 @@ public function forClass(string $className): MetadataCollection
      */
     public function forMethod(string $className, string $methodName): MetadataCollection
     {
+        assert(class_exists($className));
+        assert(method_exists($className, $methodName));
+
         $key = $className . '::' . $methodName;
 
         if (isset($this->methodCache[$key])) {

src/Metadata/Parser/ParserChain.php

@@ -9,6 +9,9 @@
  */
 namespace PHPUnit\Metadata\Parser;
 
+use function assert;
+use function class_exists;
+use function method_exists;
 use PHPUnit\Metadata\MetadataCollection;
 
 /**
@@ -32,6 +35,8 @@ public function __construct(Parser $attributeReader, Parser $annotationReader)
      */
     public function forClass(string $className): MetadataCollection
     {
+        assert(class_exists($className));
+
         $metadata = $this->attributeReader->forClass($className);
 
         if (!$metadata->isEmpty()) {
@@ -47,6 +52,9 @@ public function forClass(string $className): MetadataCollection
      */
     public function forMethod(string $className, string $methodName): MetadataCollection
     {
+        assert(class_exists($className));
+        assert(method_exists($className, $methodName));
+
         $metadata = $this->attributeReader->forMethod($className, $methodName);
 
         if (!$metadata->isEmpty()) {

tests/_files/Metadata/Attribute/tests/PhpunitAttributeThatDoesNotExistTest.php

@@ -0,0 +1,22 @@
+<?php declare(strict_types=1);
+/*
+ * This file is part of PHPUnit.
+ *
+ * (c) Sebastian Bergmann <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+namespace PHPUnit\TestFixture\Metadata\Attribute;
+
+use PHPUnit\Framework\Attributes\PhpunitAttributeThatDoesNotExist;
+use PHPUnit\Framework\TestCase;
+
+#[PhpunitAttributeThatDoesNotExist]
+final class PhpunitAttributeThatDoesNotExistTest extends TestCase
+{
+    #[PhpunitAttributeThatDoesNotExist]
+    public function testOne(): void
+    {
+    }
+}

tests/unit/Metadata/Parser/AttributeParserTestCase.php

@@ -37,6 +37,7 @@
 use PHPUnit\TestFixture\Metadata\Attribute\LargeTest;
 use PHPUnit\TestFixture\Metadata\Attribute\MediumTest;
 use PHPUnit\TestFixture\Metadata\Attribute\NonPhpunitAttributeTest;
+use PHPUnit\TestFixture\Metadata\Attribute\PhpunitAttributeThatDoesNotExistTest;
 use PHPUnit\TestFixture\Metadata\Attribute\PreserveGlobalStateTest;
 use PHPUnit\TestFixture\Metadata\Attribute\ProcessIsolationTest;
 use PHPUnit\TestFixture\Metadata\Attribute\RequiresFunctionTest;
@@ -995,5 +996,12 @@ public function test_ignores_attributes_not_owned_by_PHPUnit(): void
         $this->assertTrue($metadata->isEmpty());
     }
 
+    public function test_ignores_attributes_in_PHPUnit_namespace_that_do_not_exist(): void
+    {
+        $metadata = $this->parser()->forClassAndMethod(PhpunitAttributeThatDoesNotExistTest::class, 'testOne');
+
+        $this->assertTrue($metadata->isEmpty());
+    }
+
     abstract protected function parser(): Parser;
 }