preactjs / preact / 5047

100%

push

travis-ci

Exclude devtools tests from coverage (#1808)

* Ignore code coverage reporting for devtools.

The devtools code uses newer ECMAScript features because the
devtools extension is only available for Firefox and Chrome. But
we didn't have such a distinction in our tests suite. That leads
to failing tests on IE11.

We tried running those tests only on FF and Chrome, but because
karma-coverage always collects coverage data from the latest
running browser, it would lead to coverage failures. The latest
browser is typically slowest and you may have guessed it already:
This will pretty much always be IE11.

The nearterm conclusion is to just disable coverage reporting
for the devtools code completely...

* Only run devtools tests in FF + Chrome

* Check if early return reduces cli output noise

778 of 803 branches covered (96.89%)

800 of 804 relevant lines covered (99.5%)

748.71 hits per line