Migrate phpunit metadata to attributes

.github/workflows/php.yml

@@ -78,7 +78,7 @@ jobs:
         run: composer install --no-progress --prefer-dist --optimize-autoloader
 
       - name: Check code for hard dependencies missing in composer.json
-        run: composer-require-checker check composer.json
+        run: composer-require-checker check --config-file=tools/composer-require-checker.json composer.json
 
       - name: Check code for unused dependencies in composer.json
         run: composer-unused

composer.json

@@ -37,7 +37,7 @@
         "simplesamlphp/assert": "^1.1"
     },
     "require-dev": {
-        "simplesamlphp/simplesamlphp-test-framework": "^1.5"
+        "simplesamlphp/simplesamlphp-test-framework": "^1.6"
     },
     "support": {
         "issues": "https://github.com/simplesamlphp/xml-common/issues",

src/TestUtils/SchemaValidationTestTrait.php

@@ -7,6 +7,7 @@
 use DOMDocument;
 use Exception;
 use LibXMLError; // Officially spelled with a lower-case `l`, but that breaks composer-require-checker
+use PHPUnit\Framework\Attributes\Depends;
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\XML\Exception\SchemaViolationException;
 use XMLReader;
@@ -37,9 +38,8 @@ trait SchemaValidationTestTrait
 
     /**
      * Test schema validation.
-     *
-     * @depends testSerialization
      */
+    #[Depends('testSerialization')]
     public function testSchemaValidation(): void
     {
         if (!class_exists(self::$testedClass)) {

src/TestUtils/SerializableElementTestTrait.php

@@ -5,6 +5,7 @@
 namespace SimpleSAML\XML\TestUtils;
 
 use DOMDocument;
+use PHPUnit\Framework\Attributes\Depends;
 
 use function class_exists;
 
@@ -56,10 +57,9 @@ public function testUnmarshalling(): void
 
     /**
      * Test serialization / unserialization.
-     *
-     * @depends testMarshalling
-     * @depends testUnmarshalling
      */
+    #[Depends('testMarshalling')]
+    #[Depends('testUnmarshalling')]
     public function testSerialization(): void
     {
         if (!class_exists(self::$testedClass)) {

tests/Utils/RandomTest.php

@@ -4,16 +4,16 @@
 
 namespace SimpleSAML\Test\XML\Utils;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\XML\Utils\Random;
 
 use function strlen;
 
 /**
  * Tests for SimpleSAML\XML\Utils\Random.
- *
- * @covers \SimpleSAML\XML\Utils\Random
  */
+#[CoversClass(Random::class)]
 class RandomTest extends TestCase
 {
     /**

tests/XML/AbstractElementTest.php

@@ -4,8 +4,10 @@
 
 namespace SimpleSAML\Test\XML;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\Test\XML\Element;
+use SimpleSAML\XML\AbstractElement;
 use SimpleSAML\XML\DOMDocumentFactory;
 use SimpleSAML\XML\Exception\MissingAttributeException;
 use SimpleSAML\XML\TestUtils\SerializableElementTestTrait;
@@ -16,11 +18,10 @@
 /**
  * Class \SimpleSAML\XML\AbstractElementTest
  *
- * @covers \SimpleSAML\XML\TestUtils\SerializableElementTestTrait
- * @covers \SimpleSAML\XML\AbstractElement
- *
  * @package simplesamlphp\xml-common
  */
+#[CoversClass(SerializableElementTestTrait::class)]
+#[CoversClass(AbstractElement::class)]
 final class AbstractElementTest extends TestCase
 {
     use SerializableElementTestTrait;

tests/XML/AttributeTest.php

@@ -5,6 +5,7 @@
 namespace SimpleSAML\Test\XML;
 
 use DOMDocument;
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\XML\Attribute;
 use SimpleSAML\XML\DOMDocumentFactory;
@@ -13,11 +14,10 @@
 /**
  * Class \SimpleSAML\XML\AttributeTest
  *
- * @covers \SimpleSAML\XML\Attribute
- * @covers \SimpleSAML\XML\TestUtils\ArrayizableElementTestTrait
- *
  * @package simplesamlphp\xml-common
  */
+#[CoversClass(ArrayizableElementTestTrait::class)]
+#[CoversClass(Attribute::class)]
 final class AttributeTest extends TestCase
 {
     use ArrayizableElementTestTrait;

tests/XML/Base64ElementTraitTest.php

@@ -4,9 +4,14 @@
 
 namespace SimpleSAML\Test\XML;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\Test\XML\XMLDumper;
+use SimpleSAML\XML\AbstractElement;
+use SimpleSAML\XML\Base64ElementTrait;
 use SimpleSAML\XML\DOMDocumentFactory;
+use SimpleSAML\XML\StringElementTrait;
 use SimpleSAML\XML\TestUtils\SerializableElementTestTrait;
 
 use function dirname;
@@ -15,13 +20,12 @@
 /**
  * Class \SimpleSAML\XML\Base64ElementTraitTest
  *
- * @covers \SimpleSAML\XML\TestUtils\SerializableElementTestTrait
- * @covers \SimpleSAML\XML\Base64ElementTrait
- * @covers \SimpleSAML\XML\StringElementTrait
- * @covers \SimpleSAML\XML\AbstractElement
- *
  * @package simplesamlphp\xml-common
  */
+#[CoversClass(SerializableElementTestTrait::class)]
+#[CoversClass(Base64ElementTrait::class)]
+#[CoversClass(StringElementTrait::class)]
+#[CoversClass(AbstractElement::class)]
 final class Base64ElementTraitTest extends TestCase
 {
     use SerializableElementTestTrait;
@@ -65,8 +69,8 @@ public function testUnmarshalling(): void
 
     /**
      * @param non-empty-string $xml
-     * @dataProvider provideBase64Cases
      */
+    #[DataProvider('provideBase64Cases')]
     public function testBase64Cases(string $xml): void
     {
         $xmlRepresentation = DOMDocumentFactory::fromString($xml);

tests/XML/BooleanElementTraitTest.php

@@ -4,8 +4,11 @@
 
 namespace SimpleSAML\Test\XML;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\Test\XML\BooleanElement;
+use SimpleSAML\XML\AbstractElement;
+use SimpleSAML\XML\BooleanElementTrait;
 use SimpleSAML\XML\DOMDocumentFactory;
 use SimpleSAML\XML\TestUtils\SerializableElementTestTrait;
 
@@ -15,12 +18,11 @@
 /**
  * Class \SimpleSAML\XML\BooleanElementTraitTest
  *
- * @covers \SimpleSAML\XML\TestUtils\SerializableElementTestTrait
- * @covers \SimpleSAML\XML\BooleanElementTrait
- * @covers \SimpleSAML\XML\AbstractElement
- *
  * @package simplesamlphp\xml-common
  */
+#[CoversClass(SerializableElementTestTrait::class)]
+#[CoversClass(BooleanElementTrait::class)]
+#[CoversClass(AbstractElement::class)]
 final class BooleanElementTraitTest extends TestCase
 {
     use SerializableElementTestTrait;

tests/XML/ChunkTest.php

@@ -4,6 +4,7 @@
 
 namespace SimpleSAML\Test\XML;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\XML\Chunk;
 use SimpleSAML\XML\DOMDocumentFactory;
@@ -16,11 +17,10 @@
 /**
  * Class \SimpleSAML\XML\ChunkTest
  *
- * @covers \SimpleSAML\XML\TestUtils\SerializableElementTestTrait
- * @covers \SimpleSAML\XML\Chunk
- *
  * @package simplesamlphp\xml-common
  */
+#[CoversClass(SerializableElementTestTrait::class)]
+#[CoversClass(Chunk::class)]
 final class ChunkTest extends TestCase
 {
     use SerializableElementTestTrait;

tests/XML/DOMDocumentFactoryTest.php

@@ -4,6 +4,8 @@
 
 namespace SimpleSAML\Test\XML;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\TestCase;
 use RuntimeException;
 use SimpleSAML\Assert\AssertionFailedException;
@@ -13,24 +15,19 @@
 use function strval;
 
 /**
- * @covers \SimpleSAML\XML\DOMDocumentFactory
  * @package simplesamlphp\xml-common
  */
+#[CoversClass(DOMDocumentFactory::class)]
+#[Group('domdocument')]
 final class DOMDocumentFactoryTest extends TestCase
 {
-    /**
-     * @group domdocument
-     */
     public function testNotXmlStringRaisesAnException(): void
     {
         $this->expectException(UnparseableXMLException::class);
         DOMDocumentFactory::fromString('this is not xml');
     }
 
 
-    /**
-     * @group domdocument
-     */
     public function testXmlStringIsCorrectlyLoaded(): void
     {
         $xml = '<root/>';
@@ -41,8 +38,6 @@ public function testXmlStringIsCorrectlyLoaded(): void
     }
 
 
-    /**
-     */
     public function testFileThatDoesNotExistIsNotAccepted(): void
     {
         $this->expectException(RuntimeException::class);
@@ -51,19 +46,13 @@ public function testFileThatDoesNotExistIsNotAccepted(): void
     }
 
 
-    /**
-     * @group domdocument
-     */
     public function testFileThatDoesNotContainXMLCannotBeLoaded(): void
     {
         $this->expectException(RuntimeException::class);
         DOMDocumentFactory::fromFile('resources/xml/domdocument_invalid_xml.xml');
     }
 
 
-    /**
-     * @group domdocument
-     */
     public function testFileWithValidXMLCanBeLoaded(): void
     {
         $file = 'resources/xml/domdocument_valid_xml.xml';
@@ -73,9 +62,6 @@ public function testFileWithValidXMLCanBeLoaded(): void
     }
 
 
-    /**
-     * @group domdocument
-     */
     public function testFileThatContainsDocTypeIsNotAccepted(): void
     {
         $file = 'resources/xml/domdocument_doctype.xml';
@@ -87,9 +73,6 @@ public function testFileThatContainsDocTypeIsNotAccepted(): void
     }
 
 
-    /**
-     * @group domdocument
-     */
     public function testStringThatContainsDocTypeIsNotAccepted(): void
     {
         $xml = '<!DOCTYPE foo [<!ELEMENT foo ANY > <!ENTITY xxe SYSTEM "file:///dev/random" >]><foo />';
@@ -101,9 +84,6 @@ public function testStringThatContainsDocTypeIsNotAccepted(): void
     }
 
 
-    /**
-     * @group domdocument
-     */
     public function testEmptyFileIsNotValid(): void
     {
         $file = 'resources/xml/domdocument_empty.xml';
@@ -113,9 +93,6 @@ public function testEmptyFileIsNotValid(): void
     }
 
 
-    /**
-     * @group                    domdocument
-     */
     public function testEmptyStringIsNotValid(): void
     {
         $this->expectException(AssertionFailedException::class);

tests/XML/Exception/IOExceptionTest.php

@@ -4,16 +4,16 @@
 
 namespace SimpleSAML\Test\XML;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\XML\Exception\IOException;
 
 /**
  * Empty shell class for testing IOException.
  *
  * @package simplesaml/xml-security
- *
- * @covers \SimpleSAML\XML\Exception\IOException
  */
+#[CoversClass(IOException::class)]
 final class IOExceptionTest extends TestCase
 {
     /**

tests/XML/Exception/UnparseableXMLExceptionTest.php

@@ -5,16 +5,16 @@
 namespace SimpleSAML\Test\XML;
 
 use LibXMLError;
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\XML\Exception\UnparseableXMLException;
 
 /**
  * Empty shell class for testing UnparseableXMLException.
  *
  * @package simplesaml/xml-security
- *
- * @covers \SimpleSAML\XML\Exception\UnparseableXMLException
  */
+#[CoversClass(UnparseableXMLException::class)]
 final class UnparseableXMLExceptionTest extends TestCase
 {
     /** @var \LibXMLError $libxmlerror */

tests/XML/ExtendableAttributesTest.php

@@ -4,6 +4,7 @@
 
 namespace SimpleSAML\Test\XML;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleSAML\Test\XML\ExtendableAttributesElement;
 use SimpleSAML\XML\Attribute;
@@ -16,11 +17,10 @@
 /**
  * Class \SimpleSAML\XML\ExtendableAttributesTest
  *
- * @covers \SimpleSAML\XML\TestUtils\SchemaValidationTestTrait
- * @covers \SimpleSAML\XML\TestUtils\SerializableElementTestTrait
- *
  * @package simplesamlphp\xml-common
  */
+#[CoversClass(SchemaValidationTestTrait::class)]
+#[CoversClass(SerializableElementTestTrait::class)]
 final class ExtendableAttributesTest extends TestCase
 {
     use SchemaValidationTestTrait;