Add test for suite marked only

mochajs · Aug 2, 2017 · 6cdfa05 · 6cdfa05
1 parent 245c9cc
commit 6cdfa05
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/test/integration/fixtures/options/forbid-only/only-suite.js b/test/integration/fixtures/options/forbid-only/only-suite.js
@@ -0,0 +1,5 @@
+'use strict';
+
+describe.only('forbid only - suite marked with only', function () {
+  it('test1', function () {});
+});
diff --git a/test/integration/options.spec.js b/test/integration/options.spec.js
@@ -182,6 +182,8 @@ describe('options', function () {
   });
 
   describe('--forbid-only', function () {
+    var onlyErrorMessage = '`.only` forbidden';
+
     before(function () {
       args = ['--forbid-only'];
     });
@@ -198,7 +200,16 @@ describe('options', function () {
       run('options/forbid-only/only.js', args, function (err, res) {
         assert(!err);
         assert.equal(res.code, 1);
-        assert.equal(res.failures[0].err.message, '`.only` forbidden');
+        assert.equal(res.failures[0].err.message, onlyErrorMessage);
+        done();
+      });
+    });
+
+    it('fails if there are tests in suites marked only', function (done) {
+      run('options/forbid-only/only-suite.js', args, function (err, res) {
+        assert(!err);
+        assert.equal(res.code, 1);
+        assert.equal(res.failures[0].err.message, onlyErrorMessage);
         done();
       });
     });