You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
(Originally posted in #466. Moved here so I don't lose track of it once the source PR is merged.)
I was thinking of submitting a (separate) PR that adds an actual code modification to the e2e tests, so that we could get a look at the diff coverage reporting as well. That's the part, if anything, that I think might need adjusting with branch coverage enabled.
(I figured I'd drop the test-ed function to just contain
...And then have the e2e test function insert the if c: and if d: blocks. (Or maybe even if c: ... elif d: which might make for better branch coverage testing.)
The text was updated successfully, but these errors were encountered:
(Originally posted in #466. Moved here so I don't lose track of it once the source PR is merged.)
I was thinking of submitting a (separate) PR that adds an actual code modification to the e2e tests, so that we could get a look at the diff coverage reporting as well. That's the part, if anything, that I think might need adjusting with branch coverage enabled.
(I figured I'd drop the test-ed function to just contain
...And then have the e2e test function insert the
if c:
andif d:
blocks. (Or maybe evenif c:
...elif d:
which might make for better branch coverage testing.)The text was updated successfully, but these errors were encountered: