Branch test coverage seems not to be returned when fully covered

SonarQube Cloud
,
1

When a project is completely covered by tests the branch coverage seems to be not returned. So the caller doesnt know if the coverage is good or just not calculated

2

Hi,

I’m not sure we’ve ever gotten far enough in coverage to see this error. :joy:

Could you provide some screenshots or the Web API call where you’re seeing this?

 
Thx,
Ann

3

Microservice style with project of 120 lines and some tests :slight_smile: Will gather those infos from the team

4

https://sonarcloud.io/api/measures/component?component=testificast_CastFramework/cast-pdf-plugin&ps=1000&metricKeys=bugs,reliability_rating,reliability_remediation_effort,cognitive_complexity,complexity,violations,blocker_violations,critical_violations,major_violations,minor_violations,info_violations,code_smells,sqale_index,sqale_rating,vulnerabilities,security_rating,security_hotspots,comment_lines_density,ncloc,branch_coverage,coverage,line_coverage,test_success_density,duplicated_lines,duplicated_lines_density,alert_status,test_errors,skipped_tests,tests,functions,test_execution_time,ncloc_language_distribution

image
image401Ă—611 17.4 KB

image
image1106Ă—673 31 KB

5

Hi,

The URL doesn’t work for me, so it must be a private project. Oh well. Thanks anyway for providing it.

For the screenshots, other than getting a 0 for Lines to Cover, this all looks okay to me. What part of it do you think is wrong?

 
Ann

6

Just the branch coverage is missing in the api call and in the overview. expected is that it return 100% branch covered

7

Or another question regarding this. We have Overall coverage, line coverage and branch coverage. A line can have multiple branches so when we have an overall coverage of 100% both the line and the branch must also be 100% , right? So i would guess the developers hiding some redundant information here which of course the client need to imply by himself and code it on top of just receiving the data and show as it is

8

Hi,

Actually, I just pulled up one of our own projects to see which metrics were listed for it. I accidentally landed on one that happens to have 100% coverage (color me surprised!) and our listing is fuller than yours:

Selection_913
Selection_913266Ă—770 22.2 KB

So this seems not to be about 100% coverage but about your unique case.

So I’m going to call in more expert eyes.

 
Ann

10

But it looks a bit the same. As soon as there are 0 Conditions to cover, the Condition coverage is missing. When not all coverd:
image

11

im also a bit confused with the conditions/branches . there was a time where it all was called branch, boolean and exception catch branch. might be the catch of exception is not there anymore

12

Hi,

Look at the top section of my screenshot: the Coverage on New Code section shows coverage of 100%, with uncovered conditions of 0, and 0 conditions (still left?) to cover.

I think something’s going wrong (or wronger) in your particular case. To be honest, I find “0 lines to cover” and “0 conditions to cover” very odd. I would expect those to be the numbers of cover-able lines and conditions, not what’s left to do. So that’s wrong too IMO, but you have this additional wrong thing of metrics not showing up.

And I’ve flagged this for the team, so hopefully they’ll handle all the wrongness at once. :smiley:

 
Ann

1 Like
13

that 0s are for new code, which doesnt exist in the timframe it seems

14

but the overall code shows the line, and the catch is catching 3 different exceptions, so i would expect branches to exist for it

17

@Henning_Luther So to summarize the issue, do you mean to say that the branch coverage is missing for the overall code?

18

yes. i mean i dont see any coverage and also not even existing branches. i would expect for every exception cath to be one branch. but i might be wrong. and if there are no branches the question might be, is the coverage not existent or 100%

image
image1365Ă—431 34.5 KB

21

Hey so sorry for not getting back to this thread. I would like to know more about this issue. I can explain a few scenarios and let me know if that helps:

  1. I change a part of code, say a configuration file. The analysis tell me there are no coverage information as this file is not part of coverage in tests. This is expected
  2. I change a few lines of code which doesn’t have any conditions. I would expect line coverage on this and no conditional coverage as this seems unnecessary
  3. I added few conditions in my code. I would expect conditional coverage to be returned

I think in the end it would also come to the coverage report you are using. Let me know if this helps

23

i will have a look again later, since i already forgot about it :slight_smile:

1 Like