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

MSFDG: cannon e2e test #107

Open
wants to merge 25 commits into
base: develop
Choose a base branch
from
Open

MSFDG: cannon e2e test #107

wants to merge 25 commits into from

Conversation

dajuguan
Copy link

@dajuguan dajuguan commented Dec 5, 2024

Description

This PR add a cannon e2e test case to verify the MSFDG features.

Tests

Pass:

# change maxGameDepth to 20 to test stepV2 function
go test  -run ^TestOutputCannonGame$ github.com/ethereum-optimism/optimism/op-e2e2/faultproofs -count=1

Fail:

Other tests in op-e2e2/faultproofs/output_cannon_test.go are left to be done.

JustXxx and others added 25 commits November 2, 2024 20:15
@JustXxx
Merge remote-tracking branch 'origin/po/common-datatypes' into bill/o…
9541939 
…p-challenger2-attackv2
@dajuguan
feat(MSFDG): add getSubValues
ad007d7
@JustXxx
Merge remote-tracking branch 'origin/develop' into bill/op-challenger…
854429a 
…2-attackv2
@JustXxx
op-challenger2 add attackV2
6c1e5b9
@JustXxx
Merge remote-tracking branch 'origin/po/getsubclaims' into bill/op-ch…
5d65304 
…allenger2-attackv2
@JustXxx
op-challenger2 add stepV2
360b88b
@JustXxx
merge po/oracle_proof
ef4cc9e
@dajuguan
add getStepData tests for solver_test.go
42d3a98
@JustXxx
modify test
53c81ac
@JustXxx
merge
94d5923
@dajuguan
fix step maxAttackBranch test case
97e5c1c
@JustXxx
fix bug by rules
591fa5d
@JustXxx
Merge remote-tracking branch 'origin/bill/op-challenger2-attackv2' in…
1cab66d 
…to bill/op-challenger2-attackv2
@dajuguan
add getstepData testcase
0805fd4
@JustXxx
Merge remote-tracking branch 'origin/bill/op-challenger2-attackv2' in…
6b7839b 
…to bill/op-challenger2-attackv2
@JustXxx
modify unittest TestCalculateNextActions
f65fdc3
@JustXxx
Merge remote-tracking branch 'origin/develop' into bill/op-challenger…
c026566 
…2-attackv2
@dajuguan
fix: TestMultipleRoundsWithNbits1/2
32ca079
@dajuguan
fix tests: responder_test and TestGetStepDataWithOutputRoot test
0176d08
@dajuguan
fix: getTxByHash and filterLog call in devnet env
670ca68
@dajuguan
contract:
162ec26 
1. rename addLocalData to addLocalData2 to avoid conflicting in tx.Pack
2. fix subValues at splitDepth+nbits must be 1
@dajuguan
fix: VMStateRoot only has one subValue
2716b1e
@dajuguan
op-challenger2: cosmatic and pass all tests
772c2ac
@dajuguan
test: alphabet game e2etest for MSFDG
c1513a0
@dajuguan
test(op-e2e): add cannon MSDFG test
5ed755e
This was referenced Dec 5, 2024
Open
Closed
Closed
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
@dajuguan @JustXxx