Eslint rule for checking that should-promised assertions return
Install
$ npm i -D eslint-plugin-should-promised
Configure
{
"plugins":[
"should-promised"
],
"rules":{
"should-promised/return-promise":2
}
}
Rules
Require promise assertions to return or await (return-promise)
This rule is intended to be used with the should assertion library.
When testing an async function by returning a promise to mocha it is important to remember to actually return the promise. Forgetting to return will cause the test case to pass even if the promise is eventually rejected.
This rule will point out when a should assertion is made without returning.
Rule Details
This rule looks for any of the properties Promise, fulfilled, fulfilledWith, rejected, rejectedWith, finally and eventually.
The following patterns are considered warnings:
describe('forgetting to return the promise',()=>{
it('should report when not returning the promise from should.be.fulfilled',()=>{
promiseFn().should.be.fulfilled();
});
it('should report when not returning the promise from should.eventually',()=>{
promiseFn().should.eventually.eql(1);
});
});
These patterns would not be considered warnings:
describe('returning the promise',()=>{
it('should not report when returning the promise from should.be.fulfilled',()=>{
returnpromiseFn().should.be.fulfilled();
});
it('should allow implicit return in a single expression arrow function',()=>
promiseFn().should.be.fulfilled());
it('should not report when using async/await',async()=>{
