ci(phpunit): use attributes (#94)

cdn77 · Feb 4, 2024 · e011bae · e011bae
1 parent 57bdb42
commit e011bae
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 8 deletions.
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -8,6 +8,10 @@
     executionOrder="random"
     colors="true"
     bootstrap="tests/bootstrap.php"
+    failOnEmptyTestSuite="true"
+    failOnDeprecation="true"
+    failOnNotice="true"
+    failOnWarning="true"
 >
     <testsuites>
         <testsuite name="Test Suite">

diff --git a/tests/TestCheck/EveryTestHasGroupTest.php b/tests/TestCheck/EveryTestHasGroupTest.php
@@ -8,15 +8,15 @@
 use Cdn77\TestUtils\Tests\BaseTestCase;
 use Generator;
 use PHPUnit\Framework\AssertionFailedError;
+use PHPUnit\Framework\Attributes\DataProvider;
 
 final class EveryTestHasGroupTest extends BaseTestCase
 {
     /**
      * @param non-empty-list<string>|null $requiredGroups
      * @param list<string>|null $supportedGroups
-     *
-     * @dataProvider providerSuccess
      */
+    #[DataProvider('providerSuccess')]
     public function testSuccess(
         string $filePath,
         array|null $requiredGroups,
@@ -38,9 +38,8 @@ public static function providerSuccess(): Generator
     /**
      * @param non-empty-list<string>|null $requiredGroups
      * @param list<string>|null $supportedGroups
-     *
-     * @dataProvider providerFail
      */
+    #[DataProvider('providerFail')]
     public function testFail(
         string $filePath,
         array|null $requiredGroups,

diff --git a/tests/TestCheck/EveryTestHasSameNamespaceAsCoveredClassTest.php b/tests/TestCheck/EveryTestHasSameNamespaceAsCoveredClassTest.php
@@ -8,10 +8,11 @@
 use Cdn77\TestUtils\Tests\BaseTestCase;
 use Generator;
 use PHPUnit\Framework\AssertionFailedError;
+use PHPUnit\Framework\Attributes\DataProvider;
 
 final class EveryTestHasSameNamespaceAsCoveredClassTest extends BaseTestCase
 {
-    /** @dataProvider providerSuccess */
+    #[DataProvider('providerSuccess')]
     public function testSuccess(string $filePath): void
     {
         $check = new EveryTestHasSameNamespaceAsCoveredClass(
@@ -37,7 +38,7 @@ public static function providerSuccess(): Generator
               }
     }
 
-    /** @dataProvider providerFail */
+    #[DataProvider('providerFail')]
     public function testFail(string $filePath, string $error): void
     {
         $this->expectException(AssertionFailedError::class);

diff --git a/tests/TestCheck/EveryTestInheritsFromTestCaseBaseClassTest.php b/tests/TestCheck/EveryTestInheritsFromTestCaseBaseClassTest.php
@@ -8,10 +8,11 @@
 use Cdn77\TestUtils\Tests\BaseTestCase;
 use Generator;
 use PHPUnit\Framework\AssertionFailedError;
+use PHPUnit\Framework\Attributes\DataProvider;
 
 final class EveryTestInheritsFromTestCaseBaseClassTest extends BaseTestCase
 {
-    /** @dataProvider providerSuccess */
+    #[DataProvider('providerSuccess')]
     public function testSuccess(string $filePath): void
     {
         $check = new EveryTestInheritsFromTestCaseBaseClass(
@@ -29,7 +30,7 @@ public static function providerSuccess(): Generator
         yield ['../../BaseTestCase.php'];
     }
 
-    /** @dataProvider providerFail */
+    #[DataProvider('providerFail')]
     public function testFail(string $filePath): void
     {
         try {