-
Notifications
You must be signed in to change notification settings - Fork 40
Description
Just noticed this on a recent PR run:
Coverage report
File | Statements | Missing | Coverage | Coverage (new stmts) |
---|---|---|---|---|
src/pydot | ||||
classes.py | ||||
core.py | ||||
dot_parser.py | ||||
test | ||||
conftest.py | ||||
test_classes.py | ||||
test_pydot.py | ||||
Project Total |
This report was generated by python-coverage-comment-action
...There were no code changes, so there were no coverage changes, of course. But why does test_pydot.py
staying at 93% coverage get a green badge, while classes.py
staying at 96% gets an orange badge? Something fishy there.
Possible it has something to do with the branch coverage, though I don't think so.
(Although, speaking of branch coverage, having it included does make those summaries a bit "off". For example, 384
÷ 402
is actually 95.5%, not 93%. 93% is the correct coverage value, because missing branches lower the total coverage for the file... but it means that displaying both the percentages and the lines-of-coverage fractions is a bit confusing/misleading, since one doesn't directly correspond to the other.)