Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updates
sinon
to 2.1.0 and makes a corresponding update tosinon-chai
to 2.9.0. I'm doing this because I want to be able to usesinon.stub.resolves()
andsinon.stub.rejects()
for testing some promise-heavy code.I only found one deprecation that affected us (and it was only printing a warning, but I cleaned it up anyway):
stub(obj, 'methodName', func)
is deprecated in favor of more idiomaticstub(obj, 'methodName').callsFake(func)
. It turns out most of the places we were doing this didn't really need a fake function anyway, they were returningundefined
(which can just bestub(obj, 'methodName')
) or a fixed value (stub(obj, 'methodName').returns(value)
).