Re: [webrtc-pc] Adopt "test as you commit" policy

> but before we can adopt it the current test suite must decently covering (though it is in pretty good shape in this case as you point out)

Do you mean that coverage needs to be better than indicated by, or just that OK coverage is a prerequisite which is already met? Of the ~80 specs that have adopted a similar working mode so far, there were none where the detailed coverage was even known, and probably only a handful (maybe DOM and Service Worker) where the coverage was this good.

In practice, when making a change one goes to see if there's already coverage, and if there isn't *and* it's too much work to build up the required scaffolding, one just files an issue about the fact and move along without tests.

GitHub Notification of comment by foolip
Please view or discuss this issue at using your GitHub account

Received on Tuesday, 10 October 2017 09:27:45 UTC