c8 7.12.0 → 7.14.0

package/README.md

@@ -1,6 +1,6 @@
 # c8 - native V8 code-coverage
 
-![ci](https://img.shields.io/github/workflow/status/bcoe/c8/ci?label=ci&logo=github)
+[![ci](https://github.com/bcoe/c8/actions/workflows/ci.yaml/badge.svg)](https://github.com/bcoe/c8/actions/workflows/ci.yaml)
 ![nycrc config on GitHub](https://img.shields.io/nycrc/bcoe/c8)
 [![Conventional Commits](https://img.shields.io/badge/Conventional%20Commits-1.0.0-yellow.svg)](https://www.conventionalcommits.org/)
 
@@ -40,6 +40,7 @@ Here is a list of common options. Run `c8 --help` for the full list and document
 | `-e`, `--extension` | only files matching these extensions will show coverage | `string \| Array<string>` | [list](https://github.com/istanbuljs/schema/blob/master/default-extension.js) |
 | `--skip-full` | do not show files with 100% statement, branch, and function coverage | `boolean` | `false` |
 | `--check-coverage` | check whether coverage is within thresholds provided | `boolean` | `false` |
+| `--per-file` | check thresholds per file | `boolean` | `false` |
 | `--temp-directory` | directory V8 coverage data is written to and read from | `string` | `process.env.NODE_V8_COVERAGE` |
 | `--clean` | should temp files be deleted before script execution | `boolean` | `true` |
 

package/index.d.ts

@@ -8,6 +8,7 @@ export declare class Report {
     include?: string | string[],
     reporter: string[],
     reportsDirectory?: string,
+    reporterOptions?: Record<string, Record<string, unknown>>
     tempDirectory?: string,
     watermarks?: Partial<{
       statements: Watermark;
@@ -20,7 +21,9 @@ export declare class Report {
     resolve?: string,
     all?: boolean,
     src?: Array<string>,
-    allowExternal?: boolean
+    allowExternal?: boolean,
+    skipFull?: boolean,
+    excludeNodeModules?: boolean
   })
   run(): Promise<void>;
 }

package/lib/commands/report.js

@@ -34,7 +34,8 @@ exports.outputReport = async function (argv) {
     allowExternal: argv.allowExternal,
     src: argv.src,
     skipFull: argv.skipFull,
-    excludeNodeModules: argv.excludeNodeModules
+    excludeNodeModules: argv.excludeNodeModules,
+    mergeAsync: argv.mergeAsync
   })
   await report.run()
   if (argv.checkCoverage) await checkCoverages(argv, report)

package/lib/parse-args.js

@@ -152,6 +152,12 @@ function buildYargs (withCommands = false) {
       describe: 'supplying --allowExternal will cause c8 to allow files from outside of your cwd. This applies both to ' +
         'files discovered in coverage temp files and also src files discovered if using the --all flag.'
     })
+    .options('merge-async', {
+      default: false,
+      type: 'boolean',
+      describe: 'supplying --merge-async will merge all v8 coverage reports asynchronously and incrementally. ' +
+        'This is to avoid OOM issues with Node.js runtime.'
+    })
     .pkgConf('c8')
     .demandCommand(1)
     .check((argv) => {

package/lib/report.js

@@ -2,6 +2,12 @@ const Exclude = require('test-exclude')
 const libCoverage = require('istanbul-lib-coverage')
 const libReport = require('istanbul-lib-report')
 const reports = require('istanbul-reports')
+let readFile
+try {
+  ;({ readFile } = require('fs/promises'))
+} catch (err) {
+  ;({ readFile } = require('fs').promises)
+}
 const { readdirSync, readFileSync, statSync } = require('fs')
 const { isAbsolute, resolve, extname } = require('path')
 const { pathToFileURL, fileURLToPath } = require('url')
@@ -19,6 +25,7 @@ class Report {
     excludeAfterRemap,
     include,
     reporter,
+    reporterOptions,
     reportsDirectory,
     tempDirectory,
     watermarks,
@@ -29,9 +36,11 @@ class Report {
     src,
     allowExternal = false,
     skipFull,
-    excludeNodeModules
+    excludeNodeModules,
+    mergeAsync
   }) {
     this.reporter = reporter
+    this.reporterOptions = reporterOptions || {}
     this.reportsDirectory = reportsDirectory
     this.tempDirectory = tempDirectory
     this.watermarks = watermarks
@@ -51,6 +60,7 @@ class Report {
     this.all = all
     this.src = this._getSrc(src)
     this.skipFull = skipFull
+    this.mergeAsync = mergeAsync
   }
 
   _getSrc (src) {
@@ -74,7 +84,8 @@ class Report {
       reports.create(_reporter, {
         skipEmpty: false,
         skipFull: this.skipFull,
-        maxCols: process.stdout.columns || 100
+        maxCols: process.stdout.columns || 100,
+        ...this.reporterOptions[_reporter]
       }).execute(context)
     }
   }
@@ -87,7 +98,13 @@ class Report {
     if (this._allCoverageFiles) return this._allCoverageFiles
 
     const map = libCoverage.createCoverageMap()
-    const v8ProcessCov = this._getMergedProcessCov()
+    let v8ProcessCov
+
+    if (this.mergeAsync) {
+      v8ProcessCov = await this._getMergedProcessCovAsync()
+    } else {
+      v8ProcessCov = this._getMergedProcessCov()
+    }
     const resultCountPerPath = new Map()
     const possibleCjsEsmBridges = new Map()
 
@@ -185,43 +202,106 @@ class Report {
     }
 
     if (this.all) {
-      const emptyReports = []
+      const emptyReports = this._includeUncoveredFiles(fileIndex)
       v8ProcessCovs.unshift({
         result: emptyReports
       })
-      const workingDirs = this.src
-      const { extension } = this.exclude
-      for (const workingDir of workingDirs) {
-        this.exclude.globSync(workingDir).forEach((f) => {
-          const fullPath = resolve(workingDir, f)
-          if (!fileIndex.has(fullPath)) {
-            const ext = extname(fullPath)
-            if (extension.includes(ext)) {
-              const stat = statSync(fullPath)
-              const sourceMap = getSourceMapFromFile(fullPath)
-              if (sourceMap) {
-                this.sourceMapCache[pathToFileURL(fullPath)] = { data: sourceMap }
-              }
-              emptyReports.push({
-                scriptId: 0,
-                url: resolve(fullPath),
-                functions: [{
-                  functionName: '(empty-report)',
-                  ranges: [{
-                    startOffset: 0,
-                    endOffset: stat.size,
-                    count: 0
-                  }],
-                  isBlockCoverage: true
-                }]
-              })
-            }
+    }
+
+    return mergeProcessCovs(v8ProcessCovs)
+  }
+
+  /**
+   * Returns the merged V8 process coverage.
+   *
+   * It asynchronously and incrementally reads and merges individual process coverages
+   * generated by Node. This can be used via the `--merge-async` CLI arg.  It's intended
+   * to be used across a large multi-process test run.
+   *
+   * @return {ProcessCov} Merged V8 process coverage.
+   * @private
+   */
+  async _getMergedProcessCovAsync () {
+    const { mergeProcessCovs } = require('@bcoe/v8-coverage')
+    const fileIndex = new Set() // Set<string>
+    let mergedCov = null
+    for (const file of readdirSync(this.tempDirectory)) {
+      try {
+        const rawFile = await readFile(
+          resolve(this.tempDirectory, file),
+          'utf8'
+        )
+        let report = JSON.parse(rawFile)
+
+        if (this._isCoverageObject(report)) {
+          if (report['source-map-cache']) {
+            Object.assign(this.sourceMapCache, this._normalizeSourceMapCache(report['source-map-cache']))
           }
-        })
+          report = this._normalizeProcessCov(report, fileIndex)
+          if (mergedCov) {
+            mergedCov = mergeProcessCovs([mergedCov, report])
+          } else {
+            mergedCov = mergeProcessCovs([report])
+          }
+        }
+      } catch (err) {
+        debuglog(`${err.stack}`)
       }
     }
 
-    return mergeProcessCovs(v8ProcessCovs)
+    if (this.all) {
+      const emptyReports = this._includeUncoveredFiles(fileIndex)
+      const emptyReport = {
+        result: emptyReports
+      }
+
+      mergedCov = mergeProcessCovs([emptyReport, mergedCov])
+    }
+
+    return mergedCov
+  }
+
+  /**
+   * Adds empty coverage reports to account for uncovered/untested code.
+   * This is only done when the `--all` flag is present.
+   *
+   * @param {Set} fileIndex list of files that have coverage
+   * @returns {Array} list of empty coverage reports
+   */
+  _includeUncoveredFiles (fileIndex) {
+    const emptyReports = []
+    const workingDirs = this.src
+    const { extension } = this.exclude
+    for (const workingDir of workingDirs) {
+      this.exclude.globSync(workingDir).forEach((f) => {
+        const fullPath = resolve(workingDir, f)
+        if (!fileIndex.has(fullPath)) {
+          const ext = extname(fullPath)
+          if (extension.includes(ext)) {
+            const stat = statSync(fullPath)
+            const sourceMap = getSourceMapFromFile(fullPath)
+            if (sourceMap) {
+              this.sourceMapCache[pathToFileURL(fullPath)] = { data: sourceMap }
+            }
+            emptyReports.push({
+              scriptId: 0,
+              url: resolve(fullPath),
+              functions: [{
+                functionName: '(empty-report)',
+                ranges: [{
+                  startOffset: 0,
+                  endOffset: stat.size,
+                  count: 0
+                }],
+                isBlockCoverage: true
+              }]
+            })
+          }
+        }
+      })
+    }
+
+    return emptyReports
   }
 
   /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "c8",
-  "version": "7.12.0",
+  "version": "7.14.0",
   "description": "output coverage reports using Node.js' built in coverage",
   "main": "./index.js",
   "types": "./index.d.ts",
@@ -47,7 +47,7 @@
     "yargs-parser": "^20.2.9"
   },
   "devDependencies": {
-    "@types/node": "^17.0.23",
+    "@types/node": "^18.0.0",
     "chai": "^4.3.6",
     "chai-jest-snapshot": "^2.0.0",
     "cross-env": "^7.0.3",