Skip to content

Commit 1cc0afa

Browse files
fix: Optimize E2E EVM tests (#2421)
<!-- Please read and fill out this form before submitting your PR. Please make sure you have reviewed our contributors guide before submitting your first PR. NOTE: PR titles should follow semantic commits: https://www.conventionalcommits.org/en/v1.0.0/ --> ## Overview Optimizes E2E EVM tests time by 39%. 8.5 minutes -> 5.2 minutes. <!-- Please provide an explanation of the PR, including the appropriate context, background, goal, and rationale. If there is an issue with this information, please provide a tl;dr and link the issue. Ex: Closes #<issue number> --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Reduced timeouts and polling intervals across end-to-end tests to accelerate test execution. * Updated environment variables and healthcheck settings for related services to use shorter intervals and new logging configurations. * **Tests** * Removed specific restart-related and double-spend nonce handling tests from E2E test suites. * Streamlined test setup and readiness checks for improved efficiency. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 6c142f4 commit 1cc0afa

File tree

5 files changed

+45
-594
lines changed

5 files changed

+45
-594
lines changed

execution/evm/docker/docker-compose-full-node.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@ services:
88
- ./jwttoken:/jwt
99
healthcheck:
1010
test: ["CMD", "test", "-f", "/jwt/jwt.hex"]
11-
interval: 5s
12-
timeout: 5s
13-
retries: 3
11+
interval: 2s
12+
timeout: 2s
13+
retries: 2
1414
command: >
1515
/bin/sh -c "mkdir -p /jwt &&
1616
if [ ! -f /jwt/jwt.hex ]; then
@@ -35,6 +35,8 @@ services:
3535
- ./chain:/root/chain:ro
3636
- ./jwttoken:/root/jwt:ro
3737
- logs-full-node:/root/logs
38+
environment:
39+
- RUST_LOG=warn,lumen_rollkit=info
3840
entrypoint: /bin/sh -c
3941
command:
4042
- |

execution/evm/docker/docker-compose.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@ services:
88
- ./jwttoken:/jwt
99
healthcheck:
1010
test: ["CMD", "test", "-f", "/jwt/jwt.hex"]
11-
interval: 5s
12-
timeout: 5s
13-
retries: 3
11+
interval: 2s
12+
timeout: 2s
13+
retries: 2
1414
command: >
1515
/bin/sh -c "mkdir -p /jwt &&
1616
if [ ! -f /jwt/jwt.hex ]; then
@@ -35,9 +35,8 @@ services:
3535
- ./chain:/root/chain:ro
3636
- ./jwttoken:/root/jwt:ro
3737
- logs:/root/logs
38-
# environment:
39-
# - RUST_LOG=info,lumen=debug,lumen_node=debug,lumen_rollkit=debug
40-
# - RUST_BACKTRACE=1
38+
environment:
39+
- RUST_LOG=warn,lumen_rollkit=info
4140
entrypoint: /bin/sh -c
4241
command:
4342
- |

test/e2e/evm_full_node_e2e_test.go

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,6 @@
1919
// 4. TestEvmSequencerFullNodeRestartE2E - Distributed restart and recovery testing
2020
// - StandardRestart: Normal start -> Normal restart
2121
// - LazyModeRestart: Normal start -> Lazy restart
22-
// - LazyToStandardRestart: Lazy start -> Normal restart
23-
// - LazyToLazyRestart: Lazy start -> Lazy restart
2422
package e2e
2523

2624
import (
@@ -902,8 +900,6 @@ func restartSequencerAndFullNode(t *testing.T, sut *SystemUnderTest, sequencerHo
902900
// Sub-tests:
903901
// 1. StandardRestart: Normal start -> Normal restart
904902
// 2. LazyModeRestart: Normal start -> Lazy restart
905-
// 3. LazyToStandardRestart: Lazy start -> Normal restart
906-
// 4. LazyToLazyRestart: Lazy start -> Lazy restart
907903
//
908904
// Test Flow:
909905
// 1. Sets up Local DA layer, sequencer, and full node with P2P connections (in specified initial mode)
@@ -948,14 +944,6 @@ func TestEvmSequencerFullNodeRestartE2E(t *testing.T) {
948944
t.Run("LazyModeRestart", func(t *testing.T) {
949945
testSequencerFullNodeRestart(t, false, true) // normal -> lazy
950946
})
951-
952-
t.Run("LazyToStandardRestart", func(t *testing.T) {
953-
testSequencerFullNodeRestart(t, true, false) // lazy -> normal
954-
})
955-
956-
t.Run("LazyToLazyRestart", func(t *testing.T) {
957-
testSequencerFullNodeRestart(t, true, true) // lazy -> lazy
958-
})
959947
}
960948

961949
// testSequencerFullNodeRestart contains the shared test logic for all restart test combinations.

0 commit comments

Comments
 (0)