runner.js: this.skip() in beforeEach hooks

mochajs · Mar 7, 2019 · 24d3383 · 24d3383
1 parent e654253
commit 24d3383
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/lib/runner.js b/lib/runner.js
@@ -388,8 +388,12 @@ Runner.prototype.hook = function(name, fn) {
       }
       if (err) {
         if (err instanceof Pending) {
-          if (name === HOOK_TYPE_BEFORE_EACH || name === HOOK_TYPE_AFTER_EACH) {
+          if (name === HOOK_TYPE_AFTER_EACH) {
             self.test.pending = true;
+          } else if (name === HOOK_TYPE_BEFORE_EACH) {
+            self.test.pending = true;
+            self.emit(constants.EVENT_HOOK_END, hook);
+            return fn(err);
           } else {
             suite.tests.forEach(function(test) {
               test.pending = true;
@@ -636,7 +640,12 @@ Runner.prototype.runTests = function(suite, fn) {
           self.emit(constants.EVENT_TEST_PENDING, test);
         }
         self.emit(constants.EVENT_TEST_END, test);
-        return next();
+        if (err instanceof Pending) {
+          self.suite = errSuite;
+          return self.hookUp(HOOK_TYPE_AFTER_EACH, next);
+        } else {
+          return next();
+        }
       }
       if (err) {
         return hookErr(err, errSuite, false);