Ubuntu 24.04 and update test enviroment + PCOV to avoid segfault with

Xdebug 3.4.2

Author: clue

.github/workflows/ci.yml

@@ -7,7 +7,7 @@ on:
 jobs:
   PHPUnit:
     name: PHPUnit (PHP ${{ matrix.php }})
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     strategy:
       matrix:
         php:
@@ -29,7 +29,7 @@ jobs:
       - uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php }}
-          coverage: xdebug
+          coverage: ${{ matrix.php < 8.0 && 'xdebug' || 'pcov' }}
           ini-file: development
       - run: composer install
       - run: docker pull busybox:latest

composer.json

@@ -14,18 +14,18 @@
         "php": ">=5.3",
         "clue/json-stream": "^0.1",
         "react/event-loop": "^1.2",
-        "react/http": "^1.8",
-        "react/promise": "^3.1 || ^2.11 || ^1.3",
+        "react/http": "^1.11",
+        "react/promise": "^3.2 || ^2.11 || ^1.3",
         "react/promise-stream": "^1.6",
-        "react/socket": "^1.12",
-        "react/stream": "^1.2",
+        "react/socket": "^1.16",
+        "react/stream": "^1.4",
         "rize/uri-template": "^0.3"
     },
     "require-dev": {
         "clue/caret-notation": "^0.2",
         "clue/tar-react": "^0.2",
         "phpunit/phpunit": "^9.6 || ^5.7 || ^4.8.36",
-        "react/async": "^4 || ^3 || ^2"
+        "react/async": "^4.2 || ^3 || ^2"
     },
     "autoload": {
         "psr-4": {

src/Client.php

@@ -56,8 +56,12 @@ class Client
      * @param ?string        $url
      * @throws \InvalidArgumentException
      */
-    public function __construct(LoopInterface $loop = null, $url = null)
+    public function __construct($loop = null, $url = null)
     {
+        if ($loop !== null && !$loop instanceof LoopInterface) { // manual type check to support legacy PHP < 7.1
+            throw new \InvalidArgumentException('Argument #1 ($loop) expected null|React\EventLoop\LoopInterface');
+        }
+
         if ($url === null) {
             $url = 'unix:///var/run/docker.sock';
         }

tests/ClientTest.php

@@ -59,6 +59,12 @@ public function testCtorWithLoop()
         new Client($loop);
     }
 
+    public function testCtorThrowsForInvalidLoop()
+    {
+        $this->setExpectedException('InvalidArgumentException', 'Argument #1 ($loop) expected null|React\EventLoop\LoopInterface');
+        new Client('loop');
+    }
+
     public function testCtorWithInvalidUrlThrows()
     {
         $this->setExpectedException('InvalidArgumentException');

tests/FunctionalClientTest.php

@@ -478,11 +478,9 @@ public function testCreateConnectDisconnectAndRemoveNetwork()
         $promise = $this->client->events($start, $end, array('network' => array($network['Id'])));
         $ret = \React\Async\await($promise);
 
-        // expects "create", "disconnect", "destroy" events ("connect" will be skipped because we don't start the container)
-        $this->assertCount(3, $ret);
+        $this->assertCount(2, $ret);
         $this->assertEquals('create', $ret[0]['Action']);
-        $this->assertEquals('disconnect', $ret[1]['Action']);
-        $this->assertEquals('destroy', $ret[2]['Action']);
+        $this->assertEquals('destroy', $ret[1]['Action']);
     }
 
     /**