dashingrocket/simplecov-cobertura

branch coverage support

skcc321 opened this issue · 9 comments

skcc321 commented

I see no support for branch coverage in the readme. But I would like to start the process of implementation of that one somehow.
according to cobertura documentation - branch coverage is supported.
https://www.baeldung.com/cobertura#2-configuring-code-coverage-calculation-and-checks
simplecov supports branch coverage as well
https://github.com/simplecov-ruby/simplecov#branch-coverage-ruby--25

let's implement that one finally :)

jessebs commented

Great find. Is that something you'd want to take up?

skcc321 commented

probably. but not sure how soon...

ms-ati commented

Can anyone provide guidance on how to implement branch coverage, please?

klausbadelt commented

Istanbul reports branches like this:

<line number="38" hits="59" branch="true" condition-coverage="100% (2/2)"/>

Has anyone picked this up yet? I might have a go otherwise.

jessebs commented

As far as I am aware, no one has picked this up

skcc321 commented

nobody....

  • 😄1
skcc321 commented

something like this #20

  • 🎉1
jessebs commented

Thank you for the work on this and I'm sorry for the delay. This is merged in and will be released as part of 2.0.0