RFR: 8219: Missing coverage in application reports [v9]

Marcus Hirt hirt at openjdk.org
Fri Oct 11 17:53:18 UTC 2024


On Thu, 26 Sep 2024 15:05:57 GMT, Alex Macdonald <aptmac at openjdk.org> wrote:

>> This PR fixes (&cleans up) the coverage reports under jmc/application.
>> 
>> The argLine was being set in a handful of the application test packages without the surefireArgLine that's required to have jacoco run coverage.
>> 
>> Additionally I've removed a bunch of the feature packages from the report (as there isn't anything to scan), and included missing packages that have been added in the last few years. There were also a handful of versions that needed to be updated as well.
>> 
>> The Jacoco version in core was bumped to 8.0.10, but the application version stayed at 8.0.7, this has been updated.
>
> Alex Macdonald has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
> 
>  - Add tycho.testArgLine.local to Jolokia test argLine
>  - adjust the argline for tycho-maven-plugin
>  - 8219: Missing coverage in application reports

Marked as reviewed by hirt (Lead).

Ok. Let's take it, but be prepared to follow up with a fix if the integration goes bad.

-------------

PR Review: https://git.openjdk.org/jmc/pull/562#pullrequestreview-2363312905
PR Comment: https://git.openjdk.org/jmc/pull/562#issuecomment-2407877976


More information about the jmc-dev mailing list