Merge pull request #43 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

@@ -16,7 +16,7 @@ jobs:
           - 8.0
           - 7.4
           - 7.3
-        # - 7.2 # skip temporarily due to version conflicts
+          - 7.2
           - 7.1
           - 7.0
           - 5.6
@@ -32,10 +32,7 @@ jobs:
       - run: composer remove react/mysql --dev --no-interaction # do not install react/mysql example on legacy PHP
         if: ${{ matrix.php == 5.3 }}
       - 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

@@ -20,7 +20,7 @@
         "react/stream": "^1.2"
     },
     "require-dev": {
-        "phpunit/phpunit": "^9.6 || ^5.7 || ^4.8.36",
+        "phpunit/phpunit": "^9.6 || ^8.5 || ^5.7 || ^4.8.36",
         "react/async": "^4 || ^3 || ^2",
         "react/http": "^1.5",
         "react/mysql": "^0.5.5",