Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Incorrect coverage reporting 🐛

See original GitHub issue

The coverage report for 87135a5d0c3c5a27db83509fba01774dd20201d9 claims that DictionaryExtensions.cs has 75% coverage. It only has one method:

https://github.com/tonerdo/coverlet/blob/87135a5d0c3c5a27db83509fba01774dd20201d9/src/coverlet.core/Extensions/DictionaryExtensions.cs#L7-L14

According to the report, lines 9, 10 and 13 are covered but not line 12 (thus the 75%). How can 13 be covered but not 12? Is this a bug or am I misreading something? The only test for this method…

https://github.com/tonerdo/coverlet/blob/87135a5d0c3c5a27db83509fba01774dd20201d9/test/coverlet.core.tests/Extensions/DictionaryExtensionsTests.cs#L10-L17

…seems to cover it 100%.

Issue Analytics

State:
Created 5 years ago
Comments:11 (3 by maintainers)

Top GitHub Comments

3reactions

tonerdocommented, Mar 22, 2018

@atifaziz I have noticed this little kinks here and there. Currently trying to find the root cause, intend to have it fixed by this weekend

1reaction

tonerdocommented, Mar 22, 2018

@OneCyrus @atifaziz This issue has been fixed in #10 and you’ll see that the coverage for this repo increased as a result.

It was actually a pretty straightforward fix and a good side effect is it sets the foundation for branch coverage support. This fix will be in a new NuGet package I’ll release this weekend, in the future I’ll setup a nightly CI package publish so improvements can get out faster.

Cheers