Skip to content

Commit 03b2dfe

Browse files
dfa1claude
andcommitted
ci: add integration jacoco-it report to Sonar coverage paths
Integration tests (vortex-integration module) cover reader/writer/core classes but their JaCoCo data lands in integration/target/site/jacoco-it/ jacoco.xml. Without this path Sonar sees 0 cross-module coverage for BitpackedEncodingDecoder and other decoder classes. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 8ab9ec7 commit 03b2dfe

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

pom.xml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,11 +84,14 @@
8484
**/fbs/**,
8585
**/proto/**
8686
</sonar.coverage.exclusions>
87-
<!-- Both surefire (jacoco.xml) and failsafe (jacoco-it.xml) reports sent to Sonar.
88-
The failsafe report is produced by the report-integration execution in the coverage profile. -->
87+
<!-- Coverage sources: per-module surefire, per-module failsafe, and the
88+
integration-module failsafe report. Integration tests live in the
89+
vortex-integration module and exercise classes in reader/writer/core —
90+
without the third path those cross-module hits are invisible to Sonar. -->
8991
<sonar.coverage.jacoco.xmlReportPaths>
9092
${project.build.directory}/site/jacoco/jacoco.xml,
91-
${project.build.directory}/site/jacoco-it/jacoco.xml
93+
${project.build.directory}/site/jacoco-it/jacoco.xml,
94+
${maven.multiModuleProjectDirectory}/integration/target/site/jacoco-it/jacoco.xml
9295
</sonar.coverage.jacoco.xmlReportPaths>
9396
<!-- Default for the {@code @{argLine}} placeholder in surefire/failsafe configs. JaCoCo's
9497
{@code prepare-agent} goal overwrites this with the agent arguments when active; if

0 commit comments

Comments
 (0)