Closed
Description
The following code skips the test in mocha v2, but causes an error in v3
Test
describe('broken skip behaviour', function() {
it('should not report done() called multiple times', function(done) {
this.skip()
done()
})
})
v2 Output
broken skip behaviour
- should not report done() called multiple times
0 passing (11ms)
1 pending
v3 Output
broken skip behaviour
- should not report done() called multiple times
1) should not report done() called multiple times
0 passing (15ms)
1 pending
1 failing
1) broken skip behaviour should not report done() called multiple times:
Error: done() called multiple times
at Suite.<anonymous> (test/index.test.js:3:5)
at Object.<anonymous> (test/index.test.js:1:63)
at require (internal/module.js:20:19)
at Array.forEach (native)
at node.js:463:3
Unfortunately I rely on this behaviour for skipping tests in Yadda.
I've tried with various version of mocha from 1-3 and it looks like I can simply remove the call to done() in all versions. Can you confirm?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment