Replace covers and dataProvider annotations

Author: Art4

tests/Unit/Api/Group/UpdateTest.php

@@ -5,6 +5,7 @@
 namespace Redmine\Tests\Unit\Api\Group;
 
 use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\Group;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
@@ -15,6 +16,7 @@ class UpdateTest extends TestCase
     /**
      * @dataProvider getUpdateData
      */
+    #[DataProvider('getUpdateData')]
     public function testUpdateReturnsCorrectResponse($id, $parameters, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(

tests/Unit/Api/Issue/AddNoteToIssueTest.php

@@ -4,18 +4,19 @@
 
 namespace Redmine\Tests\Unit\Api\Issue;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\Issue;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
 
-/**
- * @covers \Redmine\Api\Issue::addNoteToIssue
- */
+#[CoversClass(Issue::class)]
 class AddNoteToIssueTest extends TestCase
 {
     /**
      * @dataProvider getAddNoteToIssueData
      */
+    #[DataProvider('getAddNoteToIssueData')]
     public function testAddNoteToIssueReturnsCorrectResponse($id, $note, $isPrivate, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(

tests/Unit/Api/Issue/AttachManyTest.php

@@ -2,18 +2,19 @@
 
 namespace Redmine\Tests\Unit\Api\Issue;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\Issue;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
 
-/**
- * @covers \Redmine\Api\Issue::attachMany
- */
+#[CoversClass(Issue::class)]
 class AttachManyTest extends TestCase
 {
     /**
      * @dataProvider getAttachManyData
      */
+    #[DataProvider('getAttachManyData')]
     public function testAttachManyReturnsCorrectResponse($issueId, $parameters, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(

tests/Unit/Api/Issue/AttachTest.php

@@ -2,18 +2,19 @@
 
 namespace Redmine\Tests\Unit\Api\Issue;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\Issue;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
 
-/**
- * @covers \Redmine\Api\Issue::attach
- */
+#[CoversClass(Issue::class)]
 class AttachTest extends TestCase
 {
     /**
      * @dataProvider getAttachData
      */
+    #[DataProvider('getAttachData')]
     public function testAttachReturnsCorrectResponse($issueId, $parameters, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(

tests/Unit/Api/Issue/SetIssueStatusTest.php

@@ -2,14 +2,12 @@
 
 namespace Redmine\Tests\Unit\Api\Issue;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\Issue;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
-use SimpleXMLElement;
 
-/**
- * @covers \Redmine\Api\Issue::setIssueStatus
- */
+#[CoversClass(Issue::class)]
 class SetIssueStatusTest extends TestCase
 {
     public function testSetIssueStatusReturnsCorrectResponse()

tests/Unit/Api/Issue/UpdateTest.php

@@ -4,18 +4,19 @@
 
 namespace Redmine\Tests\Unit\Api\Issue;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\Issue;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
 
-/**
- * @covers \Redmine\Api\Issue::update
- */
+#[CoversClass(Issue::class)]
 class UpdateTest extends TestCase
 {
     /**
      * @dataProvider getUpdateData
      */
+    #[DataProvider('getUpdateData')]
     public function testUpdateReturnsCorrectResponse($id, $parameters, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(
@@ -112,14 +113,6 @@ public static function getUpdateData(): array
         ];
     }
 
-    /**
-     * @covers \Redmine\Api\Issue::cleanParams
-     * @covers \Redmine\Api\Issue::getIssueCategoryApi
-     * @covers \Redmine\Api\Issue::getIssueStatusApi
-     * @covers \Redmine\Api\Issue::getProjectApi
-     * @covers \Redmine\Api\Issue::getTrackerApi
-     * @covers \Redmine\Api\Issue::getUserApi
-     */
     public function testUpdateCleansParameters()
     {
         $client = AssertingHttpClient::create(

tests/Unit/Api/IssueCategory/UpdateTest.php

@@ -4,18 +4,19 @@
 
 namespace Redmine\Tests\Unit\Api\IssueCategory;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\IssueCategory;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
 
-/**
- * @covers \Redmine\Api\IssueCategory::update
- */
+#[CoversClass(IssueCategory::class)]
 class UpdateTest extends TestCase
 {
     /**
      * @dataProvider getUpdateData
      */
+    #[DataProvider('getUpdateData')]
     public function testUpdateReturnsCorrectResponse($id, $parameters, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(

tests/Unit/Api/Membership/UpdateTest.php

@@ -2,21 +2,21 @@
 
 namespace Redmine\Tests\Unit\Api\Membership;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\Membership;
 use Redmine\Exception\MissingParameterException;
 use Redmine\Http\HttpClient;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
-use SimpleXMLElement;
 
-/**
- * @covers \Redmine\Api\Membership::update
- */
+#[CoversClass(Membership::class)]
 class UpdateTest extends TestCase
 {
     /**
      * @dataProvider getUpdateData
      */
+    #[DataProvider('getUpdateData')]
     public function testUpdateReturnsCorrectResponse($id, $parameters, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(
@@ -104,6 +104,7 @@ public function testUpdateThrowsExceptionWithEmptyParameters()
     /**
      * @dataProvider incompleteUpdateParameterProvider
      */
+    #[DataProvider('incompleteUpdateParameterProvider')]
     public function testUpdateThrowsExceptionIfMandatoyParametersAreMissing($parameters)
     {
         // Create the used mock objects

tests/Unit/Api/Project/UpdateTest.php

@@ -4,18 +4,19 @@
 
 namespace Redmine\Tests\Unit\Api\Project;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\Project;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
 
-/**
- * @covers \Redmine\Api\Project::update
- */
+#[CoversClass(Project::class)]
 class UpdateTest extends TestCase
 {
     /**
      * @dataProvider getUpdateData
      */
+    #[DataProvider('getUpdateData')]
     public function testUpdateReturnsCorrectResponse($id, $parameters, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(

tests/Unit/Api/TimeEntry/UpdateTest.php

@@ -4,18 +4,19 @@
 
 namespace Redmine\Tests\Unit\Api\TimeEntry;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Redmine\Api\TimeEntry;
 use Redmine\Tests\Fixtures\AssertingHttpClient;
 
-/**
- * @covers \Redmine\Api\TimeEntry::update
- */
+#[CoversClass(TimeEntry::class)]
 class UpdateTest extends TestCase
 {
     /**
      * @dataProvider getUpdateData
      */
+    #[DataProvider('getUpdateData')]
     public function testUpdateReturnsCorrectResponse($id, $parameters, $expectedPath, $expectedBody, $responseCode, $response)
     {
         $client = AssertingHttpClient::create(