From 118ce7f4567d9ef235c99bc321aa63721b11d5c4 Mon Sep 17 00:00:00 2001
From: EvanJRichard <evan@oplabs.co>
Date: Thu, 2 Nov 2023 09:13:39 -0400
Subject: [PATCH] Fix usage of IsCancun.
---
op-e2e/actions/dencun_fork_test.go | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/op-e2e/actions/dencun_fork_test.go b/op-e2e/actions/dencun_fork_test.go
index 8d776f37f..9c7931f1d 100644
--- a/op-e2e/actions/dencun_fork_test.go
+++ b/op-e2e/actions/dencun_fork_test.go
@@ -21,7 +21,8 @@ func TestDencunL1Fork(gt *testing.T) {
_, _, miner, sequencer, _, verifier, _, batcher := setupReorgTestActors(t, dp, sd, log)
- require.False(t, sd.L1Cfg.Config.IsCancun(miner.l1Chain.CurrentBlock().Time), "dencun not active yet") // QUESTION: what other arg should be supplied?
+ l1Head := miner.l1Chain.CurrentBlock()
+ require.False(t, sd.L1Cfg.Config.IsCancun(l1Head.Number, l1Head.Time), "dencun not active yet") // QUESTION: what other arg should be supplied?
// start op-nodes
sequencer.ActL2PipelineFull(t)
@@ -33,8 +34,8 @@ func TestDencunL1Fork(gt *testing.T) {
miner.ActEmptyBlock(t)
// verify Cancun is active
- l1Head := miner.l1Chain.CurrentBlock()
- require.True(t, sd.L1Cfg.Config.IsCancun(l1Head.Time), "dencun active")
+ l1Head = miner.l1Chain.CurrentBlock()
+ require.True(t, sd.L1Cfg.Config.IsCancun(l1Head.Number, l1Head.Time), "dencun active")
//BEFORE MERGE OF PR #7993: Also, add a few blob txs in as dummy data TODO
--
2.23.0