Explicit try_preserving_privacy err handleing

payjoin · Oct 7, 2024 · 6803c8b · 6803c8b
1 parent 5866dc7
commit 6803c8b
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 5 deletions.
diff --git a/payjoin-cli/src/app/v1.rs b/payjoin-cli/src/app/v1.rs
@@ -388,7 +388,9 @@ fn try_contributing_inputs(
         .map(|i| (i.amount, OutPoint { txid: i.txid, vout: i.vout }))
         .collect();
 
-    let selected_outpoint = payjoin.try_preserving_privacy(candidate_inputs).expect("gg");
+    let selected_outpoint = payjoin
+        .try_preserving_privacy(candidate_inputs)
+        .map_err(|e| anyhow!("Failed to make privacy preserving selection: {}", e))?;
     let selected_utxo = available_inputs
         .iter()
         .find(|i| i.txid == selected_outpoint.txid && i.vout == selected_outpoint.vout)

diff --git a/payjoin-cli/src/app/v2.rs b/payjoin-cli/src/app/v2.rs
@@ -367,7 +367,9 @@ fn try_contributing_inputs(
         .map(|i| (i.amount, OutPoint { txid: i.txid, vout: i.vout }))
         .collect();
 
-    let selected_outpoint = payjoin.try_preserving_privacy(candidate_inputs).expect("gg");
+    let selected_outpoint = payjoin
+        .try_preserving_privacy(candidate_inputs)
+        .map_err(|e| anyhow!("Failed to make privacy preserving selection: {}", e))?;
     let selected_utxo = available_inputs
         .iter()
         .find(|i| i.txid == selected_outpoint.txid && i.vout == selected_outpoint.vout)

diff --git a/payjoin/tests/integration.rs b/payjoin/tests/integration.rs
@@ -632,7 +632,9 @@ mod integration {
                 .map(|i| (i.amount, OutPoint { txid: i.txid, vout: i.vout }))
                 .collect();
 
-            let selected_outpoint = payjoin.try_preserving_privacy(candidate_inputs).expect("gg");
+            let selected_outpoint = payjoin
+                .try_preserving_privacy(candidate_inputs)
+                .expect("Failed to make privacy preserving selection");
             let selected_utxo = available_inputs
                 .iter()
                 .find(|i| i.txid == selected_outpoint.txid && i.vout == selected_outpoint.vout)
@@ -1091,8 +1093,9 @@ mod integration {
                     .map(|i| (i.amount, OutPoint { txid: i.txid, vout: i.vout }))
                     .collect();
 
-                let selected_outpoint =
-                    payjoin.try_preserving_privacy(candidate_inputs).expect("gg");
+                let selected_outpoint = payjoin
+                    .try_preserving_privacy(candidate_inputs)
+                    .expect("Failed to make privacy preserving selection");
                 let selected_utxo = available_inputs
                     .iter()
                     .find(|i| i.txid == selected_outpoint.txid && i.vout == selected_outpoint.vout)