add a non-sequencing node to localnet and test it#507
Merged
ClaytonNorthey92 merged 10 commits intomainfrom Jul 2, 2025
Merged
add a non-sequencing node to localnet and test it#507ClaytonNorthey92 merged 10 commits intomainfrom
ClaytonNorthey92 merged 10 commits intomainfrom
Conversation
ClaytonNorthey92
force-pushed
the
clayton/non-sequencing-localnet
branch
from
24bc0f8 to
79597fe
Compare
ClaytonNorthey92
force-pushed
the
clayton/non-sequencing-localnet
branch
from
92a9ac3 to
d9043b1
Compare
ClaytonNorthey92
commented
Jun 9, 2025
e2e/monitor/main_test.go
Outdated
| res.Body.Close() | ||
| res2.Body.Close() | ||
|
|
||
| t.Logf("will parse reponses\n%s\n%s", string(resBody), string(resBody2)) |
Contributor
Author
There was a problem hiding this comment.
this is a bit noisy, probably not necessary
ClaytonNorthey92
force-pushed
the
clayton/non-sequencing-localnet
branch
from
eb20024 to
9ca86ea
Compare
added test cases for the following: * assert that non-sequencing tip is close to the sequencer * assert that non-sequencing hvm btc tip is the same as the sequencer * walk the l2 chain and assert that the output root is the same all of the way down, ignore sync status * assert that safe and final blocks are progressing * repeat tests for both sequencer and non-sequencer (deploying contracts, sending txs, bridging, etc) fixes #508
ClaytonNorthey92
force-pushed
the
clayton/non-sequencing-localnet
branch
from
b15f4cb to
49b6e11
Compare
max-sanchez
approved these changes
Jul 2, 2025
marcopeereboom
pushed a commit
that referenced
this pull request
Jul 9, 2025
* add a non-sequencing node to localnet and test it added test cases for the following: * assert that non-sequencing tip is close to the sequencer * assert that non-sequencing hvm btc tip is the same as the sequencer * walk the l2 chain and assert that the output root is the same all of the way down, ignore sync status * assert that safe and final blocks are progressing * repeat tests for both sequencer and non-sequencer (deploying contracts, sending txs, bridging, etc) fixes #508 * removed commented out code * linting fixes * linting fixes * linting fixes * remove retries * added retries where optimism creates its own ctx timeout * removed versbose log line * better printing * ran linter
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Summary
added test cases for the following:
the way down, ignore sync status
contracts, sending txs, bridging, etc)
fixes #508
Changes
See summary.