-
Notifications
You must be signed in to change notification settings - Fork 7
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.
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
Open
yiweichi
wants to merge
80
commits into
main
Choose a base branch
from
morty-add-anvil-to-test-fixture
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
80 commits
Select commit
Hold shift + click to select a range
d80987f
tests: add test fixture for easier abstraction
greged93 0e79430
Merge branch 'main' into tests/improve-framework
greged93 395add0
fix: lints
greged93 3c6f416
fix: smol rework
greged93 e7da229
fix: fmt
greged93 81c5db8
tests: extend helpers and migrate
greged93 2d48822
tests: answer comments + finish migration
greged93 fe784c1
tests: migrate remaining tests
greged93 066609d
Merge branch 'main' into tests/improve-framework
greged93 8c2bfc7
tests: migration after merge
greged93 a64812f
tests: cleaning + fix
greged93 02bbc3e
tests: disable eth wire bridge
greged93 537b6a9
tests: use expect_tx
greged93 a94f09b
fix: clippy
greged93 1a95fd3
Merge branch 'main' into tests/improve-framework
greged93 7d8ab02
chore: answer comments
greged93 c3a51d6
Merge branch 'main' into tests/improve-framework
yiweichi 361a66e
merge main branch
yiweichi b50e8c0
test: add anvil to testFixture
yiweichi c820cbb
test: add anvil to test fixture
yiweichi 3d0c580
Merge branch 'main' into morty-add-anvil-to-test-fixture
yiweichi 4df57f1
feat: add test l1 reorg
yiweichi 46638aa
feat: add test l1 reorg
yiweichi c703985
Merge branch 'main' into morty-add-anvil-to-test-fixture
yiweichi 82d4178
test: add test_l1_sync_batch_commit
yiweichi 14ccd64
test: add test_l1_sync_batch_finalized
yiweichi b38b802
test: add l1 reorg tests
yiweichi 8765085
test: add l1 reorg tests
yiweichi 0b8dd48
fix: chainspec dev l1 config
yiweichi d560c36
Merge branch 'main' into morty-add-anvil-to-test-fixture
yiweichi 371bc70
fix: ci
yiweichi 7672169
improve anvil config
yiweichi a66cae7
improve comments
yiweichi 1eb8f41
fix: l1 sync test && address some comments
yiweichi 11344f8
fix: l1 reorg test
yiweichi c50f330
address comment
yiweichi 29dee41
feat: use AnvilApi
yiweichi 6e2db8a
fix ci
yiweichi c7fa7e4
Merge branch 'main' into morty-add-anvil-to-test-fixture
yiweichi e7f00c3
fix: read test tx in one
yiweichi 731ae47
add test-utils
yiweichi 5a03a6b
fix ci
yiweichi 10b540c
fix: address comment
yiweichi 926f00e
test: add test_l1_sync_batch_revert_before_l1_synced
yiweichi a1aac4f
Merge branch 'main' into morty-add-anvil-to-test-fixture
yiweichi 18c2a4c
fix: ci
yiweichi 61ee962
add database to test fixture
yiweichi 618bd53
fix: test_l1_sync_batch_finalized
yiweichi 8c9f40f
update comment
yiweichi 087ff57
fix: patch svm-rs and svm-rs-builder
yiweichi aad3527
Merge branch 'main' into morty-add-anvil-to-test-fixture
yiweichi 1c168fa
clean up anvil_state.json (this is not a fix)
yiweichi 23c0d99
clean up anvil_state.json (this is not a fix)
yiweichi b503a8e
feat: remove svm patch
yiweichi 6cbd20d
put finalized_block fix behind #[cfg(feature = test-utils)]
yiweichi 94736e7
fmt
yiweichi 42a991c
fix: use database handle from chain-orchastrator handle
yiweichi fd91faf
Merge branch 'main' into morty-add-anvil-to-test-fixture
yiweichi 27c3434
feat: add reboot tests (#467)
yiweichi cbd6096
merge mian
yiweichi 46e2d5d
merge mian
yiweichi 3db8192
fix: tests
yiweichi bc57c6c
fix: ci
yiweichi 8a0db0b
fix: ci
yiweichi 12181a9
Merge branch 'main' into morty-add-anvil-to-test-fixture
yiweichi dcd83c8
increase DEFAULT_EVENT_WAIT_TIMEOUT for ci
yiweichi a803877
make DEFAULT_EVENT_WAIT_TIMEOUT configurable
yiweichi 1ec255e
ci: log failure event waiting
yiweichi a4a433f
test
yiweichi c2ba869
test
yiweichi 866663d
test
yiweichi f994717
test
yiweichi 8a0f806
test
yiweichi dffda1e
test
yiweichi 372323d
test
yiweichi a538e83
test
yiweichi 005a130
test
yiweichi 5fba429
test
yiweichi 01509c1
test
yiweichi f396937
feat: shutdown refactor (#472)
frisitano File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -51,9 +51,32 @@ jobs: | |
| - name: Run integration tests | ||
| run: | | ||
| cargo nextest run --all-features --workspace --locked \ | ||
| --no-tests=pass -E 'kind(test) and not test(docker)' \ | ||
| --no-tests=pass -E 'kind(test) and not test(docker) and not binary(l1_sync)' \ | ||
| -- --skip test_should_consolidate_to_block_15k | ||
|
|
||
| integration-l1-sync: | ||
| runs-on: | ||
| group: scroll-reth-runner-group | ||
| timeout-minutes: 60 | ||
| env: | ||
| RUST_BACKTRACE: 1 | ||
| steps: | ||
| - uses: actions/checkout@v6 | ||
| - uses: rui314/setup-mold@v1 | ||
| - uses: dtolnay/rust-toolchain@nightly | ||
| with: | ||
| toolchain: 'nightly' | ||
| - uses: Swatinem/rust-cache@v2 | ||
| with: | ||
| cache-on-failure: true | ||
| - uses: taiki-e/install-action@nextest | ||
| - name: Run L1 sync integration tests (serial) | ||
| run: | | ||
| cargo nextest run --all-features --workspace --locked \ | ||
| --no-tests=pass -E 'binary(l1_sync)' --test-threads=1 \ | ||
| --failure-output immediate \ | ||
| --success-output never | ||
|
|
||
|
Comment on lines
+57
to
+79
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lets remove this. |
||
| integration-docker-compose: | ||
| runs-on: | ||
| group: scroll-reth-runner-group | ||
|
|
||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's change the anvil
portto use 0 and then run all tests at the same time.