-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Add OP E2E mineblock test with isthmus activated at genesis #15447
Copy link
Copy link
Closed
Labels
A-op-rethRelated to Optimism and op-rethRelated to Optimism and op-rethC-testA change that impacts how or what we testA change that impacts how or what we testD-good-first-issueNice and easy! A great choice to get startedNice and easy! A great choice to get started
Description
Describe the feature
we have this for OP:
reth/crates/optimism/node/tests/e2e/testsuite.rs
Lines 13 to 21 in 0c9dd9a
| #[tokio::test] | |
| async fn test_testsuite_op_assert_mine_block() -> Result<()> { | |
| reth_tracing::init_test_tracing(); | |
| let setup = Setup::default() | |
| .with_chain_spec(Arc::new( | |
| OpChainSpecBuilder::default() | |
| .chain(OP_MAINNET.chain) | |
| .genesis(serde_json::from_str(include_str!("../assets/genesis.json")).unwrap()) |
but this suses a chainspec that doesn't enable isthmus at genesis
TODO
- duplicate this test but modify the chainspec so that isthmus is also activated at genesis
we can use this as base:
reth/crates/optimism/node/tests/e2e/testsuite.rs
Lines 19 to 21 in 0c9dd9a
| OpChainSpecBuilder::default() | |
| .chain(OP_MAINNET.chain) | |
| .genesis(serde_json::from_str(include_str!("../assets/genesis.json")).unwrap()) |
but enable isthmus at genesis
or duplicate the genesis.json and enable isthmus there
Additional context
No response
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-op-rethRelated to Optimism and op-rethRelated to Optimism and op-rethC-testA change that impacts how or what we testA change that impacts how or what we testD-good-first-issueNice and easy! A great choice to get startedNice and easy! A great choice to get started
Type
Projects
Status
Done