Skip to content

test: exclude write-coverage from coverage report

Rodrigo Muino Tomonari requested to merge github/fork/bcoe/nyc-config into master

With a goal of pitching in/familiarizing myself with the Node.js codebase, I wanted to help inch test coverage towards 100%.

With this in mind, I noticed that write-coverage.js is not tested. Since this is a utility used for outputting test coverage reports it seemed appropriate to add this to a list of excluded files (this just became possible).

In this pull request:

  • I've added a .nycrc configuration file, which can be used to configure the nyc test coverage tool.
  • I've added an exclude rule that removes write-coverage.js from coverage reports.
  • I've pulled reporter configuration into the .nycrc config file (I also added the additional text reporter, which I find provides useful high-level information for folks playing the test-coverage MMORPG).
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test, coverage

Merge request reports