Use correct matcher
What does this MR do and why?
Use correct matcher
In !97711 (merged), this was
changed incorrectly from toBeCalledWith
to toHaveBeenCalledTimes
.
The test still passed because state.namespaceId
happens to be 1
, and
the second argument is ignored.
Screenshots or screen recordings
n/a
How to set up and validate locally
- Tests pass?
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Mark Florian