replace dataProvider annotations with DataProvider attribute

tests/Functional/DotNotationTest.php

@@ -20,9 +20,6 @@ class DotNotationTest extends TestCase
 {
     use HelperTrait;
 
-    /**
-     * @test
-     */
     public function testCompleteResourceObjectWithMultipleRelationships(): void
     {
         $manager = new ErrorAbortManager(new Factory());
@@ -189,9 +186,6 @@ public function testCompleteResourceObjectWithMultipleRelationships(): void
         $this->assertSame('http://example.com/comments/12', $document->get('included.2.links.self'));
     }
 
-    /**
-     * @test
-     */
     public function testGetNotExistentValueThrowsException(): void
     {
         $manager = new ErrorAbortManager(new Factory());

tests/Functional/ErrorParsingTest.php

@@ -16,9 +16,6 @@ class ErrorParsingTest extends TestCase
 {
     use HelperTrait;
 
-    /**
-     * @test
-     */
     public function testParseErrors(): void
     {
         $string = $this->getJsonString('09_errors.json');
@@ -89,9 +86,6 @@ public function testParseErrors(): void
         $this->assertFalse($errors->has('3'));
     }
 
-    /**
-     * @test
-     */
     public function testParseErrorWithLinks(): void
     {
         $string = $this->getJsonString('10_error_with_links.json');

tests/Functional/ParsingTest.php

@@ -14,6 +14,7 @@
 use Art4\JsonApiClient\Manager\ErrorAbortManager;
 use Art4\JsonApiClient\Tests\Fixtures\HelperTrait;
 use Art4\JsonApiClient\V1\Factory;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class ParsingTest extends TestCase
@@ -38,9 +39,7 @@ public static function createParserProvider(): array
         ];
     }
 
-    /**
-     * @dataProvider createParserProvider
-     */
+    #[DataProvider('createParserProvider')]
     public function testParseSimpleResourceWithDifferentParser(callable $parser): void
     {
         $string = $this->getJsonString('01_simple_resource.json');
@@ -67,9 +66,6 @@ public function testParseSimpleResourceWithDifferentParser(callable $parser): vo
         $this->assertInstanceOf('Art4\JsonApiClient\V1\RelationshipCollection', $resource->get('relationships'));
     }
 
-    /**
-     * @test
-     */
     public function testParseSimpleResourceIdentifier(): void
     {
         $string = $this->getJsonString('02_simple_resource_identifier.json');
@@ -91,9 +87,6 @@ public function testParseSimpleResourceIdentifier(): void
         $this->assertSame($resource->get('id'), '1');
     }
 
-    /**
-     * @test
-     */
     public function testParseResourceObject(): void
     {
         $string = $this->getJsonString('03_resource_object.json');
@@ -159,9 +152,6 @@ public function testParseResourceObject(): void
         $this->assertSame($data->get('id'), '9');
     }
 
-    /**
-     * @test
-     */
     public function testParseCompleteResourceObjectWithMultipleRelationships(): void
     {
         $string = $this->getJsonString('04_complete_document_with_multiple_relationships.json');
@@ -355,9 +345,6 @@ public function testParseCompleteResourceObjectWithMultipleRelationships(): void
         $this->assertSame($links->get('self'), 'http://example.com/comments/12');
     }
 
-    /**
-     * @test
-     */
     public function testParsePaginationExample(): void
     {
         $string = $this->getJsonString('06_pagination_example.json');
@@ -409,9 +396,6 @@ public function testParsePaginationExample(): void
         $this->assertSame($links->get('last'), 'http://example.com/articles?page[number]=13&page[size]=1');
     }
 
-    /**
-     * @test
-     */
     public function testParseRelationshipExample(): void
     {
         $string = $this->getJsonString('07_relationship_example_without_data.json');
@@ -475,9 +459,6 @@ public function testParseRelationshipExample(): void
         $this->assertSame($document->get('data.0.relationships.comments.links.related.meta.count'), 10);
     }
 
-    /**
-     * @test
-     */
     public function testParseObjectLinksExample(): void
     {
         $string = $this->getJsonString('08_object_links.json');
@@ -518,9 +499,6 @@ public function testParseObjectLinksExample(): void
         $this->assertSame('bar', $document->get('jsonapi.meta.foo'));
     }
 
-    /**
-     * @test
-     */
     public function testParseResourceIdentifierWithMeta(): void
     {
         $string = $this->getJsonString('11_resource_identifier_with_meta.json');
@@ -544,9 +522,6 @@ public function testParseResourceIdentifierWithMeta(): void
         $this->assertSame($resource->get('id'), '2');
     }
 
-    /**
-     * @test
-     */
     public function testParseNullResource(): void
     {
         $string = $this->getJsonString('12_null_resource.json');
@@ -565,9 +540,6 @@ public function testParseNullResource(): void
         $this->assertInstanceOf('Art4\JsonApiClient\V1\ResourceNull', $resource);
     }
 
-    /**
-     * @test
-     */
     public function testParseResourceIdentifierCollectionWithMeta(): void
     {
         $string = $this->getJsonString('13_collection_with_resource_identifier_with_meta.json');
@@ -602,9 +574,6 @@ public function testParseResourceIdentifierCollectionWithMeta(): void
         $this->assertSame($resource1->get('id'), '2');
     }
 
-    /**
-     * @test
-     */
     public function testParseCreateResourceWithoutId(): void
     {
         $string = $this->getJsonString('14_create_resource_without_id.json');
@@ -614,9 +583,6 @@ public function testParseCreateResourceWithoutId(): void
         $this->assertSame(['data'], $document->getKeys());
     }
 
-    /**
-     * @test
-     */
     public function testParseCreateShortResourceWithoutId(): void
     {
         $string = $this->getJsonString('15_create_resource_without_id.json');
@@ -626,19 +592,13 @@ public function testParseCreateShortResourceWithoutId(): void
         $this->assertSame(['data'], $document->getKeys());
     }
 
-    /**
-     * @test
-     */
     public function testExceptionIfIdIsNotString(): void
     {
         $this->expectException(\Art4\JsonApiClient\Exception\ValidationException::class);
         $string = $this->getJsonString('16_type_and_id_as_integer.json');
         $document = Parser::parseResponseString($string);
     }
 
-    /**
-     * @test
-     */
     public function testParseLinksInRelationshipsCorrectly(): void
     {
         $string = $this->getJsonString('17_relationship_links.json');

tests/Functional/SerializerTest.php

@@ -15,6 +15,7 @@
 use Art4\JsonApiClient\Tests\Fixtures\HelperTrait;
 use Art4\JsonApiClient\V1\Factory;
 use Exception;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class SerializerTest extends TestCase
@@ -61,9 +62,7 @@ public static function jsonapiDataProvider(): array
         return $files;
     }
 
-    /**
-     * @dataProvider jsonapiDataProvider
-     */
+    #[DataProvider('jsonapiDataProvider')]
     public function testParseJsonapiDataWithErrorAbortManager(string $filename, bool $isRequest): void
     {
         $manager = new ErrorAbortManager(new Factory());

tests/Unit/Helper/AccessableTraitTest.php

@@ -11,15 +11,14 @@
 use Art4\JsonApiClient\Exception\AccessException;
 use Art4\JsonApiClient\Tests\Fixtures\AccessableTraitMock;
 use Art4\JsonApiClient\Tests\Fixtures\HelperTrait;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class AccessableTraitTest extends TestCase
 {
     use HelperTrait;
 
-    /**
-     * @dataProvider jsonValuesProviderWithoutStringAndInt
-     */
+    #[DataProvider('jsonValuesProviderWithoutStringAndInt')]
     public function testHasWithInvalidKeyTypeTriggersDeprecationError(mixed $key): void
     {
         $resource = new AccessableTraitMock();
@@ -41,9 +40,7 @@ function ($errno, $errstr) use ($key): bool {
         $resource->has($key);
     }
 
-    /**
-     * @dataProvider jsonValuesProviderWithoutStringAndInt
-     */
+    #[DataProvider('jsonValuesProviderWithoutStringAndInt')]
     public function testGetWithInvalidKeyTypeTriggersDeprecationError(mixed $key): void
     {
         $resource = new AccessableTraitMock();

tests/Unit/Input/RequestStringInputTest.php

@@ -11,6 +11,7 @@
 use Art4\JsonApiClient\Exception\InputException;
 use Art4\JsonApiClient\Input\RequestStringInput;
 use Art4\JsonApiClient\Tests\Fixtures\HelperTrait;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class RequestStringInputTest extends TestCase
@@ -24,9 +25,7 @@ public function testGetAsObjectFromStringReturnsObject(): void
         $this->assertInstanceOf(\stdClass::class, $input->getAsObject());
     }
 
-    /**
-     * @dataProvider jsonValuesProviderWithoutString
-     */
+    #[DataProvider('jsonValuesProviderWithoutString')]
     public function testCreateWithoutStringThrowsException(mixed $input): void
     {
         $this->expectException(InputException::class);
@@ -36,9 +35,7 @@ public function testCreateWithoutStringThrowsException(mixed $input): void
         new RequestStringInput($input);
     }
 
-    /**
-     * @dataProvider jsonValuesAsStringProviderWithoutObject
-     */
+    #[DataProvider('jsonValuesAsStringProviderWithoutObject')]
     public function testGetAsObjectWithInvalidStringsThrowsException(string $input): void
     {
         $input = new RequestStringInput($input);

tests/Unit/Input/ResponseStringInputTest.php

@@ -11,6 +11,7 @@
 use Art4\JsonApiClient\Exception\InputException;
 use Art4\JsonApiClient\Input\ResponseStringInput;
 use Art4\JsonApiClient\Tests\Fixtures\HelperTrait;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class ResponseStringInputTest extends TestCase
@@ -24,9 +25,7 @@ public function testGetAsObjectFromStringReturnsObject(): void
         $this->assertInstanceOf(\stdClass::class, $input->getAsObject());
     }
 
-    /**
-     * @dataProvider jsonValuesProviderWithoutString
-     */
+    #[DataProvider('jsonValuesProviderWithoutString')]
     public function testCreateWithoutStringThrowsException(mixed $input): void
     {
         $this->expectException(InputException::class);
@@ -36,9 +35,7 @@ public function testCreateWithoutStringThrowsException(mixed $input): void
         new ResponseStringInput($input);
     }
 
-    /**
-     * @dataProvider jsonValuesAsStringProviderWithoutObject
-     */
+    #[DataProvider('jsonValuesAsStringProviderWithoutObject')]
     public function testGetAsObjectWithInvalidStringsThrowsException(string $input): void
     {
         $input = new ResponseStringInput($input);

tests/Unit/Manager/ErrorAbortManagerTest.php

@@ -14,9 +14,6 @@
 
 class ErrorAbortManagerTest extends TestCase
 {
-    /**
-     * @test
-     */
     public function testCreateWithConstructorReturnsSelf(): void
     {
         $factory = $this->createMock(Factory::class);
@@ -25,9 +22,6 @@ public function testCreateWithConstructorReturnsSelf(): void
         $this->assertSame($factory, $manager->getFactory());
     }
 
-    /**
-     * @test
-     */
     public function testGetParamReturnsDefault(): void
     {
         $factory = $this->createMock(Factory::class);

tests/Unit/V1/AttributesTest.php

@@ -13,6 +13,7 @@
 use Art4\JsonApiClient\Exception\ValidationException;
 use Art4\JsonApiClient\Tests\Fixtures\HelperTrait;
 use Art4\JsonApiClient\V1\Attributes;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
 class AttributesTest extends TestCase
@@ -80,9 +81,7 @@ public function testCreateWithObject(): void
         ], $attributes->getKeys());
     }
 
-    /**
-     * @dataProvider jsonValuesProviderWithoutObject
-     */
+    #[DataProvider('jsonValuesProviderWithoutObject')]
     public function testCreateWithDataProvider(mixed $input): void
     {
         $this->expectException(ValidationException::class);
@@ -186,9 +185,6 @@ public function testCreateWithLinksPropertyThrowsException(): void
         );
     }
 
-    /**
-     * @test
-     */
     public function testGetOnANonExistingKeyThrowsException(): void
     {
         $object = new \stdClass();