referee
referee copied to clipboard
Feature: expand verify to ensure no async assertions are pending
Extracted form the discussion of #47.
https://github.com/sinonjs/referee/issues/47#issuecomment-436646220
* If #45 lands, then expand `verify` to ensure all promises from `resolves` and `rejects` are resolved or rejected
The following example will pass, because we don't let the test runner handle the returned promise.
describe("some test", function() {
it("should resolve", function() {
referee.assert.resolves(Promise.resolve(42), 43);
}):
});
This issue is an approach to verify if all async assertions have been properly resolved.
@mroderick, @mantoni
Go for it!
@mgred, if you are not looking into this, I can give it a try.
@mgred, if you are not looking into this, I can give it a try.
I don't think they'll be too upset if your work overlaps ... and after six months, I think it's fair to take it up 👍
@mroderick , Thanks. I will look into this.