feat: exclude unexpected file in sourcemap result

bcoe · Jan 24, 2022 · 9b25d8e · 9b25d8e
1 parent 6ca0e6a
commit 9b25d8e
Show file tree

Hide file tree

Showing 5 changed files with 97 additions and 0 deletions.
diff --git a/lib/report.js b/lib/report.js
@@ -127,6 +127,15 @@ class Report {
         map.merge(converter.toIstanbul())
       }
     }
+
+    if (!this.all) {
+      for (const fileURL in map.data) {
+        if (!this.exclude.shouldInstrument(fileURL)) {
+          delete map.data[fileURL]
+        }
+      }
+    }
+
     this._allCoverageFiles = map
     return this._allCoverageFiles
   }

diff --git a/test/fixtures/bundle.js b/test/fixtures/bundle.js
diff --git a/test/fixtures/bundle.js.map b/test/fixtures/bundle.js.map
diff --git a/test/integration.js b/test/integration.js
@@ -646,4 +646,16 @@ describe('c8', () => {
       output.toString('utf8').should.matchSnapshot()
     })
   })
+
+  it('exclude unexpected file in result', () => {
+    const { output } = spawnSync(nodePath, [
+      c8Path,
+      '--exclude="test/*.js"',
+      '--temp-directory=tmp/bundle',
+      '--clean=false',
+      nodePath,
+      require.resolve('./fixtures/bundle.js')
+    ])
+    output.toString('utf8').should.matchSnapshot() 
+  })
 })
diff --git a/test/integration.js.snap b/test/integration.js.snap
@@ -330,6 +330,17 @@ All files   |      75 |       50 |     100 |      75 |
 ,"
 `;
 
+exports[`c8 exclude unexpected file in result 1`] = `
+",reachable
+----------|---------|----------|---------|---------|-------------------
+File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
+----------|---------|----------|---------|---------|-------------------
+All files |     100 |      100 |     100 |     100 |                   
+ index.js |     100 |      100 |     100 |     100 |                   
+----------|---------|----------|---------|---------|-------------------
+,"
+`;
+
 exports[`c8 merges reports from subprocesses together 1`] = `
 ",first