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

Add storage byzantine E2E test cases #3414

Closed
ptrus opened this issue Oct 15, 2020 · 0 comments · Fixed by #3442
Closed

Add storage byzantine E2E test cases #3414

ptrus opened this issue Oct 15, 2020 · 0 comments · Fixed by #3442
Assignees
Labels
c:testing Category: testing

Comments

@ptrus
Copy link
Member

ptrus commented Oct 15, 2020

Add storage byzantine tests to test the failure-indicating commitments added in #3391

Scenarios:

  • honestly respond to txnscheduler requests so that round starts, then reject -> should trigger discrepancy and round failure with ErrMajorityFailure
  • honestly respond to txnscheduler and other executor worker request, but fail on runtime requests (todo: not sure if there's a nice way to do this, maybe there's some methods that only runtime uses?) -> should trigger discrepancy and round failure with ErrMajorityFailure
  • also add executor byzantine test that submits failure indicating commitments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:testing Category: testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant