Fix chain fixtures to use ChainHandler rather than always using ChainHandlerCached (#5243)

* Fix chain fixtures to use CHainHandler rather than always using ChainHandlerCached

* Remove unecessary ChainAppConfig.start()
This commit is contained in:
Chris Stewart 2023-09-25 11:13:59 -05:00 committed by GitHub
parent c4f0bd3d39
commit 12e5dfc021
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -17,7 +17,10 @@ import org.bitcoins.core.util.FutureUtil
import org.bitcoins.crypto.DoubleSha256DigestBE
import org.bitcoins.rpc.client.common.BitcoindRpcClient
import org.bitcoins.rpc.client.v19.V19BlockFilterRpc
import org.bitcoins.testkit.chain.ChainUnitTest.createChainHandler
import org.bitcoins.testkit.chain.ChainUnitTest.{
createChainHandler,
createChainHandlerCached
}
import org.bitcoins.testkit.chain.fixture._
import org.bitcoins.testkit.chain.models.{
ReorgFixtureBlockHeaderDAO,
@ -225,7 +228,7 @@ trait ChainUnitTest
def createChainHandlerCachedWithGenesisFilter(): Future[
ChainHandlerCached] = {
for {
chainHandler <- createChainHandler()
chainHandler <- createChainHandlerCached()
filterHeaderChainApi <- chainHandler.processFilterHeader(
ChainTestUtil.genesisFilterHeaderDb.filterHeader,
ChainTestUtil.genesisHeaderDb.hashBE)
@ -450,7 +453,12 @@ object ChainUnitTest extends ChainVerificationLogger {
def createChainHandler()(implicit
ec: ExecutionContext,
appConfig: ChainAppConfig): Future[ChainHandler] = {
createChainHandlerCached()
val handlerWithGenesisHeaderF =
ChainUnitTest.setupHeaderTableWithGenesisHeader()
for {
_ <- handlerWithGenesisHeaderF
c = ChainHandler.fromDatabase()
} yield c
}
def createChainHandlerCached()(implicit