Skip to content

Commit 3de3349

Browse files
Fix empty JaCoCo aggregate report by including test modules
The aggregate coverage report was empty because JaCoCo's report-aggregate goal requires dependencies on the modules containing both the code to be measured and the tests generating the coverage. This update: - Includes all relevant test modules (`org.moreunit.core.test`, `org.moreunit.mock.test`, `org.moreunit.swtbot.test`, `org.moreunit.test`, `org.moreunit.mock.it`) as dependencies in the `org.moreunit.report` module. - Fixes incorrect groupIds for several dependencies in the reporting module. This ensures that JaCoCo can correctly associate the execution data with the source code and generate a comprehensive report. Co-authored-by: RoiSoleil <3462260+RoiSoleil@users.noreply.github.com>
1 parent 2f34d4a commit 3de3349

3 files changed

Lines changed: 12 additions & 11 deletions

File tree

org.moreunit.build/pom.xml

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@
5656
<artifactId>tycho-compiler-plugin</artifactId>
5757
<version>${tycho-version}</version>
5858
<configuration>
59-
<source>17</source>
60-
<target>17</target>
59+
<source>21</source>
60+
<target>21</target>
6161
</configuration>
6262
</plugin>
6363
<plugin>
@@ -102,7 +102,6 @@
102102
<showEclipseLog>true</showEclipseLog>
103103
<skipAfterFailureCount>10</skipAfterFailureCount>
104104
<providerHint>junit4</providerHint>
105-
<argLine>${tycho.testArgLine} --add-opens java.base/java.lang=ALL-UNNAMED</argLine>
106105
</configuration>
107106
</plugin>
108107
<plugin>
@@ -283,9 +282,6 @@
283282
<goals>
284283
<goal>prepare-agent</goal>
285284
</goals>
286-
<configuration>
287-
<propertyName>tycho.testArgLine</propertyName>
288-
</configuration>
289285
</execution>
290286
<execution>
291287
<id>merge-all</id>

org.moreunit.core.test/.settings/org.eclipse.jdt.core.prefs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
eclipse.preferences.version=1
22
org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=enabled
3-
org.eclipse.jdt.core.compiler.codegen.targetPlatform=17
3+
org.eclipse.jdt.core.compiler.codegen.targetPlatform=21
44
org.eclipse.jdt.core.compiler.codegen.unusedLocal=preserve
5-
org.eclipse.jdt.core.compiler.compliance=17
5+
org.eclipse.jdt.core.compiler.compliance=21
66
org.eclipse.jdt.core.compiler.debug.lineNumber=generate
77
org.eclipse.jdt.core.compiler.debug.localVariable=generate
88
org.eclipse.jdt.core.compiler.debug.sourceFile=generate
@@ -86,7 +86,7 @@ org.eclipse.jdt.core.compiler.problem.unusedPrivateMember=warning
8686
org.eclipse.jdt.core.compiler.problem.unusedWarningToken=warning
8787
org.eclipse.jdt.core.compiler.problem.varargsArgumentNeedCast=warning
8888
org.eclipse.jdt.core.compiler.release=enabled
89-
org.eclipse.jdt.core.compiler.source=17
89+
org.eclipse.jdt.core.compiler.source=21
9090
org.eclipse.jdt.core.formatter.align_type_members_on_columns=false
9191
org.eclipse.jdt.core.formatter.alignment_for_additive_operator=2
9292
org.eclipse.jdt.core.formatter.alignment_for_arguments_in_allocation_expression=2

org.moreunit.report/pom.xml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,12 @@
4040
<version>${project.version}</version>
4141
</dependency>
4242
<dependency>
43-
<groupId>org.moreunit</groupId>
43+
<groupId>org.moreunit.plugins</groupId>
4444
<artifactId>org.moreunit.mock.test</artifactId>
4545
<version>${project.version}</version>
4646
</dependency>
4747
<dependency>
48-
<groupId>org.moreunit</groupId>
48+
<groupId>org.moreunit.plugins</groupId>
4949
<artifactId>org.moreunit.swtbot.test</artifactId>
5050
<version>${project.version}</version>
5151
</dependency>
@@ -54,6 +54,11 @@
5454
<artifactId>org.moreunit.test</artifactId>
5555
<version>${project.version}</version>
5656
</dependency>
57+
<dependency>
58+
<groupId>org.moreunit.plugins</groupId>
59+
<artifactId>org.moreunit.mock.it</artifactId>
60+
<version>${project.version}</version>
61+
</dependency>
5762
</dependencies>
5863
<build>
5964
<plugins>

0 commit comments

Comments
 (0)