CI: Add code coverage

Author: GaryJones

.github/workflows/integrations.yml

@@ -84,4 +84,17 @@ jobs:
         run: composer prepare-ci --no-interaction
 
       - name: Run integration tests
+        if: ${{ matrix.coverage == false }}
         run: composer test --no-interaction
+
+      - name: Run integration tests with code coverage
+        if: ${{ matrix.coverage == true  }}
+        run: composer coverage-ci --no-interaction
+
+      - name: Send coverage report to Codecov
+        if: ${{ success() && matrix.coverage == true }}
+        uses: codecov/codecov-action@v3
+        with:
+          files: ./clover.xml
+          fail_ci_if_error: true
+          verbose: true

.gitignore

@@ -1,4 +1,5 @@
-/.phpunit.result.cache
+/.phpunit.cache
+/clover.xml
 /composer.lock
 /package-lock.json
 /vendor

composer.json

@@ -33,7 +33,7 @@
 			"@php ./vendor/bin/phpcbf"
 		],
 		"coverage": [
-			"@php ./vendor/bin/phpunit --coverage-html ./build/coverage-html"
+			"@php ./vendor/bin/phpunit --coverage-html ./.phpunit.cache/coverage-html"
 		],
 		"coverage-ci": [
 			"@php ./vendor/bin/phpunit"

phpunit.xml.dist

@@ -1,32 +1,33 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit
-		xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-		xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.5/phpunit.xsd"
-		bootstrap="./tests/Integration/bootstrap.php"
-		backupGlobals="false"
-		beStrictAboutCoversAnnotation="true"
-		beStrictAboutTestsThatDoNotTestAnything="true"
-		colors="true"
-		convertDeprecationsToExceptions="true"
-		testdox="true"
->
-	<php>
-		<server name="SERVER_PORT" value="80"/>
-		<server name="HTTP_HOST" value="localhost"/>
-		<server name="REMOTE_ADDR" value="127.0.0.1"/>
-	</php>
-	<testsuites>
-		<testsuite name="integration">
-			<directory>./tests/Integration</directory>
-		</testsuite>
-	</testsuites>
-	<coverage>
-		<include>
-			<directory suffix=".php">src</directory>
-			<file>ad-code-manager.php</file>
-		</include>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.6/phpunit.xsd"
+         bootstrap="./tests/Integration/bootstrap.php"
+         cacheResultFile=".phpunit.cache/test-results"
+         executionOrder="depends,defects"
+         forceCoversAnnotation="false"
+         beStrictAboutCoversAnnotation="true"
+         beStrictAboutOutputDuringTests="true"
+         beStrictAboutTodoAnnotatedTests="true"
+		 colors="true"
+         convertDeprecationsToExceptions="true"
+         failOnRisky="true"
+         failOnWarning="true"
+		 testdox="true"
+         verbose="true">
+    <testsuites>
+        <testsuite name="integration">
+            <directory>tests/Integration</directory>
+        </testsuite>
+    </testsuites>
+
+    <coverage cacheDirectory=".phpunit.cache/code-coverage"
+              processUncoveredFiles="true">
+        <include>
+            <directory suffix=".php">src</directory>
+        </include>
 		<report>
 			<text outputFile="php://stdout"/>
+			<clover outputFile="clover.xml"/>
 		</report>
-	</coverage>
+    </coverage>
 </phpunit>

tests/Integration/UI/PluginActionsTest.php

@@ -22,6 +22,15 @@ final class PluginActionsTest extends TestCase {
 	 * Check that plugins screen will add a hook to change the plugin action links.
 	 *
 	 * @covers \Automattic\AdCodeManager\UI\Plugin_Actions::run
+	 * @uses ACM_Provider::__construct
+	 * @uses Ad_Code_Manager::action_init
+	 * @uses Ad_Code_Manager::action_load_providers
+ 	 * @uses Ad_Code_Manager::get_ad_codes
+	 * @uses Ad_Code_Manager::get_option
+	 * @uses Ad_Code_Manager::get_options
+	 * @uses Ad_Code_Manager::register_acm_post_type
+	 * @uses Ad_Code_Manager::register_ad_codes
+	 * @uses Doubleclick_For_Publishers_ACM_Provider::__construct
 	 * @group ui
 	 */
 	public function test_plugins_screen_has_filter_to_add_a_settings_action_link(): void {
@@ -36,6 +45,15 @@ public function test_plugins_screen_has_filter_to_add_a_settings_action_link():
 	 *
 	 * @covers Automattic\AdCodeManager\UI\Plugin_Actions::run
 	 * @covers Automattic\AdCodeManager\UI\Plugin_Actions::add_plugin_meta_links
+	 * @uses ACM_Provider::__construct
+	 * @uses Ad_Code_Manager::action_init
+	 * @uses Ad_Code_Manager::action_load_providers
+ 	 * @uses Ad_Code_Manager::get_ad_codes
+	 * @uses Ad_Code_Manager::get_option
+	 * @uses Ad_Code_Manager::get_options
+	 * @uses Ad_Code_Manager::register_acm_post_type
+	 * @uses Ad_Code_Manager::register_ad_codes
+	 * @uses Doubleclick_For_Publishers_ACM_Provider::__construct
 	 * @group ui
 	 */
 	public function test_plugins_screen_adds_a_settings_action_link(): void {