Fix sonar coverage

focus-shift · Mar 6, 2024 · 78e8e8c · 78e8e8c
1 parent f91e1a3
commit 78e8e8c
Show file tree

Hide file tree

Showing 7 changed files with 72 additions and 20 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -36,8 +36,7 @@ jobs:
       - name: Analyse with SonarCloud
         if: ${{ github.repository == 'focus-shift/jollyday' && matrix.java == 17 && matrix.os == 'ubuntu-latest' && (github.event_name != 'pull_request' || (github.event.pull_request.head.repo.full_name == github.repository && github.actor != 'dependabot[bot]')) }}
         run: >
-          ./mvnw -Pcoverage verify
-          sonar:sonar
+          ./mvnw -Pcoverage sonar:sonar
           -Dsonar.host.url=https://sonarcloud.io
           -Dsonar.organization=focus-shift
           -Dsonar.projectKey=focus-shift_jollyday

diff --git a/jollyday-core/pom.xml b/jollyday-core/pom.xml
@@ -3,6 +3,7 @@
   <modelVersion>4.0.0</modelVersion>
 
   <artifactId>jollyday-core</artifactId>
+
   <name>Jollyday Core</name>
   <description>
     Core module of the jollyday library that contains the API for the developers and

diff --git a/jollyday-jackson/pom.xml b/jollyday-jackson/pom.xml
@@ -3,8 +3,8 @@
   <modelVersion>4.0.0</modelVersion>
 
   <artifactId>jollyday-jackson</artifactId>
-  <name>Jollyday with Jackson</name>
 
+  <name>Jollyday with Jackson</name>
   <description>Jackson based jollyday implementation</description>
 
   <parent>

diff --git a/jollyday-jaxb/pom.xml b/jollyday-jaxb/pom.xml
@@ -3,8 +3,8 @@
   <modelVersion>4.0.0</modelVersion>
 
   <artifactId>jollyday-jaxb</artifactId>
-  <name>Jollyday with Jakarta XML Binding (JAXB)</name>
 
+  <name>Jollyday with Jakarta XML Binding (JAXB)</name>
   <description>Jakarta XML Binding (JAXB) based jollyday implementation</description>
 
   <parent>

diff --git a/jollyday-tests/pom.xml b/jollyday-tests/pom.xml
@@ -3,6 +3,7 @@
   <modelVersion>4.0.0</modelVersion>
 
   <artifactId>jollyday-tests</artifactId>
+
   <name>Jollyday Tests</name>
 
   <parent>
@@ -93,20 +94,6 @@
           <skip>true</skip>
         </configuration>
       </plugin>
-
-      <plugin>
-        <groupId>org.jacoco</groupId>
-        <artifactId>jacoco-maven-plugin</artifactId>
-        <executions>
-          <execution>
-            <id>report</id>
-            <goals>
-              <goal>report-aggregate</goal>
-            </goals>
-            <phase>verify</phase>
-          </execution>
-        </executions>
-      </plugin>
     </plugins>
   </build>
 </project>
diff --git a/pom.xml b/pom.xml
@@ -46,7 +46,7 @@
 
     <!--suppress UnresolvedMavenProperty - for IDEA-->
     <sonar.coverage.jacoco.xmlReportPaths>
-      ${maven.multiModuleProjectDirectory}/jollyday-tests/target/site/jacoco-aggregate/jacoco.xml
+      ${project.basedir}/target/site/jacoco-aggregate/jacoco.xml
     </sonar.coverage.jacoco.xmlReportPaths>
   </properties>
 
@@ -294,14 +294,24 @@
           <plugin>
             <groupId>org.jacoco</groupId>
             <artifactId>jacoco-maven-plugin</artifactId>
-            <version>0.8.11</version>
             <executions>
               <execution>
                 <id>prepare-agent</id>
                 <goals>
                   <goal>prepare-agent</goal>
                 </goals>
               </execution>
+              <execution>
+                <id>report</id>
+                <goals>
+                  <goal>report</goal>
+                </goals>
+                <configuration>
+                  <formats>
+                    <format>XML</format>
+                  </formats>
+                </configuration>
+              </execution>
             </executions>
           </plugin>
         </plugins>
@@ -369,6 +379,7 @@
       </activation>
       <modules>
         <module>jollyday-tests</module>
+        <module>report-aggregate</module>
       </modules>
     </profile>
 

diff --git a/report-aggregate/pom.xml b/report-aggregate/pom.xml
@@ -0,0 +1,54 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+  <modelVersion>4.0.0</modelVersion>
+  <packaging>pom</packaging>
+
+  <artifactId>report-aggregate</artifactId>
+  <name>Aggregate Coverage Report</name>
+
+  <description>Aggregate Coverage Report</description>
+
+  <parent>
+    <groupId>de.focus-shift</groupId>
+    <artifactId>jollyday</artifactId>
+    <version>0.27.0-SNAPSHOT</version>
+  </parent>
+
+  <dependencies>
+    <dependency>
+      <groupId>de.focus-shift</groupId>
+      <artifactId>jollyday-core</artifactId>
+      <version>${project.version}</version>
+    </dependency>
+    <dependency>
+      <groupId>de.focus-shift</groupId>
+      <artifactId>jollyday-jackson</artifactId>
+      <version>${project.version}</version>
+    </dependency>
+    <dependency>
+      <groupId>de.focus-shift</groupId>
+      <artifactId>jollyday-jaxb</artifactId>
+      <version>${project.version}</version>
+    </dependency>
+  </dependencies>
+
+  <build>
+    <plugins>
+      <plugin>
+        <groupId>org.jacoco</groupId>
+        <artifactId>jacoco-maven-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>report-aggregate</id>
+            <phase>verify</phase>
+            <goals>
+              <goal>report-aggregate</goal>
+            </goals>
+          </execution>
+        </executions>
+      </plugin>
+    </plugins>
+  </build>
+</project>