]> Untitled Git - bdk/commitdiff
test(wallet): small cleanups to test_wallet_transactions_relevant
authorSteve Myers <steve@notmandatory.org>
Fri, 20 Dec 2024 20:40:55 +0000 (14:40 -0600)
committerSteve Myers <steve@notmandatory.org>
Fri, 20 Dec 2024 20:41:02 +0000 (14:41 -0600)
Also small use cleanup in wallet/mod.

crates/wallet/src/wallet/mod.rs
crates/wallet/tests/wallet.rs

index d64f4b8e2d2a856ef345c5195432828ec8f837dc..81c2f8282366aaee032868e0fe1ac20fc6f5ebc9 100644 (file)
@@ -19,7 +19,6 @@ use alloc::{
     sync::Arc,
     vec::Vec,
 };
-use chain::Indexer;
 use core::{cmp::Ordering, fmt, mem, ops::Deref};
 
 use bdk_chain::{
@@ -32,7 +31,7 @@ use bdk_chain::{
     },
     tx_graph::{CalculateFeeError, CanonicalTx, TxGraph, TxUpdate},
     BlockId, ChainPosition, ConfirmationBlockTime, DescriptorExt, FullTxOut, Indexed,
-    IndexedTxGraph, Merge,
+    IndexedTxGraph, Indexer, Merge,
 };
 use bitcoin::{
     absolute,
index 7ca60022e27160948dd86a0c0c2f642eda02ff31..dcc8030b5a6da11e6da7d5bd335dc9429eb59f4d 100644 (file)
@@ -4,7 +4,7 @@ use std::sync::Arc;
 
 use anyhow::Context;
 use assert_matches::assert_matches;
-use bdk_chain::{BlockId, ChainPosition, ConfirmationBlockTime, TxUpdate};
+use bdk_chain::{BlockId, ChainPosition, ConfirmationBlockTime};
 use bdk_wallet::coin_selection::{self, LargestFirstCoinSelection};
 use bdk_wallet::descriptor::{calc_checksum, DescriptorError, IntoWalletDescriptor};
 use bdk_wallet::error::CreateTxError;
@@ -4254,22 +4254,13 @@ fn test_wallet_transactions_relevant() {
     // add not relevant transaction to test wallet
     let (other_external_desc, other_internal_desc) = get_test_tr_single_sig_xprv_and_change_desc();
     let (other_wallet, other_txid) = get_funded_wallet(other_internal_desc, other_external_desc);
-    let other_tx_node = other_wallet.get_tx(other_txid).unwrap().tx_node;
-    let other_tx_confirmationblocktime = other_tx_node.anchors.iter().last().unwrap();
-    let other_tx_update = TxUpdate {
-        txs: vec![other_tx_node.tx],
-        txouts: Default::default(),
-        anchors: [(*other_tx_confirmationblocktime, other_txid)].into(),
-        seen_ats: [(other_txid, other_tx_confirmationblocktime.confirmation_time)].into(),
-    };
     let test_wallet_update = Update {
-        last_active_indices: Default::default(),
-        tx_update: other_tx_update,
-        chain: None,
+        tx_update: other_wallet.tx_graph().clone().into(),
+        ..Default::default()
     };
     test_wallet.apply_update(test_wallet_update).unwrap();
 
-    // verify transaction from other wallet was added but is not it relevant transactions list.
+    // verify transaction from other wallet was added but is not in relevant transactions list.
     let relevant_tx_count_after = test_wallet.transactions().count();
     let full_tx_count_after = test_wallet.tx_graph().full_txs().count();
     let canonical_tx_count_after = test_wallet