Update test suite to ensure 100% code coverage

Author: clue

.github/workflows/ci.yml

@@ -33,7 +33,16 @@ jobs:
           php-version: ${{ matrix.php }}
           coverage: xdebug
       - run: composer install
-      - run: vendor/bin/phpunit --coverage-text ${{ matrix.php < 7.3 && '-c phpunit.xml.legacy' || '' }}
+      - run: vendor/bin/phpunit --coverage-text --coverage-clover clover.xml ${{ matrix.php < 7.3 && '-c phpunit.xml.legacy' || '' }}
+      - name: Check 100% code coverage
+        shell: php {0}
+        run: |
+          <?php
+          $metrics = simplexml_load_file('clover.xml')->project->metrics;
+          if ((int) $metrics['statements'] !== (int) $metrics['coveredstatements']) {
+              echo file_get_contents('clover.xml');
+              exit(1);
+          }
 
   PHPUnit-hhvm:
     name: PHPUnit (HHVM)

README.md

@@ -1,6 +1,7 @@
 # clue/reactphp-http-proxy
 
 [![CI status](https://github.com/clue/reactphp-http-proxy/actions/workflows/ci.yml/badge.svg)](https://github.com/clue/reactphp-http-proxy/actions)
+[![code coverage](https://img.shields.io/badge/code%20coverage-100%25-success)](#tests)
 [![installs on Packagist](https://img.shields.io/packagist/dt/clue/http-proxy-react?color=blue&label=installs%20on%20Packagist)](https://packagist.org/packages/clue/http-proxy-react)
 
 Async HTTP proxy connector, tunnel any TCP/IP-based protocol through an HTTP
@@ -483,6 +484,14 @@ alternatively you can also run it like this:
 vendor/bin/phpunit --exclude-group internet
 ```
 
+The test suite is set up to always ensure 100% code coverage across all
+supported environments. If you have the Xdebug extension installed, you can also
+generate a code coverage report locally like this:
+
+```bash
+XDEBUG_MODE=coverage vendor/bin/phpunit --coverage-text
+```
+
 ## License
 
 This project is released under the permissive [MIT license](LICENSE).

clover56.xml

@@ -32,5 +32,10 @@
         "psr-4": { 
             "Clue\\Tests\\React\\HttpProxy\\": "tests/" 
         }
+    },
+    "config": {
+        "platform": {
+            "php": "5.6.0"
+        }
     }
 }

composer.json

@@ -0,0 +1,36 @@
+{
+    "name": "clue/http-proxy-react",
+    "description": "Async HTTP proxy connector, tunnel any TCP/IP-based protocol through an HTTP CONNECT proxy server, built on top of ReactPHP",
+    "keywords": ["HTTP", "CONNECT", "proxy", "ReactPHP", "async"],
+    "homepage": "https://github.com/clue/reactphp-http-proxy",
+    "license": "MIT",
+    "authors": [
+        {
+            "name": "Christian Lück",
+            "email": "christian@clue.engineering"
+        }
+    ],
+    "require": {
+        "php": ">=5.3",
+        "react/promise": "^3.3 || ^2.1 || ^1.2.1",
+        "react/socket": "^1.17",
+        "ringcentral/psr7": "^1.2"
+    },
+    "require-dev": {
+        "phpunit/phpunit": "^9.6 || ^8.5 || ^5.7 || ^4.8.36",
+        "react/async": "^4.3 || ^3 || ^2",
+        "react/event-loop": "^1.6",
+        "react/http": "^1.11",
+        "react/promise-timer": "^1.11"
+    },
+    "autoload": {
+        "psr-4": { 
+            "Clue\\React\\HttpProxy\\": "src/" 
+        }
+    },
+    "autoload-dev": {
+        "psr-4": { 
+            "Clue\\Tests\\React\\HttpProxy\\": "tests/" 
+        }
+    }
+}

composer.json.bak

@@ -262,7 +262,7 @@ public function connect($uri)
             // what a lovely piece of code!
             $r = new \ReflectionProperty('Exception', 'trace');
             if (PHP_VERSION_ID < 80100) {
-                $r->setAccessible(true);
+                $r->setAccessible(true); // @codeCoverageIgnore
             }
             $trace = $r->getValue($e);