Commit adcdf910 authored by EvanJRichard's avatar EvanJRichard

More minor test tweaks: change require statement's message, get head's time...

More minor test tweaks: change require statement's message, get head's time differently, add a couple breadcrumb comments.
parent 48068627
...@@ -7,7 +7,7 @@ import ( ...@@ -7,7 +7,7 @@ import (
"github.com/stretchr/testify/require" "github.com/stretchr/testify/require"
"github.com/ethereum-optimism/optimism/op-e2e/e2eutils" "github.com/ethereum-optimism/optimism/op-e2e/e2eutils"
"github.com/ethereum-optimism/optimism/op-node/testlog" "github.com/ethereum-optimism/optimism/op-service/testlog"
) )
func TestDencunL1Fork(gt *testing.T) { func TestDencunL1Fork(gt *testing.T) {
...@@ -21,7 +21,7 @@ func TestDencunL1Fork(gt *testing.T) { ...@@ -21,7 +21,7 @@ func TestDencunL1Fork(gt *testing.T) {
_, _, miner, sequencer, _, verifier, _, batcher := setupReorgTestActors(t, dp, sd, log) _, _, miner, sequencer, _, verifier, _, batcher := setupReorgTestActors(t, dp, sd, log)
require.False(t, sd.L1Cfg.Config.IsCancun(miner.l1Chain.CurrentBlock().Time()), "not active yet") require.False(t, sd.L1Cfg.Config.IsCancun(miner.l1Chain.CurrentBlock().Time), "dencun not active yet") // QUESTION: what other arg should be supplied?
// start op-nodes // start op-nodes
sequencer.ActL2PipelineFull(t) sequencer.ActL2PipelineFull(t)
...@@ -34,7 +34,9 @@ func TestDencunL1Fork(gt *testing.T) { ...@@ -34,7 +34,9 @@ func TestDencunL1Fork(gt *testing.T) {
// verify Cancun is active // verify Cancun is active
l1Head := miner.l1Chain.CurrentBlock() l1Head := miner.l1Chain.CurrentBlock()
require.True(t, sd.L1Cfg.Config.IsCancun(l1Head.Time())) require.True(t, sd.L1Cfg.Config.IsCancun(l1Head.Time), "dencun active")
//BEFORE MERGE OF PR #7993: Also, add a few blob txs in as dummy data TODO
// build L2 chain up to and including L2 blocks referencing Cancun L1 blocks // build L2 chain up to and including L2 blocks referencing Cancun L1 blocks
sequencer.ActL1HeadSignal(t) sequencer.ActL1HeadSignal(t)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment