Fix code coverage reports

tvdijen · tvdijen · commit 35e78c054fdd · 2026-03-23T22:28:57.000+01:00
diff --git a/.github/workflows/php.yml b/.github/workflows/php.yml
@@ -55,7 +55,7 @@ jobs:
           extensions: intl, mbstring, mysql, pdo, pdo_sqlite, soap, sodium, xml
           tools: composer:v2
           ini-values: error_reporting=E_ALL
-          coverage: xdebug
+          coverage: pcov
 
       - name: Setup problem matchers for PHP
         run: echo "::add-matcher::${{ runner.tool_cache }}/php.json"
@@ -91,11 +91,11 @@ jobs:
 
       - name: Run unit tests with coverage
         if: ${{ matrix.php-versions == '8.5' }}
-        run: ./vendor/bin/phpunit
+        run: vendor/bin/phpunit
 
       - name: Run unit tests (no coverage)
         if: ${{ matrix.php-versions != '8.5' }}
-        run: ./vendor/bin/phpunit --no-coverage
+        run: vendor/bin/phpunit --no-coverage
 
       - name: Save coverage data
         if: ${{ matrix.php-versions == '8.5' }}
@@ -272,6 +272,8 @@ jobs:
     runs-on: [ubuntu-latest]
     needs: [unit-tests-linux]
     steps:
+      - uses: actions/checkout@v6
+
       - uses: actions/download-artifact@v8
         with:
           name: coverage-data
diff --git a/composer.json b/composer.json
@@ -25,6 +25,7 @@
             "SimpleSAML\\Module\\admin\\": "modules/admin/src",
             "SimpleSAML\\Module\\core\\": "modules/core/src",
             "SimpleSAML\\Module\\cron\\": "modules/cron/src",
+            "SimpleSAML\\Module\\debugsp\\": "modules/debugsp/src",
             "SimpleSAML\\Module\\exampleauth\\": "modules/exampleauth/src",
             "SimpleSAML\\Module\\multiauth\\": "modules/multiauth/src",
             "SimpleSAML\\Module\\saml\\": "modules/saml/src"
@@ -37,6 +38,7 @@
             "SimpleSAML\\Test\\Module\\admin\\": ["tests//modules/admin/src"],
             "SimpleSAML\\Test\\Module\\core\\": ["tests/modules/core/src"],
             "SimpleSAML\\Test\\Module\\cron\\": ["tests/modules/cron/src"],
+            "SimpleSAML\\Test\\Module\\debugsp\\": ["tests/modules/debugsp/src"],
             "SimpleSAML\\Test\\Module\\exampleauth\\": ["tests/modules/exampleauth/src"],
             "SimpleSAML\\Test\\Module\\multiauth\\": ["tests/modules/multiauth/src"],
             "SimpleSAML\\Test\\Module\\saml\\": ["tests/modules/saml/src"]
diff --git a/phpunit.xml b/phpunit.xml
@@ -3,8 +3,6 @@
   <coverage>
     <report>
       <clover outputFile="build/logs/clover.xml"/>
-      <html outputDirectory="build/coverage" lowUpperBound="35" highLowerBound="70"/>
-      <text outputFile="php://stdout" showUncoveredFiles="true"/>
     </report>
   </coverage>
   <testsuites>
@@ -16,7 +14,13 @@
   <logging/>
   <source>
     <include>
-      <directory suffix=".php">./modules/*/src</directory>
+      <directory suffix=".php">./modules/admin/src</directory>
+      <directory suffix=".php">./modules/core/src</directory>
+      <directory suffix=".php">./modules/cron/src</directory>
+      <directory suffix=".php">./modules/debugsp/src</directory>
+      <directory suffix=".php">./modules/exampleauth/src</directory>
+      <directory suffix=".php">./modules/multiauth/src</directory>
+      <directory suffix=".php">./modules/saml/src</directory>
       <directory suffix=".php">./src</directory>
     </include>
     <exclude>
