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.

Sign up for GitHub

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

Jump to bottom

test: add e2e tests #357

Draft
wants to merge 14 commits into
base: taehoon/implement-challenger
Choose a base branch
from
Draft

test: add e2e tests #357

wants to merge 14 commits into from

Conversation

fakedev9999
Copy link
Member

@fakedev9999 fakedev9999 commented Jan 31, 2025 •
edited
Loading

Happy cases to cover

  • Honest proposer, unchallenged, proposer wins on timeout
  • Honest proposer, unchallenged, proposer defends with proof
  • Honest proposer, unchallenged, prover defends with proof
  • Malicious proposer, challenged, challenger wins

Unhappy cases to cover

  • Honest prover, challenger attempts to challenge

More details to consider

  • challenge on specific game with uuid that we desired to challenge retrieve game address from receipt's logs.
  • Cases that should be covered with Groth16Verifier: Malicious proposer, challenged, prover attempts to defend with invalid proof

@fakedev9999 fakedev9999 force-pushed the taehoon/implement-challenger branch from 17e8adc to a4fe4b8 Compare January 31, 2025 22:53
@fakedev9999 fakedev9999 force-pushed the taehoon/implement-challenger branch from a4fe4b8 to 9396bb0 Compare February 1, 2025 02:28
@fakedev9999 fakedev9999 force-pushed the taehoon/implement-challenger branch 2 times, most recently from 1cadc70 to e843e67 Compare February 4, 2025 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fakedev9999 @ratankaliani