Skip to content

test: add test cases for handling l1 events and l1 reorg #428

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
Merged
yiweichi merged 93 commits into from
Feb 6, 2026
Merged

test: add test cases for handling l1 events and l1 reorg #428

yiweichi merged 93 commits into from
Feb 6, 2026

Conversation

@yiweichi
Copy link
Member

@yiweichi yiweichi commented Nov 18, 2025 •
edited by frisitano
Loading

This PR adds test cases for handling l1 events and l1 reorg.

  • added test cases to test the behavior of RN when handling BatchCommit, BatchFinalized and BatchRevertRange events in different sync mode.
  • added test cases to test the behavior of RN when handling L1 block reorg with BatchCommit, BatchFinalized and BatchRevertRange events.

Fixes #420, #141

greged93 and others added 21 commits November 3, 2025 15:49
@greged93
tests: add test fixture for easier abstraction
d80987f
@greged93
Merge branch 'main' into tests/improve-framework
0e79430
@greged93
fix: lints
395add0
@greged93
fix: smol rework
3c6f416
@greged93
fix: fmt
e7da229
@greged93
tests: extend helpers and migrate
81c5db8
@greged93
tests: answer comments + finish migration
2d48822
@greged93
tests: migrate remaining tests
fe784c1
@greged93
Merge branch 'main' into tests/improve-framework
066609d
@greged93
tests: migration after merge
8c2bfc7
@greged93
tests: cleaning + fix
a64812f
@greged93
tests: disable eth wire bridge
02bbc3e
@greged93
tests: use expect_tx
537b6a9
@greged93
fix: clippy
a94f09b
@greged93
Merge branch 'main' into tests/improve-framework
1a95fd3
@greged93
chore: answer comments
7d8ab02
@yiweichi
Merge branch 'main' into tests/improve-framework
c3a51d6
@yiweichi
merge main branch
361a66e
@yiweichi
test: add anvil to testFixture
b50e8c0
@yiweichi
test: add anvil to test fixture
c820cbb
@yiweichi
Merge branch 'main' into morty-add-anvil-to-test-fixture
3d0c580
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 18, 2025 •
edited
Loading

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing morty-add-anvil-to-test-fixture (7b3d881) with main (76f9c0c)

Summary

✅ 2 untouched benchmarks

yiweichi and others added 9 commits January 7, 2026 18:50
@yiweichi
test
f994717
@yiweichi
test
8a0f806
@yiweichi
test
dffda1e
@yiweichi
test
372323d
@yiweichi
test
a538e83
@yiweichi
test
005a130
@yiweichi
test
5fba429
@yiweichi
test
01509c1
@frisitano
feat: shutdown refactor (#472)
f396937 
* shutdown refactor

* clean up

* use unused ports of anvil and run tests concurrently
frisitano
frisitano requested changes Jan 12, 2026
View reviewed changes
Copy link
Collaborator

@frisitano frisitano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Added some comments inline.

@yiweichi yiweichi requested a review from frisitano February 5, 2026 17:48
frisitano
frisitano previously approved these changes Feb 6, 2026
View reviewed changes
Copy link
Collaborator

@frisitano frisitano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks goood. left a few small comments inline.

@yiweichi yiweichi requested a review from frisitano February 6, 2026 17:45
frisitano
frisitano approved these changes Feb 6, 2026
View reviewed changes
Copy link
Collaborator

@frisitano frisitano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work, lgtm!

@yiweichi yiweichi merged commit 41ca6c3 into main Feb 6, 2026
17 of 18 checks passed
@yiweichi yiweichi deleted the morty-add-anvil-to-test-fixture branch February 6, 2026 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@frisitano frisitano frisitano approved these changes

+1 more reviewer

@jonastheis jonastheis jonastheis left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Testing] Test multi-mode L1

4 participants

@yiweichi @jonastheis @frisitano @greged93