Skip to content

Commit

Permalink
fix(tests): add missing expect.fail on some tests expecting to fail
Browse files Browse the repository at this point in the history
For them to fail cleanly if parsing did not fail
instead of timing out.
  • Loading branch information
mleguen committed Apr 15, 2020
1 parent 07daeaf commit 9a81ec6
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/validation.js
Expand Up @@ -320,6 +320,7 @@ describe('validation tests', () => {
return done()
})
.parse()
expect.fail('no parsing failure')
})

it('fails in strict mode with invalid command', (done) => {
Expand All @@ -333,6 +334,7 @@ describe('validation tests', () => {
return done()
})
.parse()
expect.fail('no parsing failure')
})

it('fails in strict mode with extra positionals', (done) => {
Expand Down

0 comments on commit 9a81ec6

Please sign in to comment.