From da6022ac473bca38de20f1dbdbbb6f29c3386494 Mon Sep 17 00:00:00 2001
From: Diego Ximenes <dxmendes1@gmail.com>
Date: Fri, 3 Jan 2025 11:19:28 -0300
Subject: [PATCH] Fixes snap sync test procedure when creating arb node

---
 arbnode/node.go | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/arbnode/node.go b/arbnode/node.go
index adc5c81a2f..22d0142269 100644
--- a/arbnode/node.go
+++ b/arbnode/node.go
@@ -596,6 +596,8 @@ func getDAS(
 }
 
 func getInboxTrackerAndReader(
+	ctx context.Context,
+	exec execution.ExecutionSequencer,
 	arbDb ethdb.Database,
 	txStreamer *TransactionStreamer,
 	dapReaders []daprovider.Reader,
@@ -613,10 +615,14 @@ func getInboxTrackerAndReader(
 	}
 	firstMessageBlock := new(big.Int).SetUint64(deployInfo.DeployedAt)
 	if config.SnapSyncTest.Enabled {
+		if exec == nil {
+			return nil, nil, errors.New("snap sync test requires an execution sequencer")
+		}
+
 		batchCount := config.SnapSyncTest.BatchCount
 		delayedMessageNumber, err := exec.NextDelayedMessageNumber()
 		if err != nil {
-			return nil, err
+			return nil, nil, err
 		}
 		if batchCount > delayedMessageNumber {
 			batchCount = delayedMessageNumber
@@ -629,7 +635,7 @@ func getInboxTrackerAndReader(
 		}
 		block, err := FindBlockContainingBatchCount(ctx, deployInfo.Bridge, l1client, config.SnapSyncTest.ParentChainAssertionBlock, batchCount)
 		if err != nil {
-			return nil, err
+			return nil, nil, err
 		}
 		firstMessageBlock.SetUint64(block)
 	}
@@ -1036,7 +1042,7 @@ func createNodeImpl(
 		return nil, err
 	}
 
-	inboxTracker, inboxReader, err := getInboxTrackerAndReader(arbDb, txStreamer, dapReaders, config, configFetcher, l1client, l1Reader, deployInfo, delayedBridge, sequencerInbox)
+	inboxTracker, inboxReader, err := getInboxTrackerAndReader(ctx, executionSequencer, arbDb, txStreamer, dapReaders, config, configFetcher, l1client, l1Reader, deployInfo, delayedBridge, sequencerInbox)
 	if err != nil {
 		return nil, err
 	}