-
Notifications
You must be signed in to change notification settings - Fork 162
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
SET_RETURN_SEQ should reset return_val_seq_idx #101
Comments
@meekrosoft I think that this makes sense (and the equivalent for custom fake sequences); would you be happy to consider a PR for this change? |
jackdigitalinsight
pushed a commit
to jackdigitalinsight/fff
that referenced
this issue
Jan 3, 2023
reset return sequence index when setting a return sequence.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
Within one test case, I want to modify a fake's return value sequence. SET_RETURN_SEQ updates the sequence and length, but doesn't reset the next index.
To Reproduce
example:
Expected behavior
second protocol_update() calls to hal_uart_rx(fake) should see 3, 4, 5, 6, -1. Instead it gets 6, -1.
Compiler, toolset, platform (please complete the following information):
Suggested change*
The text was updated successfully, but these errors were encountered: