fix: exclude coverage of the CJS-ESM bridge from results

[shinnn]

Fixes #14, by excluding results that contains coverage of the CJS-ESM bridge.

Comparison

main.mjs:

import lib from './lib.cjs';

lib();

lib.cjs:

module.exports = () => 'Hi';

The current master 02d59272902

$ node bin/c8.js node --experimental-modules main.mjs
----------|----------|----------|----------|----------|-------------------|
File      |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
----------|----------|----------|----------|----------|-------------------|
All files |      100 |      100 |        0 |      100 |                   |
 lib.cjs  |      100 |      100 |        0 |      100 |                   |
 main.mjs |      100 |      100 |      100 |      100 |                   |
----------|----------|----------|----------|----------|-------------------|

This branch

$ node bin/c8.js node --experimental-modules main.mjs
----------|----------|----------|----------|----------|-------------------|
File      |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
----------|----------|----------|----------|----------|-------------------|
All files |      100 |      100 |      100 |      100 |                   |
 lib.cjs  |      100 |      100 |      100 |      100 |                   |
 main.mjs |      100 |      100 |      100 |      100 |                   |
----------|----------|----------|----------|----------|-------------------|

Checklist

• npm test, tests passing
• npm run test:snap (to update the snapshot)
• tests and/or benchmarks are included

[coveralls]

Pull Request Test Coverage Report for Build 213

• 36 of 36 (100.0%) changed or added relevant lines in 2 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.3%) to 96.947%

---

Totals
Change from base Build 210:	0.3%
Covered Lines:	429
Relevant Lines:	436

---

💛 - Coveralls

[shinnn]

The V8 coverage of the CJS-ESM bridge looks like:

[
  {
    functionName: '',
    ranges: [ [Object] ],
    isBlockCoverage: true
  },
  {
    functionName: 'get',
    ranges: [ [Object] ],
    isBlockCoverage: false
  },
  {
    functionName: 'set',
    ranges: [ [Object] ],
    isBlockCoverage: true
  }
]

[bcoe]

this seems a little fragile ... also, if we need to use feature detection, let's pull the behavior into a new file, like detect-cjs-esm-bridge.js or something along those lines.

[shinnn]

Yes, it is fragile... but better than doing nothing.

let's pull the behavior into a new file

Done. b1fb3fc

[bcoe]

@shinnn mind rebasing with master, just landed the source-map work.

[shinnn]

Rebased.

[bcoe]

@shinnn awesome \o/ thank you for all your help recently.

[shinnn]

@bcoe No problem. I'd be glad if you publish a new version to npm :)