Fix an issue and add relevant tests when describe and xdescribe fail …

…when not supplied with a callback (issue mochajs#1744).
antoval · Jul 22, 2015 · 9da51db · 9da51db
1 parent 6d20329
commit 9da51db
Show file tree

Hide file tree

Showing 5 changed files with 152 additions and 79 deletions.
diff --git a/lib/interfaces/bdd.js b/lib/interfaces/bdd.js
@@ -41,6 +41,9 @@ module.exports = function(suite) {
      */
 
     context.describe = context.context = function(title, fn) {
+      if (!fn) {
+        throw new TypeError('a callback is not supplied for the "' + title + '" suite');
+      }
       var suite = Suite.create(suites[0], title);
       suite.file = file;
       suites.unshift(suite);
@@ -57,7 +60,9 @@ module.exports = function(suite) {
       var suite = Suite.create(suites[0], title);
       suite.pending = true;
       suites.unshift(suite);
-      fn.call(suite);
+      if (fn) {
+        fn.call(suite);
+      }
       suites.shift();
     };