mirror of
https://github.com/lightningnetwork/lnd.git
synced 2025-02-23 06:35:07 +01:00
c602ac07e7
This commit moves the `HeightHintCache` implementation to the `channeldb` package and inverts the dependency relation between `chainntnfs` and `channeldb`. Many packages depend on channeldb for type definitions, interfaces, etc. `chainntnfs` is an example of that. `chainntnfs` defines the `SpendHintCache` and `ConfirmHintCache` interfaces but it also implments them (`HeightHintCache` struct). The implementation uses logic that should not leak from channeldb (ex: bucket paths). This makes our code highly coupled + it would not allow us to use any of these interfaces in a package that is imported by `channeldb` (circular dependency).
212 lines
7 KiB
Go
212 lines
7 KiB
Go
//go:build dev
|
|
// +build dev
|
|
|
|
package btcdnotify
|
|
|
|
import (
|
|
"bytes"
|
|
"testing"
|
|
|
|
"github.com/btcsuite/btcd/chaincfg/chainhash"
|
|
"github.com/btcsuite/btcd/integration/rpctest"
|
|
"github.com/lightningnetwork/lnd/blockcache"
|
|
"github.com/lightningnetwork/lnd/chainntnfs"
|
|
"github.com/lightningnetwork/lnd/channeldb"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
var (
|
|
testScript = []byte{
|
|
// OP_HASH160
|
|
0xA9,
|
|
// OP_DATA_20
|
|
0x14,
|
|
// <20-byte hash>
|
|
0xec, 0x6f, 0x7a, 0x5a, 0xa8, 0xf2, 0xb1, 0x0c, 0xa5, 0x15,
|
|
0x04, 0x52, 0x3a, 0x60, 0xd4, 0x03, 0x06, 0xf6, 0x96, 0xcd,
|
|
// OP_EQUAL
|
|
0x87,
|
|
}
|
|
)
|
|
|
|
func initHintCache(t *testing.T) *channeldb.HeightHintCache {
|
|
t.Helper()
|
|
|
|
db, err := channeldb.Open(t.TempDir())
|
|
require.NoError(t, err, "unable to create db")
|
|
t.Cleanup(func() {
|
|
require.NoError(t, db.Close())
|
|
})
|
|
|
|
testCfg := channeldb.CacheConfig{
|
|
QueryDisable: false,
|
|
}
|
|
hintCache, err := channeldb.NewHeightHintCache(testCfg, db.Backend)
|
|
require.NoError(t, err, "unable to create hint cache")
|
|
|
|
return hintCache
|
|
}
|
|
|
|
// setUpNotifier is a helper function to start a new notifier backed by a btcd
|
|
// driver.
|
|
func setUpNotifier(t *testing.T, h *rpctest.Harness) *BtcdNotifier {
|
|
hintCache := initHintCache(t)
|
|
blockCache := blockcache.NewBlockCache(10000)
|
|
|
|
rpcCfg := h.RPCConfig()
|
|
notifier, err := New(
|
|
&rpcCfg, chainntnfs.NetParams, hintCache, hintCache, blockCache,
|
|
)
|
|
require.NoError(t, err, "unable to create notifier")
|
|
if err := notifier.Start(); err != nil {
|
|
t.Fatalf("unable to start notifier: %v", err)
|
|
}
|
|
t.Cleanup(func() {
|
|
require.NoError(t, notifier.Stop())
|
|
})
|
|
|
|
return notifier
|
|
}
|
|
|
|
// TestHistoricalConfDetailsTxIndex ensures that we correctly retrieve
|
|
// historical confirmation details using the backend node's txindex.
|
|
func TestHistoricalConfDetailsTxIndex(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
harness := chainntnfs.NewMiner(
|
|
t, []string{"--txindex"}, true, 25,
|
|
)
|
|
|
|
notifier := setUpNotifier(t, harness)
|
|
|
|
// A transaction unknown to the node should not be found within the
|
|
// txindex even if it is enabled, so we should not proceed with any
|
|
// fallback methods.
|
|
var unknownHash chainhash.Hash
|
|
copy(unknownHash[:], bytes.Repeat([]byte{0x10}, 32))
|
|
unknownConfReq, err := chainntnfs.NewConfRequest(&unknownHash, testScript)
|
|
require.NoError(t, err, "unable to create conf request")
|
|
_, txStatus, err := notifier.historicalConfDetails(unknownConfReq, 0, 0)
|
|
require.NoError(t, err, "unable to retrieve historical conf details")
|
|
|
|
switch txStatus {
|
|
case chainntnfs.TxNotFoundIndex:
|
|
case chainntnfs.TxNotFoundManually:
|
|
t.Fatal("should not have proceeded with fallback method, but did")
|
|
default:
|
|
t.Fatal("should not have found non-existent transaction, but did")
|
|
}
|
|
|
|
// Now, we'll create a test transaction and attempt to retrieve its
|
|
// confirmation details.
|
|
txid, pkScript, err := chainntnfs.GetTestTxidAndScript(harness)
|
|
require.NoError(t, err, "unable to create tx")
|
|
if err := chainntnfs.WaitForMempoolTx(harness, txid); err != nil {
|
|
t.Fatalf("unable to find tx in the mempool: %v", err)
|
|
}
|
|
confReq, err := chainntnfs.NewConfRequest(txid, pkScript)
|
|
require.NoError(t, err, "unable to create conf request")
|
|
|
|
// The transaction should be found in the mempool at this point.
|
|
_, txStatus, err = notifier.historicalConfDetails(confReq, 0, 0)
|
|
require.NoError(t, err, "unable to retrieve historical conf details")
|
|
|
|
// Since it has yet to be included in a block, it should have been found
|
|
// within the mempool.
|
|
switch txStatus {
|
|
case chainntnfs.TxFoundMempool:
|
|
default:
|
|
t.Fatalf("should have found the transaction within the "+
|
|
"mempool, but did not: %v", txStatus)
|
|
}
|
|
|
|
// We'll now confirm this transaction and re-attempt to retrieve its
|
|
// confirmation details.
|
|
if _, err := harness.Client.Generate(1); err != nil {
|
|
t.Fatalf("unable to generate block: %v", err)
|
|
}
|
|
|
|
_, txStatus, err = notifier.historicalConfDetails(confReq, 0, 0)
|
|
require.NoError(t, err, "unable to retrieve historical conf details")
|
|
|
|
// Since the backend node's txindex is enabled and the transaction has
|
|
// confirmed, we should be able to retrieve it using the txindex.
|
|
switch txStatus {
|
|
case chainntnfs.TxFoundIndex:
|
|
default:
|
|
t.Fatal("should have found the transaction within the " +
|
|
"txindex, but did not")
|
|
}
|
|
}
|
|
|
|
// TestHistoricalConfDetailsNoTxIndex ensures that we correctly retrieve
|
|
// historical confirmation details using the set of fallback methods when the
|
|
// backend node's txindex is disabled.
|
|
func TestHistoricalConfDetailsNoTxIndex(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
harness := chainntnfs.NewMiner(t, nil, true, 25)
|
|
|
|
notifier := setUpNotifier(t, harness)
|
|
|
|
// Since the node has its txindex disabled, we fall back to scanning the
|
|
// chain manually. A transaction unknown to the network should not be
|
|
// found.
|
|
var unknownHash chainhash.Hash
|
|
copy(unknownHash[:], bytes.Repeat([]byte{0x10}, 32))
|
|
unknownConfReq, err := chainntnfs.NewConfRequest(&unknownHash, testScript)
|
|
require.NoError(t, err, "unable to create conf request")
|
|
_, txStatus, err := notifier.historicalConfDetails(unknownConfReq, 0, 0)
|
|
require.NoError(t, err, "unable to retrieve historical conf details")
|
|
|
|
switch txStatus {
|
|
case chainntnfs.TxNotFoundManually:
|
|
case chainntnfs.TxNotFoundIndex:
|
|
t.Fatal("should have proceeded with fallback method, but did not")
|
|
default:
|
|
t.Fatal("should not have found non-existent transaction, but did")
|
|
}
|
|
|
|
// Now, we'll create a test transaction and attempt to retrieve its
|
|
// confirmation details. We'll note its broadcast height to use as the
|
|
// height hint when manually scanning the chain.
|
|
_, currentHeight, err := harness.Client.GetBestBlock()
|
|
require.NoError(t, err, "unable to retrieve current height")
|
|
|
|
txid, pkScript, err := chainntnfs.GetTestTxidAndScript(harness)
|
|
require.NoError(t, err, "unable to create tx")
|
|
if err := chainntnfs.WaitForMempoolTx(harness, txid); err != nil {
|
|
t.Fatalf("unable to find tx in the mempool: %v", err)
|
|
}
|
|
confReq, err := chainntnfs.NewConfRequest(txid, pkScript)
|
|
require.NoError(t, err, "unable to create conf request")
|
|
|
|
_, txStatus, err = notifier.historicalConfDetails(confReq, 0, 0)
|
|
require.NoError(t, err, "unable to retrieve historical conf details")
|
|
|
|
// Since it has yet to be included in a block, it should have been found
|
|
// within the mempool.
|
|
if txStatus != chainntnfs.TxFoundMempool {
|
|
t.Fatal("should have found the transaction within the " +
|
|
"mempool, but did not")
|
|
}
|
|
|
|
// We'll now confirm this transaction and re-attempt to retrieve its
|
|
// confirmation details.
|
|
if _, err := harness.Client.Generate(1); err != nil {
|
|
t.Fatalf("unable to generate block: %v", err)
|
|
}
|
|
|
|
_, txStatus, err = notifier.historicalConfDetails(
|
|
confReq, uint32(currentHeight), uint32(currentHeight)+1,
|
|
)
|
|
require.NoError(t, err, "unable to retrieve historical conf details")
|
|
|
|
// Since the backend node's txindex is disabled and the transaction has
|
|
// confirmed, we should be able to find it by falling back to scanning
|
|
// the chain manually.
|
|
if txStatus != chainntnfs.TxFoundManually {
|
|
t.Fatal("should have found the transaction by manually " +
|
|
"scanning the chain, but did not")
|
|
}
|
|
}
|