feat: add additional patterns to default excludes (#133)

istanbuljs · Feb 13, 2018 · 4cedf63 · 4cedf63
1 parent 6cac849
commit 4cedf63
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/packages/test-exclude/index.js b/packages/test-exclude/index.js
@@ -125,9 +125,10 @@ var exportFunc = function (opts) {
 
 exportFunc.defaultExclude = [
   'coverage/**',
+  'packages/*/test/**',
   'test/**',
   'test{,-*}.js',
-  '**/*.test.js',
+  '**/*{.,-}test.js',
   '**/__tests__/**',
   '**/node_modules/**'
 ]

diff --git a/packages/test-exclude/test/test-exclude.js b/packages/test-exclude/test/test-exclude.js
@@ -115,9 +115,10 @@ describe('testExclude', function () {
   it('exports defaultExclude', function () {
     exclude.defaultExclude.should.deep.equal([
       'coverage/**',
+      'packages/*/test/**',
       'test/**',
       'test{,-*}.js',
-      '**/*.test.js',
+      '**/*{.,-}test.js',
       '**/__tests__/**',
       '**/node_modules/**'
     ])
@@ -167,6 +168,9 @@ describe('testExclude', function () {
 
       e.shouldInstrument('test.js').should.equal(false)
       e.shouldInstrument('src/app.test.js').should.equal(false)
+      e.shouldInstrument('src/app-test.js').should.equal(false)
+
+      e.shouldInstrument('packages/package-name/test/test-utils.js').should.equal(false)
 
       e.shouldInstrument('bar/baz.js').should.equal(true)
       e.shouldInstrument('bad/file.js').should.equal(true)
@@ -206,6 +210,9 @@ describe('testExclude', function () {
 
         e.shouldInstrument('test.js').should.equal(false)
         e.shouldInstrument('src/app.test.js').should.equal(false)
+        e.shouldInstrument('src/app-test.js').should.equal(false)
+
+        e.shouldInstrument('packages/package-name/test/test-utils.js').should.equal(false)
 
         e.shouldInstrument('bar/baz.js').should.equal(true)
         e.shouldInstrument('bad/file.js').should.equal(true)