Do not try to instantiate attribute classes that do not exist

Author: sebastianbergmann

ChangeLog-10.5.md

@@ -7,6 +7,7 @@ All notable changes of the PHPUnit 10.5 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
 
 ## [10.5.34] - 2024-09-13
 

src/Metadata/Parser/AttributeParser.php

@@ -95,6 +95,10 @@ public function forClass(string $className): MetadataCollection
                 continue;
             }
 
+            if (!class_exists($attribute->getName())) {
+                continue;
+            }
+
             $attributeInstance = $attribute->newInstance();
 
             switch ($attribute->getName()) {
@@ -349,6 +353,10 @@ public function forMethod(string $className, string $methodName): MetadataCollec
                 continue;
             }
 
+            if (!class_exists($attribute->getName())) {
+                continue;
+            }
+
             $attributeInstance = $attribute->newInstance();
 
             switch ($attribute->getName()) {

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

@@ -34,6 +34,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;
@@ -926,5 +927,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;
 }