nyc 11.8.0 → 11.9.0

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+<a name="11.9.0"></a>
+# [11.9.0](https://github.com/istanbuljs/nyc/compare/v11.8.0...v11.9.0) (2018-05-31)
+
+
+### Features
+
+* add option that allows instrument to exit on error ([#850](https://github.com/istanbuljs/nyc/issues/850)) ([1329a3b](https://github.com/istanbuljs/nyc/commit/1329a3b))
+
+
+
 <a name="11.8.0"></a>
 # [11.8.0](https://github.com/istanbuljs/nyc/compare/v11.7.3...v11.8.0) (2018-05-14)
 

package/index.js

@@ -1,3 +1,5 @@
+'use strict'
+
 /* global __coverage__ */
 
 const arrify = require('arrify')
@@ -268,25 +270,27 @@ NYC.prototype._maybeInstrumentSource = function (code, filename, relFile) {
 }
 
 NYC.prototype._transformFactory = function (cacheDir) {
-  var _this = this
-  var instrumenter = this.instrumenter()
-  var instrumented
+  const instrumenter = this.instrumenter()
+  let instrumented
 
-  return function (code, metadata, hash) {
-    var filename = metadata.filename
-    var sourceMap = null
+  return (code, metadata, hash) => {
+    const filename = metadata.filename
+    let sourceMap = null
 
-    if (_this._sourceMap) sourceMap = _this.sourceMaps.extractAndRegister(code, filename, hash)
+    if (this._sourceMap) sourceMap = this.sourceMaps.extractAndRegister(code, filename, hash)
 
     try {
       instrumented = instrumenter.instrumentSync(code, filename, sourceMap)
     } catch (e) {
-      // don't fail external tests due to instrumentation bugs.
       debugLog('failed to instrument ' + filename + 'with error: ' + e.stack)
-      instrumented = code
+      if (this.config.exitOnError) {
+        process.exit(1)
+      } else {
+        instrumented = code
+      }
     }
 
-    if (_this.fakeRequire) {
+    if (this.fakeRequire) {
       return 'function x () {}'
     } else {
       return instrumented

package/lib/commands/instrument.js

@@ -46,6 +46,11 @@ exports.builder = function (yargs) {
       type: 'boolean',
       description: 'should nyc handle instrumentation?'
     })
+    .option('exit-on-error', {
+      default: false,
+      type: 'boolean',
+      description: 'should nyc exit when an instrumentation failure occurs?'
+    })
     .example('$0 instrument ./lib ./output', 'instrument all .js files in ./lib with coverage and output in ./output')
 }
 
@@ -62,7 +67,8 @@ exports.handler = function (argv) {
     extension: argv.extension,
     require: argv.require,
     compact: argv.compact,
-    preserveComments: argv.preserveComments
+    preserveComments: argv.preserveComments,
+    exitOnError: argv.exitOnError
   })
 
   nyc.instrumentAllFiles(argv.input, argv.output, function (err) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nyc",
-  "version": "11.8.0",
+  "version": "11.9.0",
   "description": "the Istanbul command line interface",
   "main": "index.js",
   "scripts": {