.github/FUNDING.yml

@@ -0,0 +1 @@
+tidelift: "npm/babel-plugin-istanbul"

.gitignore

@@ -31,6 +31,7 @@ build/Release
 
 # Dependency directories
 node_modules
+!fixtures/node_modules
 jspm_packages
 
 # Optional npm cache directory

.travis.yml

@@ -1,11 +1,21 @@
 language: node_js
 
 os:
+  - windows
+  - osx
   - linux
 
-after_success: npm run coverage
-
 node_js:
-  - "4"
-  - "6"
-  - "stable"
+  - "node"
+  - "10"
+  - "8"
+
+matrix:
+  include:
+    - os: windows
+      node_js: "latest"
+  exclude:
+    - os: windows
+      node_js: "node"
+
+after_success: npm run coverage

CHANGELOG.md

@@ -1,7 +1,95 @@
-# Change Log
+# Changelog
 
 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.
 
+## [6.0.0](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v5.2.0...v6.0.0) (2019-12-20)
+
+
+### ⚠ BREAKING CHANGES
+
+* Drop node.js 6 (#226)
+
+### Features
+
+* Add support for instrumenter options ([#227](https://github.com/istanbuljs/babel-plugin-istanbul/issues/227)) ([fe08f5b](https://github.com/istanbuljs/babel-plugin-istanbul/commit/fe08f5b8282136c7ed9375fa32148586bd6a7e28)), closes [#208](https://github.com/istanbuljs/babel-plugin-istanbul/issues/208) [#212](https://github.com/istanbuljs/babel-plugin-istanbul/issues/212)
+* Drop node.js 6 ([#226](https://github.com/istanbuljs/babel-plugin-istanbul/issues/226)) ([93db21a](https://github.com/istanbuljs/babel-plugin-istanbul/commit/93db21aa2bbdbb06fb784f52c24a7847fad6be92)), closes [#209](https://github.com/istanbuljs/babel-plugin-istanbul/issues/209)
+
+## [5.2.0](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v5.1.4...v5.2.0) (2019-07-18)
+
+
+### Bug Fixes
+
+* Print explicit error if use with babel 6 is attempted ([#207](https://github.com/istanbuljs/babel-plugin-istanbul/issues/207)) ([a12cf16](https://github.com/istanbuljs/babel-plugin-istanbul/commit/a12cf16))
+
+
+### Features
+
+* Support turning off node_modules default exclude via flag ([#172](https://github.com/istanbuljs/babel-plugin-istanbul/issues/172)) ([a314f06](https://github.com/istanbuljs/babel-plugin-istanbul/commit/a314f06))
+
+
+
+## [5.1.4](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v5.1.3...v5.1.4) (2019-04-25)
+
+
+
+## [5.1.3](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v5.1.2...v5.1.3) (2019-04-23)
+
+
+### Bug Fixes
+
+* Revert [#192](https://github.com/istanbuljs/babel-plugin-istanbul/issues/192) ([201a933](https://github.com/istanbuljs/babel-plugin-istanbul/commit/201a933)) due to regression detailed at [#201](https://github.com/istanbuljs/babel-plugin-istanbul/issues/201)
+
+
+
+## [5.1.2](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v5.1.1...v5.1.2) (2019-04-10)
+
+
+### Bug Fixes
+
+* Ensure correct scope references after traversal ([#192](https://github.com/istanbuljs/babel-plugin-istanbul/issues/192)) ([201a933](https://github.com/istanbuljs/babel-plugin-istanbul/commit/201a933))
+
+
+
+<a name="5.1.1"></a>
+## [5.1.1](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v5.1.0...v5.1.1) (2019-01-11)
+
+
+### Bug Fixes
+
+* respect changes of cwd in options ([#171](https://github.com/istanbuljs/babel-plugin-istanbul/issues/171)) ([adec723](https://github.com/istanbuljs/babel-plugin-istanbul/commit/adec723))
+
+
+
+<a name="5.1.0"></a>
+# [5.1.0](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v5.0.1...v5.1.0) (2018-09-07)
+
+
+### Features
+
+* Bump dependencies. ([b509649](https://github.com/istanbuljs/babel-plugin-istanbul/commit/b509649))
+
+
+
+<a name="5.0.1"></a>
+## [5.0.1](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v5.0.0...v5.0.1) (2018-07-24)
+
+
+
+<a name="5.0.0"></a>
+# [5.0.0](https://github.com/istanbuljs/babel-plugin-istanbul/compare/v4.1.6...v5.0.0) (2018-06-27)
+
+
+### Features
+
+* upgrade to babel 7 and newest istanbul libraries ([#158](https://github.com/istanbuljs/babel-plugin-istanbul/issues/158)) ([a9e1564](https://github.com/istanbuljs/babel-plugin-istanbul/commit/a9e1564))
+
+
+### BREAKING CHANGES
+
+* Drop node 4 support, upgrade to babel 7, stop shipping @babel/plugin-syntax-object-rest-spread
+
+
+
 <a name="4.1.6"></a>
 ## <small>4.1.6 (2018-03-09)</small>
 

README.md

@@ -129,3 +129,9 @@ function instrument(sourceCode, sourceMap, fileName) {
 ## Credit where credit is due
 
 The approach used in `babel-plugin-istanbul` was inspired by [Thai Pangsakulyanont](https://github.com/dtinth)'s original library [`babel-plugin-__coverage__`](https://github.com/dtinth/babel-plugin-__coverage__).
+
+## `babel-plugin-istanbul` for enterprise
+
+Available as part of the Tidelift Subscription.
+
+The maintainers of `babel-plugin-istanbul` and thousands of other packages are working with Tidelift to deliver commercial support and maintenance for the open source dependencies you use to build your applications. Save time, reduce risk, and improve code health, while paying the maintainers of the exact dependencies you use. [Learn more.](https://tidelift.com/subscription/pkg/npm-babel-plugin-istanbul?utm_source=npm-babel-plugin-istanbul&utm_medium=referral&utm_campaign=enterprise&utm_term=repo)

babel.config.js

@@ -0,0 +1,12 @@
+'use strict'
+
+module.exports = {
+  plugins: [
+    '@babel/plugin-transform-modules-commonjs'
+  ],
+  env: {
+    test: {
+      plugins: ['./lib']
+    }
+  }
+}

fixtures/class-functions.js

@@ -0,0 +1,11 @@
+class Foo1 {
+  bar() {}
+  barz() {}
+}
+
+class Foo2 {
+  bar() {}
+  barz() {}
+}
+
+module.exports = { Foo1, Foo2 }

fixtures/config/file1.js

@@ -0,0 +1 @@
+console.log('file1')

fixtures/config/file2.js

@@ -0,0 +1 @@
+console.log('file2')

fixtures/config/nyc-alt.config.js

@@ -0,0 +1,5 @@
+'use strict'
+
+module.exports = {
+  include: ['file2.js']
+}

fixtures/config/nyc.config.js

@@ -0,0 +1,5 @@
+'use strict'
+
+module.exports = {
+  include: ['file1.js']
+}

fixtures/config/package.json

@@ -0,0 +1 @@
+{}

fixtures/issue-201.js

@@ -0,0 +1,9 @@
+import { resolve } from 'path'
+
+function test () {
+  const _path = 'path'
+
+  return resolve(_path)
+}
+
+test()

fixtures/plugin-should-cover.js

@@ -1,4 +1,4 @@
-let foo = function () {
+const foo = function () {
   console.log('foo')
 }
 foo()

fixtures/plugin-should-not-cover.js

@@ -1,4 +1,4 @@
-let bar = function () {
+const bar = function () {
   console.log('bar')
 }
 bar()

fixtures/should-cover.js

@@ -1,4 +1,4 @@
-let foo = function () {
+const foo = function () {
   console.log('foo')
 }
 foo()

fixtures/should-not-cover.js

@@ -1,4 +1,4 @@
-let bar = function () {
+const bar = function () {
   console.log('bar')
 }
 bar()

fixtures/should-respect-cwd.js

@@ -0,0 +1,4 @@
+const foo = function () {
+  console.log('foo')
+}
+foo()

package.json

@@ -1,6 +1,6 @@
 {
   "name": "babel-plugin-istanbul",
-  "version": "4.1.6",
+  "version": "6.0.0",
   "author": "Thai Pangsakulyanont @dtinth",
   "license": "BSD-3-Clause",
   "description": "A babel plugin that adds istanbul instrumentation to ES6 code",
@@ -9,36 +9,37 @@
     "lib"
   ],
   "dependencies": {
-    "babel-plugin-syntax-object-rest-spread": "^6.13.0",
-    "find-up": "^2.1.0",
-    "istanbul-lib-instrument": "^1.10.1",
-    "test-exclude": "^4.2.1"
+    "@babel/helper-plugin-utils": "^7.0.0",
+    "@istanbuljs/load-nyc-config": "^1.0.0",
+    "@istanbuljs/schema": "^0.1.2",
+    "istanbul-lib-instrument": "^4.0.0",
+    "test-exclude": "^6.0.0"
   },
   "devDependencies": {
-    "babel-cli": "^6.18.0",
-    "babel-core": "^6.24.0",
-    "babel-preset-env": "^1.6.1",
-    "babel-register": "^6.24.0",
-    "chai": "^4.1.0",
-    "coveralls": "^3.0.0",
-    "cross-env": "^3.1.4",
-    "mocha": "^4.0.0",
-    "nyc": "^11.1.0",
+    "@babel/cli": "^7.7.5",
+    "@babel/core": "^7.7.5",
+    "@babel/plugin-transform-modules-commonjs": "^7.7.5",
+    "@babel/register": "^7.7.4",
+    "chai": "^4.2.0",
+    "coveralls": "^3.0.9",
+    "cross-env": "^6.0.3",
+    "mocha": "^6.2.2",
+    "nyc": "^15.0.0",
     "pmock": "^0.2.3",
-    "standard": "^9.0.2",
-    "standard-version": "^4.0.0"
+    "standard": "^14.3.1",
+    "standard-version": "^7.1.0"
   },
   "scripts": {
     "coverage": "nyc report --reporter=text-lcov | coveralls",
     "release": "babel src --out-dir lib",
     "pretest": "standard && npm run release",
-    "test": "cross-env NODE_ENV=test nyc --reporter=lcov --reporter=text mocha test/*.js",
+    "test": "cross-env NODE_ENV=test nyc --reporter=lcov --reporter=text mocha --timeout 5000 test/*.js",
     "prepublish": "npm test && npm run release",
     "version": "standard-version"
   },
   "standard": {
     "ignore": [
-      "fixtures/has-inline-source-map.js"
+      "fixtures/*.js"
     ]
   },
   "repository": {
@@ -57,7 +58,7 @@
       "fixtures/should-cover.js"
     ],
     "require": [
-      "babel-register"
+      "@babel/register"
     ],
     "sourceMap": false,
     "instrument": false
@@ -66,10 +67,7 @@
     "url": "https://github.com/istanbuljs/babel-plugin-istanbul/issues"
   },
   "homepage": "https://github.com/istanbuljs/babel-plugin-istanbul#readme",
-  "greenkeeper": {
-    "ignore": [
-      "find-up",
-      "cross-env"
-    ]
+  "engines": {
+    "node": ">=8"
   }
 }

src/index.js

@@ -1,73 +1,130 @@
-import {realpathSync} from 'fs'
-import {dirname} from 'path'
-import {programVisitor} from 'istanbul-lib-instrument'
-import babelSyntaxObjectRestSpread from 'babel-plugin-syntax-object-rest-spread'
-
-const testExclude = require('test-exclude')
-const findUp = require('find-up')
+import path from 'path'
+import { realpathSync } from 'fs'
+import { execFileSync } from 'child_process'
+import { declare } from '@babel/helper-plugin-utils'
+import { programVisitor } from 'istanbul-lib-instrument'
+import TestExclude from 'test-exclude'
+import schema from '@istanbuljs/schema'
 
 function getRealpath (n) {
   try {
-    return realpathSync(n) || n
+    return realpathSync(n) || /* istanbul ignore next */ n
   } catch (e) {
+    /* istanbul ignore next */
     return n
   }
 }
 
+const memoize = new Map()
+/* istanbul ignore next */
+const memosep = path.sep === '/' ? ':' : ';'
+
+function loadNycConfig (cwd, opts) {
+  let memokey = cwd
+  const args = [
+    path.resolve(__dirname, 'load-nyc-config-sync.js'),
+    cwd
+  ]
+
+  if ('nycrcPath' in opts) {
+    args.push(opts.nycrcPath)
+
+    memokey += memosep + opts.nycrcPath
+  }
+
+  /* execFileSync is expensive, avoid it if possible! */
+  if (memoize.has(memokey)) {
+    return memoize.get(memokey)
+  }
+
+  const result = JSON.parse(execFileSync(process.execPath, args))
+  const error = result['load-nyc-config-sync-error']
+  if (error) {
+    throw new Error(error)
+  }
+
+  const config = {
+    ...schema.defaults.babelPluginIstanbul,
+    cwd,
+    ...result
+  }
+  memoize.set(memokey, config)
+  return config
+}
+
+function findConfig (opts) {
+  const cwd = getRealpath(opts.cwd || process.env.NYC_CWD || /* istanbul ignore next */ process.cwd())
+  const keys = Object.keys(opts)
+  const ignored = Object.keys(opts).filter(s => s === 'nycrcPath' || s === 'cwd')
+  if (keys.length > ignored.length) {
+    // explicitly configuring options in babel
+    // takes precedence.
+    return {
+      ...schema.defaults.babelPluginIstanbul,
+      cwd,
+      ...opts
+    }
+  }
+
+  if (ignored.length === 0 && process.env.NYC_CONFIG) {
+    // defaults were already applied by nyc
+    return JSON.parse(process.env.NYC_CONFIG)
+  }
+
+  return loadNycConfig(cwd, opts)
+}
+
 function makeShouldSkip () {
   let exclude
-  return function shouldSkip (file, opts) {
-    if (!exclude) {
-      const cwd = getRealpath(process.env.NYC_CWD || process.cwd())
-      const nycConfig = process.env.NYC_CONFIG ? JSON.parse(process.env.NYC_CONFIG) : {}
-
-      let config = {}
-      if (Object.keys(opts).length > 0) {
-        // explicitly configuring options in babel
-        // takes precedence.
-        config = opts
-      } else if (nycConfig.include || nycConfig.exclude) {
-        // nyc was configured in a parent process (keep these settings).
-        config = {
-          include: nycConfig.include,
-          exclude: nycConfig.exclude
-        }
-      } else {
-        // fallback to loading config from key in package.json.
-        config = {
-          configKey: 'nyc',
-          configPath: dirname(findUp.sync('package.json', { cwd }))
-        }
-      }
 
-      exclude = testExclude(Object.assign(
-        { cwd },
-        config
-      ))
+  return function shouldSkip (file, nycConfig) {
+    if (!exclude || (exclude.cwd !== nycConfig.cwd)) {
+      exclude = new TestExclude({
+        cwd: nycConfig.cwd,
+        include: nycConfig.include,
+        exclude: nycConfig.exclude,
+        extension: nycConfig.extension,
+        // Make sure this is true unless explicitly set to `false`. `undefined` is still `true`.
+        excludeNodeModules: nycConfig.excludeNodeModules !== false
+      })
     }
 
     return !exclude.shouldInstrument(file)
   }
 }
 
-function makeVisitor ({types: t}) {
+export default declare(api => {
+  api.assertVersion(7)
+
   const shouldSkip = makeShouldSkip()
+
+  const t = api.types
   return {
-    inherits: babelSyntaxObjectRestSpread,
     visitor: {
       Program: {
         enter (path) {
           this.__dv__ = null
+          this.nycConfig = findConfig(this.opts)
           const realPath = getRealpath(this.file.opts.filename)
-          if (shouldSkip(realPath, this.opts)) {
+          if (shouldSkip(realPath, this.nycConfig)) {
             return
           }
           let { inputSourceMap } = this.opts
           if (this.opts.useInlineSourceMaps !== false) {
-            inputSourceMap = inputSourceMap || this.file.opts.inputSourceMap
+            if (!inputSourceMap && this.file.inputMap) {
+              inputSourceMap = this.file.inputMap.sourcemap
+            }
           }
+          const visitorOptions = {}
+          Object.entries(schema.defaults.instrumentVisitor).forEach(([name, defaultValue]) => {
+            if (name in this.nycConfig) {
+              visitorOptions[name] = this.nycConfig[name]
+            } else {
+              visitorOptions[name] = schema.defaults.instrumentVisitor[name]
+            }
+          })
           this.__dv__ = programVisitor(t, realPath, {
-            coverageVariable: '__coverage__',
+            ...visitorOptions,
             inputSourceMap
           })
           this.__dv__.enter(path)
@@ -84,6 +141,4 @@ function makeVisitor ({types: t}) {
       }
     }
   }
-}
-
-export default makeVisitor
+})

src/load-nyc-config-sync.js

@@ -0,0 +1,14 @@
+#!/usr/bin/env node
+'use strict'
+
+const { loadNycConfig } = require('@istanbuljs/load-nyc-config')
+
+async function main () {
+  const [cwd, nycrcPath] = process.argv.slice(2)
+
+  console.log(JSON.stringify(await loadNycConfig({ cwd, nycrcPath })))
+}
+
+main().catch(error => {
+  console.log(JSON.stringify({ 'load-nyc-config-sync-error': error.message }))
+})

test/babel-plugin-istanbul.js

@@ -1,6 +1,6 @@
 /* eslint-env mocha */
 
-const babel = require('babel-core')
+import * as babel from '@babel/core'
 import makeVisitor from '../src'
 import path from 'path'
 
@@ -10,8 +10,10 @@ describe('babel-plugin-istanbul', function () {
   context('Babel plugin config', function () {
     it('should instrument file if shouldSkip returns false', function () {
       var result = babel.transformFileSync('./fixtures/plugin-should-cover.js', {
+        babelrc: false,
+        configFile: false,
         plugins: [
-          [makeVisitor({types: babel.types}), {
+          [makeVisitor, {
             include: ['fixtures/plugin-should-cover.js']
           }]
         ]
@@ -21,20 +23,54 @@ describe('babel-plugin-istanbul', function () {
 
     it('should not instrument file if shouldSkip returns true', function () {
       var result = babel.transformFileSync('./fixtures/plugin-should-not-cover.js', {
+        babelrc: false,
+        configFile: false,
         plugins: [
-          [makeVisitor({types: babel.types}), {
+          [makeVisitor, {
             include: ['fixtures/plugin-should-cover.js']
           }]
         ]
       })
       result.code.should.not.match(/statementMap/)
     })
 
+    context('local node_modules', function () {
+      it('should instrument file if shouldSkip returns false', function () {
+        var result = babel.transformFileSync('./fixtures/node_modules/should-cover.js', {
+          babelrc: false,
+          configFile: false,
+          plugins: [
+            [makeVisitor, {
+              excludeNodeModules: false,
+              exclude: ['node_modules/**'],
+              include: ['fixtures/node_modules/should-cover.js']
+            }]
+          ]
+        })
+        result.code.should.match(/statementMap/)
+      })
+
+      it('should not instrument file if shouldSkip returns true', function () {
+        var result = babel.transformFileSync('./fixtures/node_modules/should-not-cover.js', {
+          babelrc: false,
+          configFile: false,
+          plugins: [
+            [makeVisitor, {
+              include: ['fixtures/node_modules/should-not-cover.js']
+            }]
+          ]
+        })
+        result.code.should.not.match(/statementMap/)
+      })
+    })
+
     it('should call onCover callback', function () {
       var args
       babel.transformFileSync('./fixtures/plugin-should-cover.js', {
+        babelrc: false,
+        configFile: false,
         plugins: [
-          [makeVisitor({types: babel.types}), {
+          [makeVisitor, {
             onCover: function () {
               args = [].slice.call(arguments)
             },
@@ -50,8 +86,10 @@ describe('babel-plugin-istanbul', function () {
   context('source maps', function () {
     it('should use inline source map', function () {
       var result = babel.transformFileSync('./fixtures/has-inline-source-map.js', {
+        babelrc: false,
+        configFile: false,
         plugins: [
-          [makeVisitor({types: babel.types}), {
+          [makeVisitor, {
             include: ['fixtures/has-inline-source-map.js']
           }]
         ]
@@ -61,8 +99,10 @@ describe('babel-plugin-istanbul', function () {
 
     it('should not use inline source map if inputSourceMap is set to false', function () {
       var result = babel.transformFileSync('./fixtures/has-inline-source-map.js', {
+        babelrc: false,
+        configFile: false,
         plugins: [
-          [makeVisitor({types: babel.types}), {
+          [makeVisitor, {
             include: ['fixtures/has-inline-source-map.js'],
             useInlineSourceMaps: false
           }]
@@ -73,8 +113,10 @@ describe('babel-plugin-istanbul', function () {
 
     it('should use provided source map', function () {
       var result = babel.transformFileSync('./fixtures/has-inline-source-map.js', {
+        babelrc: false,
+        configFile: false,
         plugins: [
-          [makeVisitor({types: babel.types}), {
+          [makeVisitor, {
             include: ['fixtures/has-inline-source-map.js'],
             inputSourceMap: { asdfQwer: 'foo' }
           }]
@@ -84,53 +126,172 @@ describe('babel-plugin-istanbul', function () {
     })
   })
 
+  context('instrument options', function () {
+    it('should honor coverageVariable option', function () {
+      const result = babel.transformFileSync('./fixtures/should-cover.js', {
+        babelrc: false,
+        configFile: false,
+        plugins: [
+          [makeVisitor, {
+            include: ['fixtures/should-cover.js'],
+            coverageVariable: '__TEST_VARIABLE__'
+          }]
+        ]
+      })
+      result.code.should.match(/__TEST_VARIABLE__/)
+      result.code.should.not.match(/__coverage__/)
+    })
+
+    it('should honor coverageGlobalScope option', function () {
+      const result = babel.transformFileSync('./fixtures/should-cover.js', {
+        babelrc: false,
+        configFile: false,
+        plugins: [
+          [makeVisitor, {
+            include: ['fixtures/should-cover.js'],
+            coverageGlobalScope: 'window'
+          }]
+        ]
+      })
+      result.code.should.match(/new Function\("return window"\)/)
+      result.code.should.not.match(/new Function\("return this"\)/)
+    })
+
+    it('should honor coverageGlobalScope option', function () {
+      const result = babel.transformFileSync('./fixtures/should-cover.js', {
+        babelrc: false,
+        configFile: false,
+        plugins: [
+          [makeVisitor, {
+            include: ['fixtures/should-cover.js'],
+            coverageGlobalScopeFunc: false
+          }]
+        ]
+      })
+      result.code.should.match(/global\s*=\s*this/)
+      result.code.should.not.match(/global\s*=\s*new Function\("return this"\)/)
+    })
+
+    it('should honor ignoreClassMethods option', function () {
+      const result = babel.transformFileSync('./fixtures/class-functions.js', {
+        babelrc: false,
+        configFile: false,
+        plugins: [
+          [makeVisitor, {
+            include: ['fixtures/class-functions.js'],
+            ignoreClassMethods: ['bar']
+          }]
+        ]
+      })
+
+      // bar() is ignored
+      result.code.should.match(/bar\(\)\s*{\s*}/)
+      result.code.should.not.match(/bar\(\)\s*{\s*cov_.*/)
+
+      // barz() does not get instrumented
+      result.code.should.match(/barz\(\)\s*{\s*cov_.*/)
+      result.code.should.not.match(/barz\(\)\s*{\s*}/)
+    })
+  })
+
   context('package.json "nyc" config', function () {
     context('process.env.NYC_CONFIG is set', function () {
       it('should instrument file if shouldSkip returns false', function () {
         var result = babel.transformFileSync('./fixtures/should-cover.js', {
+          babelrc: false,
+          configFile: false,
           plugins: [
-            makeVisitor({types: babel.types})
+            makeVisitor
           ]
         })
         result.code.should.match(/statementMap/)
       })
 
       it('should not instrument file if shouldSkip returns true', function () {
         var result = babel.transformFileSync('./fixtures/should-not-cover.js', {
+          babelrc: false,
+          configFile: false,
           plugins: [
-            makeVisitor({types: babel.types})
+            makeVisitor
           ]
         })
         result.code.should.not.match(/statementMap/)
       })
     })
 
     context('process.env.NYC_CONFIG is not set', function () {
+      this.timeout(10000)
+
       const OLD_NYC_CONFIG = process.env.NYC_CONFIG
+      const OLD_NYC_CWD = process.env.NYC_CWD
       before(() => {
         delete process.env.NYC_CONFIG
+        delete process.env.NYC_CWD
       })
       after(() => {
         process.env.NYC_CONFIG = OLD_NYC_CONFIG
+        process.env.NYC_CWD = OLD_NYC_CWD
       })
 
       it('should instrument file if shouldSkip returns false', function () {
         var result = babel.transformFileSync('./fixtures/should-cover.js', {
+          babelrc: false,
+          configFile: false,
           plugins: [
-            makeVisitor({types: babel.types})
+            makeVisitor
           ]
         })
         result.code.should.match(/statementMap/)
       })
 
       it('should not instrument file if shouldSkip returns true', function () {
         var result = babel.transformFileSync('./fixtures/should-not-cover.js', {
+          babelrc: false,
+          configFile: false,
           plugins: [
-            makeVisitor({types: babel.types})
+            makeVisitor
           ]
         })
         result.code.should.not.match(/statementMap/)
       })
+
+      it('should load config using cwd', function () {
+        const cwd = path.resolve(__dirname, '..', 'fixtures', 'config')
+        function helper (file, match, opts) {
+          const result = babel.transformFileSync(
+            path.resolve(cwd, file),
+            {
+              babelrc: false,
+              configFile: false,
+              plugins: [
+                [makeVisitor, { cwd, ...opts }]
+              ]
+            }
+          )
+          if (match) {
+            result.code.should.match(/statementMap/)
+          } else {
+            result.code.should.not.match(/statementMap/)
+          }
+        }
+
+        helper('file1.js', true)
+        helper('file2.js', false)
+        helper('file1.js', false, { nycrcPath: 'nyc-alt.config.js' })
+        helper('file2.js', true, { nycrcPath: 'nyc-alt.config.js' })
+        ;(function () {
+          babel.transformFileSync(
+            path.resolve(cwd, 'file1.js'),
+            {
+              babelrc: false,
+              configFile: false,
+              plugins: [
+                [makeVisitor, { cwd, nycrcPath: 'missing-config.js' }]
+              ]
+            }
+          )
+        }).should.throw(/Requested configuration file missing-config.js not found/)
+      })
     })
   })
 
@@ -140,13 +301,31 @@ describe('babel-plugin-istanbul', function () {
     // regression test for https://github.com/istanbuljs/babel-plugin-istanbul/issues/78
     it('should instrument: export const foo = () => {}', function () {
       var result = babel.transformFileSync('./fixtures/issue-78.js', {
+        babelrc: false,
+        configFile: false,
         plugins: [
-          [makeVisitor({types: babel.types}), {
+          [makeVisitor, {
             include: ['fixtures/issue-78.js']
           }]
         ]
       })
       result.code.match(/statementMap/)
     })
+
+    // regression test for https://github.com/istanbuljs/babel-plugin-istanbul/issues/201
+    it('should not conflict with transform-modules-commonjs', function () {
+      var result = babel.transformFileSync('./fixtures/issue-201.js', {
+        babelrc: false,
+        configFile: false,
+        plugins: [
+          [makeVisitor, {
+            include: ['fixtures/issue-201.js']
+          }],
+          '@babel/plugin-transform-modules-commonjs'
+        ]
+      })
+      result.code.should.match(/_path.*\.resolve\)\(_path\)/)
+      result.code.should.not.match(/_path\.resolve\)\(_path\)/)
+    })
   })
 })