fix: respect changes of cwd in options (#171)

istanbuljs · Oct 16, 2018 · adec723 · adec723
1 parent 61d3c44
commit adec723
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 2 deletions.
diff --git a/fixtures/should-respect-cwd.js b/fixtures/should-respect-cwd.js
@@ -0,0 +1,4 @@
+let foo = function () {
+  console.log('foo')
+}
+foo()
diff --git a/src/index.js b/src/index.js
@@ -16,7 +16,7 @@ function getRealpath (n) {
 function makeShouldSkip () {
   let exclude
   return function shouldSkip (file, opts) {
-    if (!exclude) {
+    if (!exclude || exclude.cwd !== opts.cwd) {
       const cwd = getRealpath(process.env.NYC_CWD || process.cwd())
       const nycConfig = process.env.NYC_CONFIG ? JSON.parse(process.env.NYC_CONFIG) : {}
 
@@ -44,7 +44,6 @@ function makeShouldSkip () {
         config
       ))
     }
-
     return !exclude.shouldInstrument(file)
   }
 }

diff --git a/test/babel-plugin-istanbul.js b/test/babel-plugin-istanbul.js
@@ -148,5 +148,27 @@ describe('babel-plugin-istanbul', function () {
       })
       result.code.match(/statementMap/)
     })
+
+    it('should respect a changed cwd in options', function () {
+      const opts = {
+        cwd: path.resolve(__dirname, '..', 'lib')
+      }
+      const plugins = [
+        [makeVisitor, opts]
+      ]
+
+      var resultBefore = babel.transformFileSync('./fixtures/should-respect-cwd.js', {
+        plugins
+      })
+
+      resultBefore.code.should.not.match(/statementMap/)
+
+      opts.cwd = path.resolve(__dirname, '..', 'fixtures')
+
+      var resultAfter = babel.transformFileSync('./fixtures/should-respect-cwd.js', {
+        plugins
+      })
+      resultAfter.code.should.match(/statementMap/)
+    })
   })
 })