Bug fix: Shouldn't roll back the genesis block (#89)

Author: daithihearn

build.gradle.kts

@@ -20,7 +20,7 @@ jacoco {
 
 group = "org.vechain"
 
-val projectVersion = System.getenv("PROJECT_VERSION") ?: "6.3.0"
+val projectVersion = System.getenv("PROJECT_VERSION") ?: "6.3.1"
 version = projectVersion
 
 val isSnapshot = version.toString().endsWith("SNAPSHOT")

src/main/kotlin/org.vechain.indexer/BlockIndexer.kt

@@ -88,10 +88,14 @@ open class BlockIndexer(
     /**
      * Rolls back to a safe state to ensure data integrity.
      *
+     * Only rolls back if the block number is greater than zero.
+     *
      * @param blockNumber The block number to roll back to.
      */
     protected open fun rollbackToSafeState(blockNumber: Long) {
-        rollback(blockNumber)
+        if (blockNumber > 0) {
+            rollback(blockNumber)
+        }
     }
 
     /**

src/test/kotlin/org/vechain/indexer/BlockIndexerTest.kt

@@ -205,10 +205,8 @@ internal class BlockIndexerTest {
 
             indexer.initialise()
 
-            verify(exactly = 2) {
-                processor.getLastSyncedBlock()
-            } // Verify the rollback is performed once
-            verify(exactly = 1) { processor.rollback(0L) }
+            verify(exactly = 2) { processor.getLastSyncedBlock() }
+            verify(exactly = 0) { processor.rollback(any()) }
 
             expect {
                 // Verify the status is INITIALISED
@@ -221,6 +219,29 @@ internal class BlockIndexerTest {
             }
         }
 
+        @Test
+        fun `Should rollback to startBlock when startBlock is greater than zero and no last synced block`() =
+            runBlocking {
+                indexer =
+                    BlockIndexer(
+                        name = "TestBlockIndexer",
+                        thorClient = thorClient,
+                        processor = processor,
+                        startBlock = 50L,
+                        eventProcessor = null,
+                        pruner = null,
+                        prunerInterval = 10000L,
+                        syncLoggerInterval = 1L,
+                        inspectionClauses = null,
+                        dependsOn = null,
+                    )
+                every { processor.getLastSyncedBlock() } returns null
+
+                indexer.initialise()
+
+                verify(exactly = 1) { processor.rollback(50L) }
+            }
+
         @Test
         fun `Previous block should be null if last synced block doesn't match current block - 1`() {
             every { processor.getLastSyncedBlock() } returns