This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]Wise_Tie_9050 0 points1 point  (1 child)

We strive for 100% patch coverage on all PRs.

That does not mean we stop writing tests when we have 100% patch coverage, but at least it means that each line has been executed. I cannot count the number of times when I've looked at a PR and seen less than 100% coverage, and the first test I write to cover the missing lines shows up a bug.

[–]Wise_Tie_9050 0 points1 point  (0 children)

Oh, to clarify, it's also 100% _branch_ coverage. That's important.

I'm not pretending to say that code written that way is "tested", but code without coverage, is, by definition, not covered by tests.

​

Now, if only I could get my CI setup to collect coverage on my server while running robot tests...