Skip to content
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

spec/test: add test units for Certificate handling in the votekeeper #856

Open
3 tasks
cason opened this issue Feb 14, 2025 · 0 comments
Open
3 tasks

spec/test: add test units for Certificate handling in the votekeeper #856

cason opened this issue Feb 14, 2025 · 0 comments
Labels
mbt Model-Based Testing spec Related to specifications sync Synchronization protocols

Comments

@cason
Copy link
Contributor

cason commented Feb 14, 2025

The ability of overwritten recorded votes, in the case of equivocation (see scenarios in this discussion), has lead to the design decision of producing and exchanging Prevote certificates as part of VoteSync protocol (#576, see this comment in particular).

The handling of certificates in the votekeeper was added to the Quint spec (#364) but the associated votekeeper tests were not updated accordingly. Since these tests are used as input for MBT-generated code tests, the implementation is not aligned with the model.

Definition of Done

  • Add tests to the votekeeper unit that handle certificates
  • In particular, make sure that votes in a certificate overwrite existing equivocating votes from the same process
  • Add the produced tests to the MBT workflow, so that they are reproduced in the implementation
@cason cason added mbt Model-Based Testing spec Related to specifications sync Synchronization protocols labels Feb 14, 2025
@cason cason changed the title spec/test: add test unit where the votekeeper handle Certificates spec/test: add test units for Certificate handling in the votekeeper Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mbt Model-Based Testing spec Related to specifications sync Synchronization protocols
Projects
None yet
Development

No branches or pull requests

1 participant