CAMEL-23274: Add incremental coverage to SonarCloud PR analysis#22469
Draft
gnodet wants to merge 1 commit intoapache:ci-merge-incremental-build-detect-dependenciesfrom
Draft
CAMEL-23274: Add incremental coverage to SonarCloud PR analysis#22469gnodet wants to merge 1 commit intoapache:ci-merge-incremental-build-detect-dependenciesfrom
gnodet wants to merge 1 commit intoapache:ci-merge-incremental-build-detect-dependenciesfrom
Conversation
Contributor
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
Build on top of the core coverage baseline by adding per-module coverage for PR-affected components using the incremental-build action. Changes: - parent/pom.xml: Add jacoco:report goal to coverage profile so each tested module generates its own XML report - pom.xml: Update sonar.coverage.jacoco.xmlReportPaths to include both per-module and aggregated report paths - sonar-build.yml: Add incremental-build step with -Dcoverage in MVND_OPTS to test PR-affected modules with JaCoCo Coverage strategy: - Core: aggregated report via coverage module (camel-core tests exercise classes across multiple core source modules) - Components: per-module reports (each component's tests cover its own classes) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
85272ce to
851f479
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CAMEL-23274
Summary
Follow-up to #22468 (core coverage) and #22247 (incremental build rationalization).
-Dcoveragejacoco:reportgoal to thecoverageprofile inparent/pom.xmlso each tested module generates its own XML reportsonar.coverage.jacoco.xmlReportPathsto include both per-module and aggregated report pathsCoverage strategy
coveragemodule (camel-core tests exercise classes across camel-core-model, camel-core-processor, etc.)coverage/target/site/jacoco-aggregate/jacoco.xml*/target/site/jacoco/jacoco.xmlDependencies
Test plan
-Dcoveragein MVND_OPTS