Merge pull request #26 from clue-labs/phpunit8

Update test environment to fix tests on legacy PHP 7.2 with PHPUnit 8.5

Author: clue

.github/workflows/ci.yml

@@ -32,10 +32,7 @@ jobs:
           php-version: ${{ matrix.php }}
           coverage: xdebug
       - run: composer install
-      - run: vendor/bin/phpunit --coverage-text
-        if: ${{ matrix.php >= 7.3 }}
-      - run: vendor/bin/phpunit --coverage-text -c phpunit.xml.legacy
-        if: ${{ matrix.php < 7.3 }}
+      - run: vendor/bin/phpunit --coverage-text ${{ matrix.php < 7.3 && '-c phpunit.xml.legacy' || '' }}
 
   PHPUnit-hhvm:
     name: PHPUnit (HHVM)

composer.json

@@ -17,7 +17,7 @@
     },
     "require-dev": {
         "clue/hexdump": "0.2.*",
-        "phpunit/phpunit": "^9.6 ||^6.0 || ^5.7 || ^4.8.36"
+        "phpunit/phpunit": "^9.6 || ^8.5 || ^5.7 || ^4.8.36"
     },
     "suggest": {
         "ext-sockets": "Requires PHP 5.4+ and the low level socket API for listening on multicast addresses (socket options to send IGMP announcements)"