fix: separate test commands for coverage and non-coverage runs

- Add --no-coverage flag to default test command to avoid warnings
- Create separate test:coverage command for CI/CD coverage generation
- Remove failOnWarning from phpunit.xml.dist to prevent non-critical warnings from failing tests
- Update GitHub Actions to use appropriate test command based on coverage needs
- Fixes exit code 1 when running tests locally without xdebug/pcov installed

Author: frknasir

.github/workflows/tests.yml

@@ -41,9 +41,14 @@ jobs:
     - name: Install dependencies
       run: composer update --${{ matrix.stability }} --prefer-dist --no-interaction
 
-    - name: Execute tests
+    - name: Execute tests (without coverage)
+      if: matrix.php != '8.2' || matrix.stability != 'prefer-stable'
       run: composer test
 
+    - name: Execute tests (with coverage)
+      if: matrix.php == '8.2' && matrix.stability == 'prefer-stable'
+      run: composer test:coverage
+
     - name: Upload coverage to Codecov
       if: matrix.php == '8.2' && matrix.stability == 'prefer-stable'
       uses: codecov/codecov-action@v3

composer.json

@@ -44,7 +44,8 @@
         "symfony/var-dumper": "^6.2"
     },
     "scripts": {
-        "test": "phpunit --testdox --coverage-text --coverage-clover=coverage.clover"
+        "test": "phpunit --testdox --no-coverage",
+        "test:coverage": "phpunit --testdox --coverage-text --coverage-clover=coverage.clover"
     },
     "minimum-stability": "stable",
     "prefer-stable": true,

phpunit.xml.dist

@@ -7,7 +7,6 @@
     processIsolation="false" 
     stopOnFailure="false" 
     executionOrder="random" 
-    failOnWarning="true" 
     failOnRisky="true" 
     failOnEmptyTestSuite="true" 
     beStrictAboutOutputDuringTests="true"