jenkinsci / jenkinsci/github-pr-coverage-status-plugin
Sonar: Correct master metric (at least for Jacoco)
- Dominant language
- Java
- Stars
- 97
- Forks
- 101
- PR merge metrics
- No merged PRs in 30d
Description
In Sonar, I've got the following metrics displayed:
Coverage: 64.8%
Line Coverage: 72.8%
Condition Coverage: 38.8%
Here, it is grabbing the master as "coverage" which gets us 64.8%.
Then, we calculate the current coverage via the JacocoParser, which is just calculating off of line coverage, thus I end up comparing "coverage" versus "line coverage"...
Would it make sense to be able to select which metric to use for:
- parsing jacoco (instruction, line, or branch)
- pulling from sonar (coverage (aka instruction), line, or condition (aka branch)
Thanks.
Contributor guide
Research direction
Start by reading the JacocoParser and the Sonar metric handling described in the issue. Trace how Jacoco line coverage is calculated and how Sonar's coverage value is selected. Done means the supported metric choices are defined consistently for both sources, with tests covering instruction, line, and branch or condition coverage.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- testing-qa
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100