Skip to content

Commit 1a24061

Browse files
committed
chore: fix test compilation errors, removed useless comments
1 parent 938e6d2 commit 1a24061

File tree

3 files changed

+5
-8
lines changed

3 files changed

+5
-8
lines changed

execution/evm/test/test_helpers.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,6 @@ import (
2020
"go.uber.org/zap/zaptest"
2121
)
2222

23-
// (Docker client/network are now provided by callers)
24-
2523
// RethNodeOpt allows tests to customize the reth node builder before building.
2624
type RethNodeOpt func(b *reth.NodeBuilder)
2725

test/e2e/evm_force_inclusion_e2e_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,7 @@ func setupSequencerWithForceInclusion(t *testing.T, sut *SystemUnderTest, nodeHo
7979

8080
// Use common setup (no full node needed initially)
8181
dcli, netID := tastoradocker.Setup(t)
82-
env := setupCommonEVMEnv(t, sut, dcli, netID)
83-
// Use env fields inline below to reduce local vars
82+
env := setupCommonEVMEnv(t, sut, dcli, netID)
8483

8584
// Create passphrase file
8685
passphraseFile := createPassphraseFile(t, nodeHome)
@@ -195,8 +194,8 @@ func TestEvmFullNodeForceInclusionE2E(t *testing.T) {
195194
// --- Start Sequencer Setup ---
196195
// We manually setup sequencer here because we need the force inclusion flag,
197196
// and we need to capture variables for full node setup.
198-
dcli2, netID2 := tastoradocker.Setup(t)
199-
env := setupCommonEVMEnv(t, sut, dcli2, netID2, WithFullNode())
197+
dockerClient, networkID := tastoradocker.Setup(t)
198+
env := setupCommonEVMEnv(t, sut, dockerClient, networkID, WithFullNode())
200199

201200
passphraseFile := createPassphraseFile(t, sequencerHome)
202201
jwtSecretFile := createJWTSecretFile(t, sequencerHome, env.SequencerJWT)
@@ -288,7 +287,8 @@ func setupMaliciousSequencer(t *testing.T, sut *SystemUnderTest, nodeHome string
288287
t.Helper()
289288

290289
// Use common setup with full node support
291-
env := setupCommonEVMEnv(t, sut, WithFullNode())
290+
dockerClient, networkID := tastoradocker.Setup(t)
291+
env := setupCommonEVMEnv(t, sut, dockerClient, networkID, WithFullNode())
292292
// Use env fields inline below to reduce local vars
293293

294294
passphraseFile := createPassphraseFile(t, nodeHome)

test/e2e/evm_full_node_e2e_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1041,7 +1041,6 @@ func testSequencerFullNodeRestart(t *testing.T, initialLazyMode, restartLazyMode
10411041
t.Logf("Phase 1: Setting up sequencer (initial_lazy=%t) and full node with P2P connections...", initialLazyMode)
10421042
t.Logf("Test mode: initial_lazy=%t, restart_lazy=%t", initialLazyMode, restartLazyMode)
10431043

1044-
// Get Docker client/network and common environment
10451044
dcli, netID := tastoradocker.Setup(t)
10461045
env := setupCommonEVMEnv(t, sut, dcli, netID, WithFullNode())
10471046

0 commit comments

Comments
 (0)