refactor(test): update tests to use builder API

- update tests test_tx_graph and
test_tx_graph_conflicts to use builder API

Author: tvpeter

crates/chain/tests/test_indexed_tx_graph.rs

@@ -11,7 +11,11 @@ use bdk_chain::{
     SpkIterator,
 };
 use bdk_testenv::{
-    TestEnv, anyhow::{self}, bitcoind::{Input, Output}, block_id, hash, spk, utils::{DESCRIPTORS, new_tx}
+    anyhow::{self},
+    bitcoind::{Input, Output},
+    block_id, hash, spk,
+    utils::{new_tx, DESCRIPTORS},
+    TestEnv,
 };
 use bitcoin::{
     secp256k1::Secp256k1, Address, Amount, BlockHash, Network, OutPoint, ScriptBuf, Transaction,

crates/chain/tests/test_tx_graph.rs

@@ -1139,7 +1139,7 @@ fn test_chain_spends() {
         );
 
         // Chain position of the `tx_2` is now none, as it is older than `tx_2_conflict`
-        assert!(canonical_positions.get(&tx_2.compute_txid()).is_none());
+        assert!(!canonical_positions.contains_key(&tx_2.compute_txid()));
     }
 }
 
@@ -1278,30 +1278,20 @@ fn call_map_anchors_with_non_deterministic_anchor() {
     }
 
     let template = [
-        TxTemplate {
-            tx_name: "tx1",
-            inputs: &[TxInTemplate::Bogus],
-            outputs: &[TxOutTemplate::new(10000, Some(1))],
-            anchors: &[block_id!(1, "A")],
-            last_seen: None,
-            ..Default::default()
-        },
-        TxTemplate {
-            tx_name: "tx2",
-            inputs: &[TxInTemplate::PrevTx("tx1", 0)],
-            outputs: &[TxOutTemplate::new(20000, Some(2))],
-            anchors: &[block_id!(2, "B")],
-            ..Default::default()
-        },
-        TxTemplate {
-            tx_name: "tx3",
-            inputs: &[TxInTemplate::PrevTx("tx2", 0)],
-            outputs: &[TxOutTemplate::new(30000, Some(3))],
-            anchors: &[block_id!(3, "C"), block_id!(4, "D")],
-            ..Default::default()
-        },
+        TxTemplate::new("tx1")
+            .with_inputs(vec![TxInTemplate::Bogus])
+            .with_outputs(vec![TxOutTemplate::new(10_000, Some(1))])
+            .with_anchors(vec![block_id!(1, "A")]),
+        TxTemplate::new("tx2")
+            .with_inputs(vec![TxInTemplate::PrevTx("tx1".into(), 0)])
+            .with_outputs(vec![TxOutTemplate::new(20_000, Some(2))])
+            .with_anchors(vec![block_id!(2, "B")]),
+        TxTemplate::new("tx3")
+            .with_inputs(vec![TxInTemplate::PrevTx("tx2".into(), 0)])
+            .with_outputs(vec![TxOutTemplate::new(30_000, Some(3))])
+            .with_anchors(vec![block_id!(3, "C"), block_id!(4, "D")]),
     ];
-    let graph = init_graph(&template).tx_graph;
+    let graph = init_graph(template).tx_graph;
     let new_graph = graph.clone().map_anchors(|a| NonDeterministicAnchor {
         anchor_block: a,
         // A non-deterministic value