Adds support for async tests, though not in ideal fashion due to moch…

…a lacking support for async skip (mochajs/mocha#1625)

it.optional.js

@@ -2,6 +2,7 @@
  * Expose `TestOptional`.
  */
 exports = module.exports = TestOptional;
+var tryasync = require('./tryasync');
 
 /**
  * Initialize a new `Test` with the given `title` and callback `fn`, if test 
@@ -14,23 +15,42 @@ exports = module.exports = TestOptional;
 function TestOptional(title, titlePending, fn) {
   if (!fn) {
     fn = titlePending;
-    titlePending = 'Pending: ' + title;
+    titlePending = 'PENDING: ' + title;
   }
-
-  function processResult(result) {
-    if (result) {
-      it(titlePending);
-    } else {
-      it(title, function() {});
+
+  it(title, function(done){
+    var self = this;
+
+    function processResult(result) {
+      if (result) {
+        self.skip();
+      }
+      done();
+    };
+
+    function executeTest(){
+      if (fn.length === 0) {
+        processResult(fn());
+      } else {
+        fn.call(self, processResult);
+      }
     }
-  };
-
-  try {
-    if (fn.length === 0) {
-      return processResult(fn());
+
+    function handleErrorAsync(e){
+      self.test.title = titlePending;
+      // async skip not supported yet: https://github.com/mochajs/mocha/issues/1625
+      // self.skip();
+      done();
     }
-    fn(processResult);
-  } catch (e) {
-    it(titlePending);
-  }
+
+    try {
+      tryasync(executeTest)
+      .catch(handleErrorAsync);
+    } catch(e){
+      // sync'ed / same tick exception
+      self.test.title = titlePending;
+      self.skip();
+    }
+
+  });
 };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "it-optional",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "description": "Mocha add-on method it.optional() which marks a test as pending if it would fail",
   "main": "it.optional.js",
   "directories": {

test/it.optional.test.js

@@ -13,14 +13,20 @@ describe('optional tests', function(){
       done(new Error('oops'));
     });
 
+    it.optional('should do mno eventually', function(done){
+      setTimeout(function(){
+        done(new Error('oops'));
+      }, 5);
+    });
+
     it.optional('should do ikj eventually', function(done){
       throw new Error('uncaught error');
     });
 
     it.optional('should do abc eventually', function(done){
       setTimeout(function(){
         assert.equal('a', 'b');
-      }, 100);
+      }, 5);
     });
 
   });

tryasync.js

@@ -0,0 +1,17 @@
+// From http://stackoverflow.com/questions/14301839/javascript-asynchronous-exception-handling-with-node-js
+
+var domain = require("domain");
+module.exports = function (func) {
+  var F = function () { };
+  var dom = domain.create();
+  F.prototype.catch = function (errHandle) {
+    var args = arguments;
+    dom.on("error", function (err) {
+      return errHandle(err, dom);
+    }).run(function () {
+      func.call(null, args);
+    });
+    return this;
+  };
+  return new F();
+};