Skip to content

lib: remove an unnecessary coverage check

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

coverage

Fixes: https://github.com/nodejs/node/issues/11445 Refs: f65a48fc

Based on @addaleax's comment here.

Before (coverage.nodejs.org, master)

screen shot 2017-03-24 at 12 15 20 pm

After

screen shot 2017-03-24 at 12 15 27 pm

And the specific process._exiting lines that f65a48fc was supposed to hit are indeed hit with this change:

screen shot 2017-03-24 at 12 25 52 pm

cc @addaleax, @mhdawson @CurryKitten

Merge request reports

Loading