jenkinsci / jenkinsci/github-pr-coverage-status-plugin

Sonar: Correct master metric (at least for Jacoco)

Open
#40 1 comment 2 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.