jenkinsci / cobertura-plugin

Jenkins cobertura plugin
https://plugins.jenkins.io/cobertura/
MIT License
109 stars 133 forks source link

remove condition coverage necessity from branch coverage #134

Open LorenzMende opened 2 years ago

LorenzMende commented 2 years ago

Hi there, we are working with sbt-scoverage and use the cobertura plugin for test result aggregation/ publishing. I stumbled upon these lines in the parser:

https://github.com/jenkinsci/cobertura-plugin/blob/0ced5fe46dfdd3406ef1c6d01dceacb2c7099b3b/src/main/java/hudson/plugins/cobertura/CoberturaCoverageParser.java#L202-L205

Why is condition coverage data necessary for branch coverage parsing? Basically branch coverage and condition coverage are different by definition - and are only loosely coupled (branch coverage can be obtained without successful condition coverage).

I sincerely ask to update the functionality - I may create a PR for this issue if necessary.