From 306e45daa2838912728acf3ec6eebdc22a661976 Mon Sep 17 00:00:00 2001 From: stoffeastrom Date: Mon, 18 Mar 2019 10:57:23 +0100 Subject: [PATCH] fix(node): exclude test patterns in coverage --- commands/node/src/index.js | 2 +- packages/utils/src/index.js | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/commands/node/src/index.js b/commands/node/src/index.js index 62791402..eb5163e4 100644 --- a/commands/node/src/index.js +++ b/commands/node/src/index.js @@ -218,7 +218,7 @@ class Runner extends EventEmitter { this.logLine('Loading', file); }); this.nyc = new NYC(this.argv.nyc); - this.argv.shouldInstrument = f => this.nyc.exclude.shouldInstrument(f); + this.argv.shouldInstrument = f => !utils.isTestFile(f, this.argv) && this.nyc.exclude.shouldInstrument(f); try { this.deleteCoverage() .setup(testFiles, srcFiles) diff --git a/packages/utils/src/index.js b/packages/utils/src/index.js index 8066a81c..e0379356 100644 --- a/packages/utils/src/index.js +++ b/packages/utils/src/index.js @@ -69,6 +69,8 @@ const DEFAULT_TRANSFORM_EXCLUDE_PATTERN = [ '**/*require*.js', '**/*sinon*.js', '**/*chai*.js', + DEFAULT_TEST_EXT_PATTERN, + DEFAULT_TEST_GLOB_PATTERN, ]; const DEFAULT_INSTRUMENT_EXCLUDE_PATTERN = DEFAULT_TRANSFORM_EXCLUDE_PATTERN;