From 666f9540845e109996a84671725d29c9b5459487 Mon Sep 17 00:00:00 2001
From: Konrad Windszus <kwin@apache.org>
Date: Mon, 21 Jul 2025 12:17:58 +0200
Subject: [PATCH] JCR-5173 Create aggregated coverage report

JCR-5172 Exclude jackrabbit-jcr-tests from coverage calculation
---
 jackrabbit-parent/pom.xml | 33 +++++++++++++++++++++++++++++++++
 pom.xml                   | 17 +++++++++++++++++
 2 files changed, 50 insertions(+)

diff --git a/jackrabbit-parent/pom.xml b/jackrabbit-parent/pom.xml
index 5173333c52b..9912adeaf48 100644
--- a/jackrabbit-parent/pom.xml
+++ b/jackrabbit-parent/pom.xml
@@ -185,6 +185,39 @@
       <properties>
         <skip.coverage>false</skip.coverage>
       </properties>
+      <build>
+        <pluginManagement>
+          <plugins>
+            <!-- generates aggregate (cross-module) coverage reports, https://github.com/olamy/jacoco-aggregator-maven-plugin
+                only enable this plugin (without inheritance) in the root module, as this is an aggregator plugin
+             -->
+            <plugin>
+              <groupId>io.github.olamy.maven.plugins</groupId>
+              <artifactId>jacoco-aggregator-maven-plugin</artifactId>
+              <version>1.0.3</version>
+              <configuration>
+                <includes>
+                  <!-- exclude shaded 3rd party classes -->
+                  <include>org/apache/jackrabbit/**</include>
+                </includes>
+                <excludes>
+                  <!-- exclude classes in jackrabbit-jcr-tests -->
+                  <exclude>org/apache/jackrabbit/test/**</exclude>
+                </excludes>
+              </configuration>
+              <executions>
+                <execution>
+                  <id>aggregate</id>
+                  <phase>verify</phase>
+                  <goals>
+                    <goal>report-aggregate-all</goal>
+                  </goals>
+                </execution>
+              </executions>
+            </plugin>
+          </plugins>
+        </pluginManagement>
+      </build>
     </profile>
   </profiles>
 
diff --git a/pom.xml b/pom.xml
index b5264e64f87..375583509cf 100644
--- a/pom.xml
+++ b/pom.xml
@@ -35,6 +35,11 @@
   <name>Apache Jackrabbit</name>
   <packaging>pom</packaging>
 
+  <properties>
+    <!-- this project uses cross-module reports with one aggregate, https://docs.sonarsource.com/sonarcloud/enriching/test-coverage/java-test-coverage/#add-coverage-in-a-multi-module-maven-project -->
+    <sonar.coverage.jacoco.xmlReportPaths>${project.basedir}/target/site/jacoco-aggregate/jacoco.xml</sonar.coverage.jacoco.xmlReportPaths>
+  </properties>
+
   <modules>
     <module>jackrabbit-parent</module>
     <module>jackrabbit-jcr-commons</module>
@@ -87,6 +92,18 @@
   </build>
 
   <profiles>
+    <profile>
+      <id>coverage</id>
+      <build>
+        <plugins>
+          <plugin>
+            <groupId>io.github.olamy.maven.plugins</groupId>
+            <artifactId>jacoco-aggregator-maven-plugin</artifactId>
+            <inherited>false</inherited><!-- only execute on root level (as aggregator)-->
+          </plugin>
+        </plugins>
+      </build>
+    </profile>
     <profile>
       <id>apache-release</id>
       <properties>