Modernizing test suite to PHPUnit 11.x compatibility

1 files changed, 10 insertions, 15 deletions

diff --git a/test/ApiTest.php b/test/ApiTest.php
index 0a8290c..0786af0 100644
--- a/test/ApiTest.php
+++ b/test/ApiTest.php
@@ -1,8 +1,9 @@
 <?php
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 
-class ApiTest extends TestCase {
+final class ApiTest extends TestCase {
 
     private function _doTest($input, $expected) {
         require_once(dirname(__FILE__) . '/../http/api-inc.php');
@@ -15,30 +16,24 @@ class ApiTest extends TestCase {
         }
     }
 
-    /**
-     * @dataProvider generatedDataProvider
-     */
+    #[DataProvider('generatedDataProvider')]
     public function testRandomResults($input, $expected) {
         $this->_doTest($input, $expected);
     }
 
-    /**
-     * @dataProvider generatedDataProvider
-     */
+    #[DataProvider('generatedDataProvider')]
     public function testBoardCounts($input, $expected) {
         $input['boards'] = strval($input['over39_boards'] ? rand(40, 60) : rand(1, 39));
         unset($input['over39_boards']);
         $this->_doTest($input, $expected);
     }
 
-    /**
-     * @dataProvider bridgenetDataProvider
-     */
+    #[DataProvider('bridgenetDataProvider')]
     public function testBridgenet($input, $expected) {
         $this->_doTest($input, $expected);
     }
 
-    private function _fileDataProvider($file) {
+    private static function _fileDataProvider($file) {
         $contents = file(dirname(__FILE__) . '/' . $file);
         $count = count($contents);
         for ($i = 0; $i < $count; $i += 2) {
@@ -48,12 +43,12 @@ class ApiTest extends TestCase {
         }
     }
 
-    public function generatedDataProvider() {
-        yield from $this->_fileDataProvider('tests.txt');
+    public static function generatedDataProvider() {
+        yield from self::_fileDataProvider('tests.txt');
     }
 
-    public function bridgenetDataProvider() {
-        yield from $this->_fileDataProvider('bridgenet.txt');
+    public static function bridgenetDataProvider() {
+        yield from self::_fileDataProvider('bridgenet.txt');
     }
 
 }