Merge pull request #57 from miladrahimi/update-laminas

fix http error already sent in tests

Author: miladrahimi

.github/workflows/ci.yml

@@ -1,26 +1,36 @@
 name: CI
-on: [push, pull_request]
+
+on:
+  pull_request:
+    types: [opened, synchronize, reopened]
+
 jobs:
   run:
     strategy:
       matrix:
         include:
-          - php: '7.4'
           - php: '8.0'
           - php: '8.1'
           - php: '8.2'
+          - php: '8.3'
+          - php: '8.4'
+
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v4
+
       - name: Set up PHP
         uses: shivammathur/setup-php@v2
         with:
           php-version: "${{ matrix.php }}"
+
       - name: Install dependencies
         run: composer self-update && composer install && composer dump-autoload
+
       - name: Run tests and collect coverage
         run: vendor/bin/phpunit --coverage-clover coverage.xml .
+
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v4-beta
         env:

tests/Features/ViewTest.php

@@ -10,6 +10,7 @@
 class ViewTest extends TestCase
 {
     /**
+     * @runInSeparateProcess
      * @throws Throwable
      */
     public function test_with_the_sample_view()
@@ -28,6 +29,7 @@ public function test_with_the_sample_view()
     }
 
     /**
+     * @runInSeparateProcess
      * @throws Throwable
      */
     public function test_with_the_sample_view_and_status_201_and_headers()

tests/Units/HttpPublisherTest.php

@@ -58,6 +58,7 @@ public function test_publish_a_array_response()
     }
 
     /**
+     * @runInSeparateProcess
      * @throws Throwable
      */
     public function test_publish_a_standard_response()