Add support for PHPUnit 10 and PHPUnit 11

thephpleague · Apr 12, 2024 · d2a8b41 · d2a8b41
1 parent b94fe40
commit d2a8b41
Show file tree

Hide file tree

Showing 15 changed files with 173 additions and 433 deletions.
diff --git a/Components/AuthorityTest.php b/Components/AuthorityTest.php
@@ -16,20 +16,19 @@
 use League\Uri\Http;
 use League\Uri\Uri;
 use League\Uri\UriString;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Message\UriInterface as Psr7UriInterface;
 
 use function parse_url;
 
-/**
- * @group userinfo
- * @coversDefaultClass \League\Uri\Components\Authority
- */
+#[CoversClass(Authority::class)]
+#[Group('userinfo')]
 final class AuthorityTest extends TestCase
 {
-    /**
-     * @dataProvider validAuthorityDataProvider
-     */
+    #[DataProvider('validAuthorityDataProvider')]
     public function testConstructor(
         ?string $authority,
         ?string $host,
@@ -90,9 +89,7 @@ public static function validAuthorityDataProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider invalidAuthorityDataProvider
-     */
+    #[DataProvider('invalidAuthorityDataProvider')]
     public function testConstructorFails(string $authority): void
     {
         $this->expectException(SyntaxError::class);
@@ -116,9 +113,7 @@ public function testWithHost(): void
         self::assertNotEquals($authority, $authority->withHost('[::1]'));
     }
 
-    /**
-     * @dataProvider invalidHostDataProvider
-     */
+    #[DataProvider('invalidHostDataProvider')]
     public function testWithHostFails(?string $host): void
     {
         $this->expectException(SyntaxError::class);
@@ -164,9 +159,7 @@ public function testWithUserInfoFails(): void
         Authority::new('foo:[email protected]:443')->withUserInfo("\0foo", 'bar');
     }
 
-    /**
-     * @dataProvider stringRepresentationDataProvider
-    */
+    #[DataProvider('stringRepresentationDataProvider')]
     public function testAuthorityStringRepresentation(
         ?string $authority,
         string $string,
@@ -220,9 +213,7 @@ public static function stringRepresentationDataProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider getURIProvider
-     */
+    #[DataProvider('getURIProvider')]
     public function testCreateFromUri(UriInterface|Psr7UriInterface $uri, ?string $expected, array $components): void
     {
         $authority = Authority::fromUri($uri);

diff --git a/Components/DataPathTest.php b/Components/DataPathTest.php
@@ -15,18 +15,19 @@
 use League\Uri\Exceptions\SyntaxError;
 use League\Uri\Http;
 use League\Uri\Uri;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Message\UriInterface as Psr7UriInterface;
 
 use function base64_encode;
 use function dirname;
 use function file_get_contents;
 
-/**
- * @group path
- * @group datapath
- * @coversDefaultClass \League\Uri\Components\DataPath
- */
+#[CoversClass(DataPath::class)]
+#[Group('path')]
+#[Group('datapath')]
 final class DataPathTest extends TestCase
 {
     private string $rootPath;
@@ -71,19 +72,15 @@ public function testConstructorFailedMalformePath(): void
         DataPath::new('€');
     }
 
-    /**
-     * @dataProvider invalidDataUriPath
-     */
+    #[DataProvider('invalidDataUriPath')]
     public function testCreateFromPathFailed(string $path): void
     {
         $this->expectException(SyntaxError::class);
 
         DataPath::fromFileContents($path);
     }
 
-    /**
-     * @dataProvider invalidDataUriPath
-     */
+    #[DataProvider('invalidDataUriPath')]
     public function testConstructorFailed(string $path): void
     {
         $this->expectException(SyntaxError::class);
@@ -98,9 +95,7 @@ public static function invalidDataUriPath(): array
         ];
     }
 
-    /**
-     * @dataProvider validPathContent
-     */
+    #[DataProvider('validPathContent')]
     public function testDefaultConstructor(string $path, string $expected): void
     {
         self::assertSame($expected, DataPath::new($path)->toString());
@@ -124,9 +119,7 @@ public static function validPathContent(): array
         ];
     }
 
-    /**
-     * @dataProvider validFilePath
-     */
+    #[DataProvider('validFilePath')]
     public function testCreateFromPath(string $path, string $mimetype, string $mediatype): void
     {
         $uri = DataPath::fromFileContents($path);
@@ -162,9 +155,7 @@ public function testWithParametersOnBinaryData(): void
         self::assertSame($expected, $newUri->getParameters());
     }
 
-    /**
-     * @dataProvider invalidParametersString
-     */
+    #[DataProvider('invalidParametersString')]
     public function testWithParametersFailedWithInvalidParameters(string $path, string $parameters): void
     {
         $this->expectException(SyntaxError::class);
@@ -186,17 +177,13 @@ public static function invalidParametersString(): array
         ];
     }
 
-    /**
-     * @dataProvider fileProvider
-     */
+    #[DataProvider('fileProvider')]
     public function testToBinary(DataPath $uri): void
     {
         self::assertTrue($uri->toBinary()->isBinaryData());
     }
 
-    /**
-     * @dataProvider fileProvider
-     */
+    #[DataProvider('fileProvider')]
     public function testToAscii(DataPath $uri): void
     {
         self::assertFalse($uri->toAscii()->isBinaryData());
@@ -212,9 +199,7 @@ public static function fileProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider invalidParameters
-     */
+    #[DataProvider('invalidParameters')]
     public function testUpdateParametersFailed(string $parameters): void
     {
         $this->expectException(SyntaxError::class);
@@ -299,9 +284,7 @@ public function testInvalidMimetype(): void
     }
 
 
-    /**
-     * @dataProvider getURIProvider
-     */
+    #[DataProvider('getURIProvider')]
     public function testCreateFromUri(Psr7UriInterface|UriInterface $uri, ?string $expected): void
     {
         $path = DataPath::fromUri($uri);

diff --git a/Components/DomainTest.php b/Components/DomainTest.php
@@ -18,13 +18,14 @@
 use League\Uri\Exceptions\SyntaxError;
 use League\Uri\Http;
 use League\Uri\Uri;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Message\UriInterface as Psr7UriInterface;
 
-/**
- * @group host
- * @coversDefaultClass \League\Uri\Components\Domain
- */
+#[CoversClass(Domain::class)]
+#[Group('host')]
 final class DomainTest extends TestCase
 {
     public function testItCanBeInstantiatedWithAHostInterfaceImplementingObject(): void
@@ -60,8 +61,8 @@ public function testIterator(): void
 
     /**
      * Test valid Domain.
-     * @dataProvider validDomainProvider
      */
+    #[DataProvider('validDomainProvider')]
     public function testValidDomain(string $host, string $uri, string $iri): void
     {
         $host = Domain::new($host);
@@ -111,9 +112,7 @@ public static function validDomainProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider invalidDomainProvider
-     */
+    #[DataProvider('invalidDomainProvider')]
     public function testInvalidDomain(?string $invalid): void
     {
         $this->expectException(SyntaxError::class);
@@ -150,9 +149,7 @@ public static function invalidDomainProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider isAbsoluteProvider
-     */
+    #[DataProvider('isAbsoluteProvider')]
     public function testIsAbsolute(string $raw, bool $expected): void
     {
         self::assertSame($expected, Domain::new($raw)->isAbsolute());
@@ -174,9 +171,7 @@ public function testIpProperty(): void
         self::assertNull($host->getIp());
     }
 
-    /**
-     * @dataProvider hostnamesProvider
-     */
+    #[DataProvider('hostnamesProvider')]
     public function testValidUnicodeDomain(string $unicode, string $ascii): void
     {
         $host = Domain::new($unicode);
@@ -211,9 +206,7 @@ public static function hostnamesProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider countableProvider
-     */
+    #[DataProvider('countableProvider')]
     public function testCountable(string $host, int $nblabels): void
     {
         self::assertCount($nblabels, Domain::new($host));
@@ -227,9 +220,7 @@ public static function countableProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider createFromLabelsValid
-     */
+    #[DataProvider('createFromLabelsValid')]
     public function testCreateFromLabels(iterable $input, string $expected): void
     {
         self::assertSame($expected, (string) Domain::fromLabels(...$input));
@@ -279,9 +270,7 @@ public function testLabels(): void
         self::assertSame(['', 'localhost'], [...Domain::new('localhost.')]);
     }
 
-    /**
-     * @dataProvider withoutProvider
-     */
+    #[DataProvider('withoutProvider')]
     public function testWithout(string $host, int $without, string $res): void
     {
         self::assertSame($res, (string) Domain::new($host)->withoutLabel($without));
@@ -309,9 +298,7 @@ public function testWithoutTriggersException(): void
         Domain::new('bébé.be')->withoutLabel(-23);
     }
 
-    /**
-     * @dataProvider validPrepend
-     */
+    #[DataProvider('validPrepend')]
     public function testPrepend(string $raw, string $prepend, string $expected): void
     {
         self::assertSame($expected, (string) Domain::new($raw)->prepend($prepend));
@@ -343,9 +330,7 @@ public function testPrependNull(): void
         self::assertSame($domain->prepend(null), $domain);
     }
 
-    /**
-     * @dataProvider validAppend
-     */
+    #[DataProvider('validAppend')]
     public function testAppend(string $raw, string $append, string $expected): void
     {
         self::assertSame($expected, (string) Domain::new($raw)->append($append));
@@ -378,9 +363,7 @@ public function testAppendNull(): void
         self::assertSame($domain->append(null), $domain);
     }
 
-    /**
-     * @dataProvider replaceValid
-     */
+    #[DataProvider('replaceValid')]
     public function testReplace(string $raw, string $input, int $offset, string $expected): void
     {
         self::assertSame($expected, (string) Domain::new($raw)->withLabel($offset, $input));
@@ -414,9 +397,7 @@ public function testReplaceMustFailed(): void
         Domain::new('secure.example.com')->withLabel(23, 'foo');
     }
 
-    /**
-     * @dataProvider rootProvider
-     */
+    #[DataProvider('rootProvider')]
     public function testWithRoot(string $host, string $expected_with_root, string $expected_without_root): void
     {
         $host = Domain::new($host);
@@ -433,9 +414,7 @@ public static function rootProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider getURIProvider
-     */
+    #[DataProvider('getURIProvider')]
     public function testCreateFromUri(Psr7UriInterface|UriInterface $uri, ?string $expected): void
     {
         $domain = Domain::fromUri($uri);