fix: init g_txindex only once, downgrade from assert to exception

`g_txindex` should be initialized in `TestChainSetup`'s constructor but
in bitcoin#19806 (dash#5236), when portions of the constructor were
split into `mineBlocks()`, `g_txindex`'s init was left behind in the
latter instead of the former. This meant that every `mineBlocks()` call
would re-create a `TxIndex` instance, which is not intended behaviour.

Also, a runtime exception is more appropriate and closer to the usage of
`BOOST_REQUIRE` in other index `Start()` calls than the harsher `assert`

Author: kwvg

src/test/util/setup_common.cpp

@@ -349,6 +349,14 @@ TestChainSetup::TestChainSetup(int num_blocks, const std::vector<const char*>& e
             hash != checkpoints.mapCheckpoints.end() &&
             m_node.chainman->ActiveChain().Tip()->GetBlockHash() == hash->second);
     }
+
+    g_txindex = std::make_unique<TxIndex>(1 << 20, true);
+    if (!g_txindex->Start(m_node.chainman->ActiveChainstate())) {
+        throw std::runtime_error("TxIndex::Start() failed.");
+    }
+
+    // Allow tx index to catch up with the block index.
+    IndexWaitSynced(*g_txindex);
 }
 
 void TestChainSetup::mineBlocks(int num_blocks)
@@ -361,10 +369,6 @@ void TestChainSetup::mineBlocks(int num_blocks)
         m_coinbase_txns.push_back(b.vtx[0]);
     }
 
-    g_txindex = std::make_unique<TxIndex>(1 << 20, true);
-    assert(g_txindex->Start(m_node.chainman->ActiveChainstate()));
-
-    // Allow tx index to catch up with the block index.
     IndexWaitSynced(*g_txindex);
 }