-
Notifications
You must be signed in to change notification settings - Fork 112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature request: resolvedWith #256
Comments
Same here. I was surprised by the lack of "resolvedWith" and "fulfilledWith". But I think the problem is in how we’re should compare a Promise result with the expectation - eq vs eql
|
seems like a sensible idea to have if either of you want to contribute that, i'd be happy to review otherwise I can try find time to do it |
There may be a great reason to not include this, but it feels a bit odd (as I'm starting to write some async test cases) to provide a
rejectedWith
but not afulfilledWith
.I understand that I can use
.to.eventually.be.fulfilled.and.equal
, but I'm wondering -- are:.to.eventually.be.rejected.and.equal()
, and.to.eventually.be.rejectedWith()
not essentially the same?
If so, I'd think that the
fulfilledWith
"shortcut" should also be included. If they're not the same, then we can close this request due to my ignorance 😄The text was updated successfully, but these errors were encountered: