-
Notifications
You must be signed in to change notification settings - Fork 5
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
'to yield exchange satisfying' not documented #17
Comments
Damn, there isn't even a documentation site for unexpected-express. It's probably the plugin that I use most often myself, maybe except for unexpected-sinon. Skomagerens børn går i hullede sko... |
http://unexpected.js.org/unexpected-express/ was what I had in mind... |
Right, that's there, but there's no structured docs for the individual assertions (yet). You're right, we should change the examples to use |
From the code, I'm guessing |
@joelmukuthu Correct! Have you found yourself needing that? |
I'm introducing unexpected-express at my job and I'd like to make it possible to have exhaustive satisfactions for the new-comers (possibly to avoid omissions). I actually don't know how it would work, having to describe everything in the req/res but I just wanted to know if it's supported. Don't need it per se |
The exhaustiveness would only apply to the |
Indeed, correct! |
Well, it would be a useful addition, and probably not very hard. PR welcome :) |
Cool, will find some time to try it out :) |
unexpected-express/lib/unexpectedExpress.js
Line 47 in 33d9d53
Comment says Please prefer this one because it does use 'to satisfy' semantics, so I guess it is important :)
The text was updated successfully, but these errors were encountered: