When there is beforeEach hook and test fails in async callback, it aborts other tests and error messages report wrong blocks

[petejkim]

describe('mocha tests', function() {
  beforeEach(function() {
    // there is beforeEach hook
  });

  it('should go to the next test when it fails (outside callback)', function(done){
    throw new Error('the other test should still run');
    done();
  });

  it('should go to the next test when it fails (inside callback)', function(done) {
    setTimeout(function() {
      throw new Error('the other test should still run');
      done();
    }, 0);
  });

  it('should get to this test', function(done) {
    throw new Error('YES! This test still runs!');
    done();
  });
});

Expected output:

...

  ✖ 3 of 3 tests failed:

  0) mocha tests should go to the next test when it fails (outside callback):
     Error: the other test should still run
      at ...

  1) mocha tests should go to the next test when it fails (inside callback):
     Error: the other test should still run
      at ...

  2) mocha tests should get to this test:
     Error: YES! This test still runs!
      at ...

Actual output:

..

  ✖ 2 of 3 tests failed:

  0) mocha tests should go to the next test when it fails (outside callback):
     Error: the other test should still run
      at ...

  1) mocha tests "before each" hook:
     Error: the other test should still run
      at ...

Notice that the test is aborted and the error block reporting for the second error is also wrong.

Related Issue: Issue 94

[tj]

interesting thanks for the report ill look into it in a min

[petejkim]

awesome, thanks!!!