fix: all flag not propagated to check-coverage command (#188)

bcoe · Jan 13, 2020 · 86eaf72 · 86eaf72
1 parent 2ab701f
commit 86eaf72
Show file tree

Hide file tree

Showing 3 changed files with 73 additions and 3 deletions.
diff --git a/lib/commands/check-coverage.js b/lib/commands/check-coverage.js
@@ -20,7 +20,8 @@ exports.handler = function (argv) {
     watermarks: argv.watermarks,
     resolve: argv.resolve,
     omitRelative: argv.omitRelative,
-    wrapperLength: argv.wrapperLength
+    wrapperLength: argv.wrapperLength,
+    all: argv.all
   })
   exports.checkCoverages(argv, report)
 }

diff --git a/test/integration.js b/test/integration.js
@@ -4,6 +4,7 @@ const { spawnSync } = require('child_process')
 const { statSync } = require('fs')
 const c8Path = require.resolve('../bin/c8')
 const nodePath = process.execPath
+const tsNodePath = './node_modules/.bin/ts-node'
 const chaiJestSnapshot = require('chai-jest-snapshot')
 const rimraf = require('rimraf')
 
@@ -321,7 +322,7 @@ describe('c8', () => {
         '--exclude="test/*.js"',
         '--temp-directory=tmp/source-map',
         '--clean=true',
-        './node_modules/.bin/ts-node',
+        tsNodePath,
         require.resolve('./fixtures/ts-node-basic.ts')
       ])
       output.toString('utf8').should.matchSnapshot()
@@ -364,10 +365,55 @@ describe('c8', () => {
         '--all=true',
         '--include=test/fixtures/all/ts-only/**/*.ts',
         '--exclude="test/*.js"', // add an exclude to avoid default excludes of test/**
-        './node_modules/.bin/ts-node',
+        tsNodePath,
         require.resolve('./fixtures/all/ts-only/main.ts')
       ])
       output.toString('utf8').should.matchSnapshot()
     })
+
+    it('should allow for --all to be used in conjunction with --check-coverage', () => {
+      const { output } = spawnSync(nodePath, [
+        c8Path,
+        '--temp-directory=tmp/all-check-coverage',
+        '--clean=false',
+        '--check-coverage',
+        '--lines=100',
+        '--all=true',
+        '--include=test/fixtures/all/vanilla/**/*.js',
+        '--exclude=**/*.ts', // add an exclude to avoid default excludes of test/**
+        nodePath,
+        require.resolve('./fixtures/all/vanilla/main')
+      ])
+      output.toString('utf8').should.matchSnapshot()
+    })
+
+    it('should allow for --all to be used with the check-coverage command (2 invocations)', () => {
+      // generate v8 output
+      spawnSync(nodePath, [
+        c8Path,
+        '--temp-directory=tmp/all-check-coverage-as-command',
+        '--clean=false',
+        '--check-coverage',
+        '--lines=90',
+        '--all=true',
+        '--include=test/fixtures/all/vanilla/**/*.js',
+        '--exclude=**/*.ts', // add an exclude to avoid default excludes of test/**
+        nodePath,
+        require.resolve('./fixtures/all/vanilla/main')
+      ])
+
+      // invoke check-coverage as a command with --all
+      const { output } = spawnSync(nodePath, [
+        c8Path,
+        'check-coverage',
+        '--lines=90',
+        '--temp-directory=tmp/all-check-coverage-as-command',
+        '--clean=false',
+        '--all=true',
+        '--include=test/fixtures/all/vanilla/**/*.js',
+        '--exclude=**/*.ts' // add an exclude to avoid default excludes of test/**
+      ])
+      output.toString('utf8').should.matchSnapshot()
+    })
   })
 })
diff --git a/test/integration.js.snap b/test/integration.js.snap
@@ -65,6 +65,29 @@ All files     |   64.29 |    66.67 |      50 |   64.29 |
 ,"
 `;
 
+exports[`c8 --all should allow for --all to be used in conjunction with --check-coverage 1`] = `
+",zero
+positive
+negative
+--------------|---------|----------|---------|---------|-------------------
+File          | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
+--------------|---------|----------|---------|---------|-------------------
+All files     |   64.29 |    66.67 |      50 |   64.29 |                   
+ vanilla      |   78.26 |       75 |     100 |   78.26 |                   
+  loaded.js   |   73.68 |    71.43 |     100 |   73.68 | 4,5,16-18         
+  main.js     |     100 |      100 |     100 |     100 |                   
+ vanilla/dir  |       0 |        0 |       0 |       0 |                   
+  unloaded.js |       0 |        0 |       0 |       0 | 1-5               
+--------------|---------|----------|---------|---------|-------------------
+,ERROR: Coverage for lines (64.29%) does not meet global threshold (100%)
+"
+`;
+
+exports[`c8 --all should allow for --all to be used with the check-coverage command (2 invocations) 1`] = `
+",,ERROR: Coverage for lines (64.29%) does not meet global threshold (90%)
+"
+`;
+
 exports[`c8 ESM Modules collects coverage for ESM modules 1`] = `
 ",bar foo
 ------------|---------|----------|---------|---------|-------------------