upgrade PHPUnit to v10+

composer.json

@@ -24,7 +24,7 @@
         "ext-json": "*",
         "ext-redis": "*",
         "ext-curl": "*",
-        "phpunit/phpunit": "~9.0 || ~10.0 || ~11.0",
+        "phpunit/phpunit": "~10.0 || ~11.0",
         "swoole/ide-helper": "dev-master"
     },
     "suggest": {

tests/unit/ArrayObjectTest.php

@@ -11,12 +11,13 @@
 
 namespace Swoole;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
- * @coversNothing
  */
+#[CoversClass(ArrayObject::class)]
 class ArrayObjectTest extends TestCase
 {
     private ArrayObject $data;
@@ -29,12 +30,6 @@ class ArrayObjectTest extends TestCase
 
     private array $control_data;
 
-    /**
-     * ArrayObjectTest constructor.
-     * @covers \Swoole\ArrayObject::each()
-     * @covers \Swoole\ArrayObject::split()
-     * @param string $dataName
-     */
     public function __construct(?string $name = null, array $data = [], $dataName = '')
     {
         $_data      = '11, 33, 22, 44,12,32,55, 23,19,23';
@@ -63,19 +58,11 @@ public function __construct(?string $name = null, array $data = [], $dataName =
         parent::__construct($name, $data, $dataName);
     }
 
-    /**
-     * @covers \Swoole\ArrayObject::toArray()
-     */
     public function testToArray()
     {
         $this->assertEquals($this->data->toArray(), $this->control_data);
     }
 
-    /**
-     * @covers \Swoole\ArrayObject::each()
-     * @covers \Swoole\ArrayObject::sort()
-     * @covers \Swoole\ArrayObject::unique()
-     */
     public function testMix()
     {
         $datao = clone $this->data;
@@ -88,17 +75,11 @@ public function testMix()
         $this->assertEquals($data, $expectResult);
     }
 
-    /**
-     * @covers \Swoole\ArrayObject::serialize()
-     */
     public function testSerialize()
     {
         $this->assertEquals(serialize($this->data->toArray()), $this->data->serialize());
     }
 
-    /**
-     * @covers \Swoole\ArrayObject::unique()
-     */
     public function testUnique()
     {
         $data         = $this->data->unique()->toArray();

tests/unit/Coroutine/BarrierTest.php

@@ -11,18 +11,16 @@
 
 namespace Swoole\Coroutine;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use Swoole\Coroutine;
 
 /**
  * @internal
- * @coversNothing
  */
+#[CoversClass(Barrier::class)]
 class BarrierTest extends TestCase
 {
-    /**
-     * @covers \Swoole\Coroutine\Barrier
-     */
     public function testWait()
     {
         run(function () {
@@ -43,9 +41,6 @@ public function testWait()
         });
     }
 
-    /**
-     * @covers \Swoole\Coroutine\Barrier
-     */
     public function testWaitTimeout()
     {
         run(function () {
@@ -73,8 +68,6 @@ public function testWaitTimeout()
 
     /**
      * Test without execution switching between coroutines.
-     *
-     * @covers \Swoole\Coroutine\Barrier
      */
     public function testNoCoroutineSwitching()
     {
@@ -95,8 +88,6 @@ public function testNoCoroutineSwitching()
 
     /**
      * Test without any child coroutines created. Ideally we shouldn't use the Barrier class this way.
-     *
-     * @covers \Swoole\Coroutine\Barrier
      */
     public function testWithoutAnyChildCoroutines()
     {
@@ -109,8 +100,6 @@ public function testWithoutAnyChildCoroutines()
 
     /**
      * Test with the Barrier object destroyed in a child coroutine. Ideally we shouldn't use the Barrier class this way.
-     *
-     * @covers \Swoole\Coroutine\Barrier
      */
     public function testUnexpectedDestroy()
     {
@@ -129,8 +118,6 @@ public function testUnexpectedDestroy()
 
     /**
      * Test with the Barrier object destroyed in a child coroutine following by a coroutine switching. Ideally we shouldn't use the Barrier class this way.
-     *
-     * @covers \Swoole\Coroutine\Barrier
      */
     public function testUnexpectedDestroyWithCoroutineSwitching()
     {

tests/unit/Coroutine/FunctionTest.php

@@ -11,13 +11,18 @@
 
 namespace Swoole\Coroutine;
 
+use PHPUnit\Framework\Attributes\CoversFunction;
 use PHPUnit\Framework\TestCase;
 use Swoole\Runtime;
 
 /**
  * @internal
  * @coversNothing
  */
+#[CoversFunction('Swoole\Coroutine\batch')]
+#[CoversFunction('Swoole\Coroutine\go')]
+#[CoversFunction('Swoole\Coroutine\parallel')]
+#[CoversFunction('Swoole\Coroutine\map')]
 class FunctionTest extends TestCase
 {
     public function testBatchTimeout()

tests/unit/Coroutine/HttpFunctionTest.php

@@ -11,6 +11,7 @@
 
 namespace Swoole\Coroutine;
 
+use PHPUnit\Framework\Attributes\CoversFunction;
 use PHPUnit\Framework\TestCase;
 use Swoole\Constant;
 use Swoole\Coroutine;
@@ -22,6 +23,8 @@
  * @internal
  * @coversNothing
  */
+#[CoversFunction('Swoole\Coroutine\Http\get')]
+#[CoversFunction('Swoole\Coroutine\Http\post')]
 class HttpFunctionTest extends TestCase
 {
     public function testGet()

tests/unit/Coroutine/WaitGroupTest.php

@@ -11,12 +11,13 @@
 
 namespace Swoole\Coroutine;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
- * @coversNothing
  */
+#[CoversClass(WaitGroup::class)]
 class WaitGroupTest extends TestCase
 {
     public function testWait()

tests/unit/Curl/HandlerTest.php

@@ -11,6 +11,8 @@
 
 namespace Swoole\Curl;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\RunTestsInSeparateProcesses;
 use PHPUnit\Framework\TestCase;
 use Swoole\Coroutine;
 use Swoole\Tests\HookFlagsTrait;
@@ -19,9 +21,9 @@
  * Class HandlerTest
  *
  * @internal
- * @coversNothing
- * @runTestsInSeparateProcesses
  */
+#[CoversClass(Handler::class)]
+#[RunTestsInSeparateProcesses]
 class HandlerTest extends TestCase
 {
     use HookFlagsTrait;
@@ -44,9 +46,6 @@ public function setUp(): void
         self::setHookFlags(SWOOLE_HOOK_CURL);
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testRedirect()
     {
         Coroutine\run(function () {
@@ -62,9 +61,6 @@ public function testRedirect()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::__toString()
-     */
     public function testToString()
     {
         Coroutine\run(function () {
@@ -73,9 +69,6 @@ public function testToString()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testCustomHost()
     {
         Coroutine\run(function () {
@@ -90,9 +83,6 @@ public function testCustomHost()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testHeaderName()
     {
         Coroutine\run(function () {
@@ -109,9 +99,6 @@ public function testHeaderName()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testWriteFunction()
     {
         Coroutine\run(function () {
@@ -132,9 +119,6 @@ public function testWriteFunction()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testResolve()
     {
         Coroutine\run(function () {
@@ -157,9 +141,6 @@ public function testResolve()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testInvalidResolve()
     {
         Coroutine\run(function () {
@@ -180,9 +161,6 @@ public function testInvalidResolve()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testResolve2()
     {
         Coroutine\run(function () {
@@ -204,9 +182,6 @@ public function testResolve2()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testInvalidResolve2()
     {
         Coroutine\run(function () {
@@ -227,9 +202,6 @@ public function testInvalidResolve2()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testInvalidResolve3()
     {
         Coroutine\run(function () {
@@ -250,9 +222,6 @@ public function testInvalidResolve3()
         });
     }
 
-    /**
-     * @covers \Swoole\Curl\Handler::execute()
-     */
     public function testResolve3()
     {
         Coroutine\run(function () {

tests/unit/Database/PDOPoolTest.php

@@ -11,6 +11,7 @@
 
 namespace Swoole\Database;
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use Swoole\Coroutine;
 use Swoole\Coroutine\WaitGroup;
 use Swoole\Tests\DatabaseTestCase;
@@ -23,8 +24,8 @@
  * Class PDOPoolTest
  *
  * @internal
- * @coversNothing
  */
+#[CoversClass(PDOPool::class)]
 class PDOPoolTest extends DatabaseTestCase
 {
     use HookFlagsTrait;

tests/unit/Database/PDOStatementProxyTest.php

@@ -11,20 +11,17 @@
 
 namespace Swoole\Database;
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use Swoole\Coroutine;
 use Swoole\Tests\DatabaseTestCase;
 
 /**
- * Class PDOStatementProxyTest
- *
  * @internal
- * @coversNothing
  */
+#[CoversClass(PDOStatementProxy::class)]
 class PDOStatementProxyTest extends DatabaseTestCase
 {
-    /**
-     * @covers \Swoole\Database\PDOStatementProxy::__call()
-     */
     public function testRun()
     {
         Coroutine\run(function () {
@@ -68,10 +65,7 @@ public static function dataSetFetchMode(): array
         ];
     }
 
-    /**
-     * @dataProvider dataSetFetchMode
-     * @covers \Swoole\Database\PDOStatementProxy::setFetchMode
-     */
+    #[DataProvider('dataSetFetchMode')]
     public function testSetFetchMode(array $expected, array $args, string $message)
     {
         Coroutine\run(function () use ($expected, $args, $message) {
@@ -89,9 +83,6 @@ public function testSetFetchMode(array $expected, array $args, string $message)
         });
     }
 
-    /**
-     * @covers \Swoole\Database\PDOStatementProxy::bindParam()
-     */
     public function testBindParam()
     {
         Coroutine\run(function () {