Fix tests workflow for PSR7 libraries

williamdes · williamdes · commit 3b4b4e287aba · 2023-05-06T11:48:41.000+02:00
Ref: 37bd7d9 Signed-off-by: William Desportes <williamdes@wdes.fr>
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -191,15 +191,19 @@ jobs:
           coverage-reports: build/logs/clover.xml
 
   test-php-psr7-implementations:
-    name: Test on PHP (+ psr7-${{ matrix.psr-7-library }}) ${{ matrix.php-version }} and ${{ matrix.os }}
+    name: Test on PHP ${{ matrix.php-version }} (+ psr7-${{ matrix.psr-7-library }})
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
       matrix:
-        php-version: ['7.3']
+        php-version: ['7.2']
         os: [ubuntu-latest]
         php-extensions: ['mbstring, iconv, mysqli, zip, bz2']
-        psr-7-library: ['guzzlehttp/psr7', 'nyholm/psr7', 'laminas/laminas-diactoros']
+        psr-7-library: ['guzzlehttp/psr7']
+        include:
+          - {psr-7-library: 'laminas/laminas-diactoros', php-version: '7.4'}
+          # https://github.com/slimphp/Slim-Psr7/issues/284
+          - {psr-7-library: 'slim/psr7', php-version: '7.3'}
     steps:
       - name: Checkout code
         uses: actions/checkout@v3
@@ -226,7 +230,7 @@ jobs:
           dependency-versions: highest
 
       - name: Remove default PSR-7 implementation
-        run: composer remove slim/psr7
+        run: composer remove nyholm/psr7
 
       - name: Add PSR-7 implementation
         run: composer require ${{ matrix.psr-7-library }}
