--forbid-only now handle a empty suite marked only

Signed-off-by: Outsider <[email protected]>

lib/runner.js

@@ -12,6 +12,7 @@ var utils = require('./utils');
 var inherits = utils.inherits;
 var debug = require('debug')('mocha:runner');
 var Runnable = require('./runnable');
+var Test = require('./test');
 var stackFilter = utils.stackTraceFilter();
 var stringify = utils.stringify;
 var type = utils.type;
@@ -633,6 +634,16 @@ Runner.prototype.runSuite = function(suite, fn) {
 
   debug('run suite %s', suite.fullTitle());
 
+  if (!total && this.forbidOnly && hasOnly(suite)) {
+    suite._onlySuites.forEach(function(onlySuite) {
+      var emptyTest = new Test('', function() {});
+      emptyTest.parent = onlySuite;
+      emptyTest.ctx = onlySuite.ctx;
+      onlySuite.tests.push(emptyTest);
+      total++;
+    });
+  }
+
   if (!total || (self.failures && suite._bail)) {
     return fn();
   }

test/integration/fixtures/options/forbid-only/only-empty-suite.js

@@ -0,0 +1,3 @@
+'use strict';
+
+describe.only('forbid only - suite marked with only', function() {});

test/integration/options.spec.js

@@ -313,6 +313,17 @@ describe('options', function() {
         done();
       });
     });
+
+    it('fails if there is empty suite marked only', function(done) {
+      run('options/forbid-only/only-empty-suite.js', args, function(err, res) {
+        if (err) {
+          done(err);
+          return;
+        }
+        expect(res, 'to have failed with error', onlyErrorMessage);
+        done();
+      });
+    });
   });
 
   describe('--forbid-pending', function() {