-
Notifications
You must be signed in to change notification settings - Fork 34
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
Checkpoints must fail the test when they are flagged more number of times than initially specified #113
Comments
Flagging 4 times a strict checkpoint shall indeed fail. I'll try to reproduce locally. |
jponge
added a commit
that referenced
this issue
Mar 3, 2022
See #114 |
jponge
added a commit
that referenced
this issue
Mar 4, 2022
See #113 Correctness improvement: failing after having completed is necessary after all.
jponge
added a commit
that referenced
this issue
Mar 4, 2022
See #113 Correctness improvement: failing after having completed is necessary after all. Signed-off-by: Julien Ponge <[email protected]>
jponge
added a commit
that referenced
this issue
Mar 5, 2022
See #113 Correctness improvement: failing after having completed is necessary after all. Signed-off-by: Julien Ponge <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was expecting this test to fail but it passes:
Similar to Mockito's behavior, is it feasible for checkpoints to fail if they see more flags then expected?
This is especially relevant since
checkpoint
is the strict version unlike thelaxCheckpoint
The text was updated successfully, but these errors were encountered: