feat: adds --all functionality

.gitignore

@@ -3,3 +3,4 @@ node_modules
 .nyc_output
 coverage
 tmp
+.idea

lib/commands/report.js

@@ -19,7 +19,8 @@ exports.outputReport = async function (argv) {
     watermarks: argv.watermarks,
     resolve: argv.resolve,
     omitRelative: argv.omitRelative,
-    wrapperLength: argv.wrapperLength
+    wrapperLength: argv.wrapperLength,
+    all: argv.all
   })
   await report.run()
   if (argv.checkCoverage) checkCoverages(argv, report)

lib/parse-args.js

@@ -83,6 +83,12 @@ function buildYargs (withCommands = false) {
       type: 'boolean',
       describe: 'should temp files be deleted before script execution'
     })
+    .options('all', {
+      default: false,
+      type: 'boolean',
+      describe: 'supplying --all will cause c8 to consider all src files in the current working directory ' +
+        'when the determining coverage. Respects include/exclude.'
+    })
     .pkgConf('c8')
     .config(config)
     .demandCommand(1)

lib/report.js

@@ -3,8 +3,8 @@ const furi = require('furi')
 const libCoverage = require('istanbul-lib-coverage')
 const libReport = require('istanbul-lib-report')
 const reports = require('istanbul-reports')
-const { readdirSync, readFileSync } = require('fs')
-const { isAbsolute, resolve } = require('path')
+const { readdirSync, readFileSync, statSync } = require('fs')
+const { isAbsolute, resolve, join, relative, extname, dirname } = require('path')
 // TODO: switch back to @c88/v8-coverage once patch is landed.
 const v8toIstanbul = require('v8-to-istanbul')
 const isCjsEsmBridgeCov = require('./is-cjs-esm-bridge')
@@ -19,7 +19,8 @@ class Report {
     watermarks,
     omitRelative,
     wrapperLength,
-    resolve: resolvePaths
+    resolve: resolvePaths,
+    all
   }) {
     this.reporter = reporter
     this.reportsDirectory = reportsDirectory
@@ -33,6 +34,8 @@ class Report {
     this.omitRelative = omitRelative
     this.sourceMapCache = {}
     this.wrapperLength = wrapperLength
+    this.all = all
+    this.src = process.cwd()
   }
 
   async run () {
@@ -56,8 +59,8 @@ class Report {
     // use-case.
     if (this._allCoverageFiles) return this._allCoverageFiles
 
+    const map = libCoverage.createCoverageMap()
     const v8ProcessCov = this._getMergedProcessCov()
-    const map = libCoverage.createCoverageMap({})
     const resultCountPerPath = new Map()
     const possibleCjsEsmBridges = new Map()
 
@@ -94,7 +97,6 @@ class Report {
         map.merge(converter.toIstanbul())
       }
     }
-
     this._allCoverageFiles = map
     return this._allCoverageFiles
   }
@@ -127,6 +129,29 @@ class Report {
     return sources
   }
 
+  /**
+   * //TODO: use https://www.npmjs.com/package/convert-source-map
+   * // no need to roll this ourselves this is already in the dep tree
+   * https://sourcemaps.info/spec.html
+   * @param {String} compilation target file
+   * @returns {String} full path to source map file
+   * @private
+   */
+  _getSourceMapFromFile (file) {
+    const fileBody = readFileSync(file).toString()
+    const sourceMapLineRE = /\/\/[#@] ?sourceMappingURL=([^\s'"]+)\s*$/mg
+    const results = fileBody.match(sourceMapLineRE)
+    if (results !== null) {
+      const sourceMap = results[results.length - 1].split('=')[1]
+      if (isAbsolute(sourceMap)) {
+        return sourceMap
+      } else {
+        const base = dirname(file)
+        return join(base, sourceMap)
+      }
+    }
+  }
+
   /**
    * Returns the merged V8 process coverage.
    *
@@ -139,14 +164,50 @@ class Report {
   _getMergedProcessCov () {
     const { mergeProcessCovs } = require('@bcoe/v8-coverage')
     const v8ProcessCovs = []
+    const fileIndex = new Map() // Map<string, bool>
     for (const v8ProcessCov of this._loadReports()) {
       if (this._isCoverageObject(v8ProcessCov)) {
         if (v8ProcessCov['source-map-cache']) {
           Object.assign(this.sourceMapCache, v8ProcessCov['source-map-cache'])
         }
-        v8ProcessCovs.push(this._normalizeProcessCov(v8ProcessCov))
+        v8ProcessCovs.push(this._normalizeProcessCov(v8ProcessCov, fileIndex))
       }
     }
+
+    if (this.all) {
+      const emptyReports = []
+      v8ProcessCovs.unshift({
+        result: emptyReports
+      })
+      const workingDir = process.cwd()
+      this.exclude.globSync(workingDir).forEach((f) => {
+        const fullPath = resolve(workingDir, f)
+        if (!fileIndex.has(fullPath)) {
+          const ext = extname(f)
+          if (ext === '.js' || ext === '.ts' || ext === '.mjs') {
+            const stat = statSync(f)
+            const sourceMap = this._getSourceMapFromFile(f)
+            if (sourceMap !== undefined) {
+              this.sourceMapCache[`file://${fullPath}`] = { data: JSON.parse(readFileSync(sourceMap).toString()) }
+            }
+            emptyReports.push({
+              scriptId: 0,
+              url: resolve(f),
+              functions: [{
+                functionName: '(empty-report)',
+                ranges: [{
+                  startOffset: 0,
+                  endOffset: stat.size,
+                  count: 0
+                }],
+                isBlockCoverage: true
+              }]
+            })
+          }
+        }
+      })
+    }
+
     return mergeProcessCovs(v8ProcessCovs)
   }
 
@@ -196,12 +257,13 @@ class Report {
    * @return {v8ProcessCov} Normalized V8 process coverage.
    * @private
    */
-  _normalizeProcessCov (v8ProcessCov) {
+  _normalizeProcessCov (v8ProcessCov, fileIndex) {
     const result = []
     for (const v8ScriptCov of v8ProcessCov.result) {
       if (/^file:\/\//.test(v8ScriptCov.url)) {
         try {
           v8ScriptCov.url = furi.toSysPath(v8ScriptCov.url)
+          fileIndex.set(v8ScriptCov.url, true)
         } catch (err) {
           console.warn(err)
           continue

package.json

@@ -61,4 +61,4 @@
     "bin",
     "LICENSE"
   ]
-}
+}

test/fixtures/all/ts-compiled/dir/unloaded.ts

@@ -0,0 +1,5 @@
+export default function Unloaded(){
+  return 'Never loaded :('
+}
+
+console.log("This file shouldn't have been evaluated")

test/fixtures/all/ts-compiled/loaded.ts

@@ -0,0 +1,19 @@
+export default function getString(i){
+  if (typeof i === 'number'){
+    if (isNaN(i)){
+      return 'NaN'
+    }
+    else if (i === 0){
+      return 'zero'
+    }
+    else if (i > 0){
+      return 'positive'
+    }
+    else {
+      return 'negative'
+    }
+  }
+  else {
+    return 'wat?'
+  }
+}

test/fixtures/all/ts-compiled/main.ts

@@ -0,0 +1,4 @@
+import getString from "./loaded"
+console.log(getString(0))
+console.log(getString(1))
+console.log(getString(-1))

test/fixtures/all/ts-only/dir/unloaded.ts

@@ -0,0 +1,5 @@
+export default function Unloaded(){
+  return 'Never loaded :('
+}
+
+console.log("This file shouldn't have been evaluated")

test/fixtures/all/ts-only/loaded.ts

@@ -0,0 +1,19 @@
+export default function getString(i){
+  if (typeof i === 'number'){
+    if (isNaN(i)){
+      return 'NaN'
+    }
+    else if (i === 0){
+      return 'zero'
+    }
+    else if (i > 0){
+      return 'positive'
+    }
+    else {
+      return 'negative'
+    }
+  }
+  else {
+    return 'wat?'
+  }
+}

test/fixtures/all/ts-only/main.ts

@@ -0,0 +1,4 @@
+import getString from "./loaded"
+console.log(getString(0))
+console.log(getString(1))
+console.log(getString(-1))

test/fixtures/all/vanilla/dir/unloaded.js

@@ -0,0 +1,5 @@
+module.exports = function Unloaded(){
+  return 'Never loaded :('
+}
+
+console.log("This file shouldn't have been evaluated")

test/fixtures/all/vanilla/loaded.js

@@ -0,0 +1,19 @@
+module.exports = function getString(i){
+  if (typeof i === 'number'){
+    if (isNaN(i)){
+      return 'NaN'
+    }
+    else if (i === 0){
+      return 'zero'
+    }
+    else if (i > 0){
+      return 'positive'
+    }
+    else {
+      return 'negative'
+    }
+  }
+  else {
+    return 'wat?'
+  }
+}

test/fixtures/all/vanilla/main.js

@@ -0,0 +1,4 @@
+const loaded = require('./loaded.js');
+console.log(loaded(0))
+console.log(loaded(1))
+console.log(loaded(-1))

test/integration.js

@@ -327,4 +327,47 @@ describe('c8', () => {
       output.toString('utf8').should.matchSnapshot()
     })
   })
+  describe('--all', () => {
+    it('reports coverage for unloaded js files as 0 for line, branch and function', () => {
+      const { output } = spawnSync(nodePath, [
+        c8Path,
+        '--temp-directory=tmp/vanilla-all',
+        '--clean=false',
+        '--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('reports coverage for unloaded transpiled ts files as 0 for line, branch and function', () => {
+      const { output } = spawnSync(nodePath, [
+        c8Path,
+        '--temp-directory=tmp/all-ts',
+        '--clean=false',
+        '--all=true',
+        '--include=test/fixtures/all/ts-compiled/**/*.ts',
+        '--exclude="test/*.js"', // add an exclude to avoid default excludes of test/**
+        nodePath,
+        require.resolve('./fixtures/all/ts-compiled/main')
+      ])
+      output.toString('utf8').should.matchSnapshot()
+    })
+
+    it('reports coverage for unloaded ts files as 0 for line, branch and function when using ts-node', () => {
+      const { output } = spawnSync(nodePath, [
+        c8Path,
+        '--temp-directory=tmp/all-ts-node',
+        '--clean=false',
+        '--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',
+        require.resolve('./fixtures/all/ts-only/main.ts')
+      ])
+      output.toString('utf8').should.matchSnapshot()
+    })
+  })
 })