Test test suite

Author: SimonFrings

Diff for: .github/workflows/ci.yml

@@ -7,10 +7,11 @@ on:
 jobs:
   PHPUnit:
     name: PHPUnit (PHP ${{ matrix.php }})
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-22.04
     strategy:
       matrix:
         php:
+          - 8.3
           - 8.2
           - 8.1
           - 8.0
@@ -24,7 +25,7 @@ jobs:
           - 5.4
           - 5.3
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php }}
@@ -39,13 +40,16 @@ jobs:
 
   PHPUnit-hhvm:
     name: PHPUnit (HHVM)
-    runs-on: ubuntu-18.04
+    runs-on: ubuntu-22.04
     continue-on-error: true
     steps:
-      - uses: actions/checkout@v2
-      - uses: azjezz/setup-hhvm@v1
+      - uses: actions/checkout@v4
+      - run: cp "$(which composer)" composer.phar && ./composer.phar self-update --2.2 # downgrade Composer for HHVM
+      - name: Run hhvm composer.phar install
+        uses: docker://hhvm/hhvm:3.30-lts-latest
         with:
-          version: lts-3.30
-      - run: composer self-update --2.2 # downgrade Composer for HHVM
-      - run: hhvm $(which composer) install
-      - run: hhvm vendor/bin/phpunit
+          args: hhvm composer.phar install
+      - name: Run hhvm vendor/bin/phpunit
+        uses: docker://hhvm/hhvm:3.30-lts-latest
+        with:
+          args: hhvm vendor/bin/phpunit

Diff for: composer.json

@@ -26,10 +26,16 @@
         "react/mysql": "^0.5.5"
     },
     "autoload": {
-        "psr-4": { "Clue\\React\\SshProxy\\": "src/" },
-        "files": [ "src/Io/functions.php" ]
+        "psr-4": {
+            "Clue\\React\\SshProxy\\": "src/"
+        },
+        "files": [
+            "src/Io/functions.php"
+        ]
     },
     "autoload-dev": {
-        "psr-4": { "Clue\\Tests\\React\\SshProxy\\": "tests/"} 
+        "psr-4": {
+            "Clue\\Tests\\React\\SshProxy\\": "tests/"
+        }
     }
 }

Diff for: phpunit.xml.dist

@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<!-- PHPUnit configuration file with new format for PHPUnit 9.3+ -->
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" 
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
+<!-- PHPUnit configuration file with new format for PHPUnit 9.6+ -->
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.6/phpunit.xsd"
          bootstrap="vendor/autoload.php"
          cacheResult="false"
          colors="true"
@@ -17,4 +17,7 @@
             <directory>./src/</directory>
         </include>
     </coverage>
+    <php>
+        <ini name="error_reporting" value="-1" />
+    </php>
 </phpunit>

Diff for: phpunit.xml.legacy

@@ -15,4 +15,7 @@
             <directory>./src/</directory>
         </whitelist>
     </filter>
+    <php>
+        <ini name="error_reporting" value="-1" />
+    </php>
 </phpunit>

Diff for: src/Io/CompositeConnection.php

@@ -11,6 +11,7 @@
 /** @internal */
 class CompositeConnection extends EventEmitter implements ConnectionInterface
 {
+    private $remote;
     private $read;
     private $write;
     private $closed = false;

Diff for: tests/FunctionalSshSocksConnectorTest.php

@@ -23,15 +23,6 @@ public function setUpConnector()
         $this->connector = new SshSocksConnector($url);
     }
 
-    /**
-     * @after
-     */
-    public function tearDownSSHClientProcess()
-    {
-        // run loop in order to shut down SSH client process again
-        \React\Async\await(\React\Promise\Timer\sleep(0.001));
-    }
-
     public function testConnectInvalidProxyUriWillReturnRejectedPromise()
     {
         $this->connector = new SshSocksConnector(getenv('SSH_PROXY') . '.invalid');