Added support for it.skip()

Author: danielstjules

README.md

@@ -17,8 +17,8 @@ npm install --save mocha.parallel
 /**
  * Generates a suite for parallel execution of individual specs. While each
  * spec is ran in parallel, specs resolve in series, leading to deterministic
- * output. Compatible with both callbacks and promises. Supports hooks, but
- * not nested suites.
+ * output. Compatible with both callbacks and promises. Supports hooks, pending
+ * or skipped specs, but not nested suites.
  *
  * @example
  * parallel('setTimeout', function() {

fixtures/index.js

@@ -2,7 +2,7 @@ var path = require('path');
 
 var absolutePaths = {};
 var fixtures = ['delay', 'multiple', 'hooks', 'hooksSimple',
-  'uncaughtException'];
+  'uncaughtException', 'skip'];
 
 fixtures.forEach(function(fixture) {
   absolutePaths[fixture] = path.resolve(__dirname, fixture + '.js');

fixtures/skip.js

@@ -0,0 +1,15 @@
+var parallel = require('../index.js');
+
+parallel('suite', function() {
+  it('test1', function(done) {
+    setTimeout(done, 500);
+  });
+
+  it.skip('test2', function(done) {
+    setTimeout(done, 500);
+  });
+
+  it('test3', function(done) {
+    setTimeout(done, 500);
+  });
+});

index.js

@@ -5,8 +5,8 @@ var hookTypes = ['before', 'beforeEach', 'afterEach', 'after'];
 /**
  * Generates a suite for parallel execution of individual specs. While each
  * spec is ran in parallel, specs resolve in series, leading to deterministic
- * output. Compatible with both callbacks and promises. Supports hooks, but
- * not nested suites.
+ * output. Compatible with both callbacks and promises. Supports hooks, pending
+ * or skipped specs, but not nested suites.
  *
  * @example
  * parallel('setTimeout', function() {
@@ -75,6 +75,10 @@ module.exports = function parallel(name, fn) {
     });
 
     specs.forEach(function(spec) {
+      if (spec.skip) {
+        return it.skip(spec.name);
+      }
+
       it(spec.name, function() {
         if (spec.err) throw spec.err;
         return spec.promise.then(function() {
@@ -102,6 +106,17 @@ function patchIt(specs) {
     specs.push({
       name: name,
       getPromise: createWrapper(fn),
+      skip: null,
+      error: null,
+      promise: null
+    });
+  };
+
+  it.skip = function skip(name, fn) {
+    specs.push({
+      name: name,
+      getPromise: createWrapper(fn),
+      skip: true,
       error: null,
       promise: null
     });

test.js

@@ -92,4 +92,17 @@ describe('parallel', function() {
       done();
     });
   });
+
+  it('is compatible with it.skip for pending specs', function(done) {
+    var cmd = './node_modules/.bin/mocha ' + fixtures.skip;
+    exec(cmd, function(err, stdout, stderr) {
+      if (err) return done(err);
+
+      assert(!stderr.length);
+      assert(stdout.indexOf('2 passing') !== -1);
+      assert(stdout.indexOf('1 pending') !== -1);
+
+      done();
+    });
+  });
 });