Merge pull request #21 from aternosorg/align-tests-workflow

Collect code coverage of the client in unit tests

Author: pavog

.github/workflows/tests.yaml

@@ -27,7 +27,7 @@ jobs:
         with:
           php-version: ${{ matrix.php-version }}
           extensions: curl, json, mbstring
-          coverage: none
+          coverage: pcov
           tools: composer:v2
 
       - name: Set composer cache directory
@@ -46,8 +46,8 @@ jobs:
       - name: Install composer dependencies
         run: composer install --no-interaction --prefer-dist --no-progress
 
-      - name: Run phpunit unit tests
-        run: vendor/bin/phpunit --testsuite=unit --colors=always --testdox
+      - name: Run phpunit unit tests with coverage
+        run: vendor/bin/phpunit --testsuite=unit --colors=always --testdox --coverage-clover=coverage.xml
 
   # integration tests send requests to the live CurseForge API and need the CURSEFORGE_API_KEY secret,
   # so they only run on the master branch and when started manually

phpunit.xml.dist

@@ -1,27 +1,27 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="./vendor/phpunit/phpunit/phpunit.xsd"
-         bootstrap="./vendor/autoload.php"
+         bootstrap="vendor/autoload.php"
          colors="true"
          executionOrder="random"
          displayDetailsOnTestsThatTriggerDeprecations="true"
          displayDetailsOnTestsThatTriggerErrors="true"
          displayDetailsOnTestsThatTriggerNotices="true"
          displayDetailsOnTestsThatTriggerWarnings="true">
-  <source>
-    <include>
-      <directory>src</directory>
-    </include>
-  </source>
-  <testsuites>
-    <testsuite name="integration">
-      <directory>./tests/Integration/Client</directory>
-    </testsuite>
-    <testsuite name="unit">
-      <directory>tests/Unit/Client</directory>
-    </testsuite>
-  </testsuites>
-  <php>
-    <ini name="error_reporting" value="E_ALL"/>
-  </php>
+    <testsuites>
+        <testsuite name="unit">
+            <directory>tests/Unit/Client</directory>
+        </testsuite>
+        <testsuite name="integration">
+            <directory>tests/Integration/Client</directory>
+        </testsuite>
+    </testsuites>
+    <source>
+        <include>
+            <directory>src/Client</directory>
+        </include>
+    </source>
+    <php>
+        <ini name="error_reporting" value="E_ALL"/>
+    </php>
 </phpunit>