feat: add coverage to default excludes (#23)

BREAKING CHANGE: additional coverage folder is now excluded

Author: JaKXz

index.js

@@ -60,7 +60,7 @@ TestExclude.prototype.shouldInstrument = function (filename, relFile) {
   // Don't instrument files that are outside of the current working directory.
   if (/^\.\./.test(path.relative(this.cwd, filename))) return false
 
-  relFile = relFile.replace(/^\.[\\\/]/, '') // remove leading './' or '.\'.
+  relFile = relFile.replace(/^\.[\\/]/, '') // remove leading './' or '.\'.
   return (!this.include || micromatch.any(relFile, this.include, {dotfiles: true})) && !micromatch.any(relFile, this.exclude, {dotfiles: true})
 }
 
@@ -98,6 +98,7 @@ var exportFunc = function (opts) {
 }
 
 exportFunc.defaultExclude = [
+  'coverage/**',
   'test/**',
   'test{,-*}.js',
   '**/*.test.js',

package.json

@@ -5,7 +5,7 @@
   "main": "index.js",
   "scripts": {
     "pretest": "standard",
-    "test": "nyc mocha ./test/*.js",
+    "test": "nyc mocha",
     "release": "standard-version"
   },
   "repository": {
@@ -43,7 +43,7 @@
   "nyc": {
     "check-coverage": true,
     "statements": 100,
-    "branches": 96,
+    "branches": 97,
     "lines": 100,
     "functions": 100,
     "reporter": [

test/test-exclude.js

@@ -23,6 +23,10 @@ describe('testExclude', function () {
     exclude().shouldInstrument('../foo.js').should.equal(false)
   })
 
+  it('does not instrument files in the coverage folder by default', function () {
+    exclude().shouldInstrument('coverage/foo.js').should.equal(false)
+  })
+
   it('applies exclude rule ahead of include rule', function () {
     const e = exclude({
       include: ['test.js', 'foo.js'],
@@ -84,6 +88,7 @@ describe('testExclude', function () {
 
   it('exports defaultExclude', function () {
     exclude.defaultExclude.should.deep.equal([
+      'coverage/**',
       'test/**',
       'test{,-*}.js',
       '**/*.test.js',