expect: Improve report when mock-spy matcher fails, part 1 #8640
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.
Summary
Improve Matcher error reports for all 20 mock-spy matchers
Indentation changed in
spyMatchers.ts
because offunction
to accessthis: MatcherState
but the actual changes are limited to:expectedArgument
andoptions
ensureNoExpected
callsensureExpectedIsNumber
callsensureMock
callsensureMock
functionThe
matcherName.slice(1)
is temporary to omit.
in matcher error snapshots onlyFuture pull requests in this series will remove
.
from matcher names increate
function callsBreaking Residue for Jest 25
ensureMock
fromensureMockOrSpy
because Jasmine spy does not store return values, true?ensureExpectedIsNumber
withensureExpectedIsNonNegativeInteger
ensureNthIsPositiveInteger
function which verifies safe integer and does not callparseInt
Test plan
Updated 76 snapshots
lastCalledWith
lastReturnedWith
nthCalledWith
nthReturnedWith
toBeCalled
toBeCalledTimes
toBeCalledWith
toHaveBeenCalled
toHaveBeenCalledTimes
toHaveBeenCalledWith
toHaveBeenLastCalledWith
toHaveBeenNthCalledWith
toHaveLastReturnedWith
toHaveNthReturnedWith
toHaveReturned
toHaveReturnedTimes
toHaveReturnedWith
toReturn
toReturnTimes
toReturnWith