#1577 Add "--exclude" Option

bin/_mocha

@@ -10,6 +10,7 @@
 const program = require('commander');
 const path = require('path');
 const fs = require('fs');
+const minimatch = require('minimatch');
 const resolve = path.resolve;
 const exists = fs.existsSync;
 const Mocha = require('../');
@@ -205,7 +206,8 @@ program
   .option('--allow-uncaught', 'enable uncaught errors to propagate')
   .option('--forbid-only', 'causes test marked with only to fail the suite')
   .option('--forbid-pending', 'causes pending tests and test marked with skip to fail the suite')
-  .option('--file <file>', 'include a file to be ran during the suite', collect, []);
+  .option('--file <file>', 'include a file to be ran during the suite', collect, [])
+  .option('--exclude <files>', 'comma-delimited list of files or glob patterns to ignore', list, []);
 
 program._name = 'mocha';
 
@@ -494,6 +496,13 @@ args.forEach(arg => {
     throw err;
   }
 
+  if (typeof newFiles !== 'undefined') {
+    if (typeof newFiles === 'string') {
+      newFiles = [newFiles];
+    }
+    newFiles = newFiles.filter(fileName => program.exclude.every(pattern => !minimatch(fileName, pattern)));
+  }
+
   files = files.concat(newFiles);
 });
 

package.json

@@ -19,6 +19,7 @@
     "Adrian Ludwig (https://github.com/adrian-ludwig)",
     "Ainthe Kitchen <a.in.the.k@gmail.com> (https://github.com/ainthek)",
     "ajaykodali (https://github.com/ajaykodali)",
+    "Alex Bainter <alexbainter+github@gmail.com> (https://github.com/metalex9)",
     "Alex Early (https://github.com/aearly)",
     "Alex Pham <thedark1337@thedark1337.com> (https://github.com/thedark1337)",
     "amsul (https://github.com/amsul)",
@@ -314,6 +315,7 @@
     "glob": "7.1.2",
     "growl": "1.10.3",
     "he": "1.1.1",
+    "minimatch": "^3.0.4",
     "mkdirp": "0.5.1",
     "supports-color": "4.4.0"
   },

test/integration/fixtures/options/exclude/fail.spec.js

@@ -0,0 +1,7 @@
+'use strict';
+
+describe('exclude test fail', function () {
+  it('should not run this test', function () {
+    throw new Error('should not run');
+  });
+});

test/integration/fixtures/options/exclude/nested/fail.spec.js

@@ -0,0 +1,7 @@
+'use strict';
+
+describe('exclude test nested fail', function () {
+  it('should not run this test', function () {
+    throw new Error('should not run');
+  });
+});

test/integration/fixtures/options/exclude/nested/pass.spec.js

@@ -0,0 +1,5 @@
+'use strict';
+
+describe('exclude test nested pass', function () {
+  it('should find this test', function () {});
+});

test/integration/fixtures/options/exclude/pass.spec.js

@@ -0,0 +1,5 @@
+'use strict';
+
+describe('exclude test pass', function () {
+  it('should find this test', function () {});
+});

test/integration/options.spec.js

@@ -415,4 +415,41 @@ describe('options', function () {
       }, path.join(__dirname, 'fixtures', 'options', 'help'));
     });
   });
+
+  describe('--exclude', function () {
+    /*
+     * Runs mocha in ./fixtures/options/exclude with the given args.
+     * Calls handleResult with the result
+     */
+    function runExcludeTest (args, handleResult, done) {
+      directInvoke(args, function (error, result) {
+        if (error) {
+          return done(error);
+        }
+        handleResult(result);
+        done();
+      }, path.join(__dirname, 'fixtures', 'options', 'exclude'));
+    }
+
+    it('should exclude specific files', function (done) {
+      runExcludeTest(['*.spec.js', '--exclude', 'fail.spec.js'], function (result) {
+        expect(result.output).to.contain('1 passing');
+        expect(result.code).to.equal(0);
+      }, done);
+    });
+
+    it('should exclude globbed files', function (done) {
+      runExcludeTest(['**/*.spec.js', '--exclude', '**/fail.spec.js'], function (result) {
+        expect(result.output).to.contain('2 passing');
+        expect(result.code).to.equal(0);
+      }, done);
+    });
+
+    it('should exclude multiple patterns', function (done) {
+      runExcludeTest(['**/*.spec.js', '--exclude', 'fail.spec.js,nested/fail.spec.js'], function (result) {
+        expect(result.output).to.contain('2 passing');
+        expect(result.code).to.equal(0);
+      }, done);
+    });
+  });
 });