Merge pull request #1585 from get10101/fix/do-not-swallow-error-on-br…

…oadcasting-tx

fix: Do not swallow error when failing to broadcast transaction

crates/ln-dlc-node/src/ldk_node_wallet.rs

@@ -1,5 +1,6 @@
 use crate::fee_rate_estimator::EstimateFeeRate;
 use crate::node::Storage;
+use anyhow::anyhow;
 use anyhow::bail;
 use anyhow::Context;
 use anyhow::Error;
@@ -283,7 +284,7 @@ where
 
         self.blockchain
             .broadcast(tx)
-            .with_context(|| format!("Failed to broadcast transaction {txid}"))?;
+            .map_err(|e| anyhow!("Failed to broadcast transaction {txid}. {e:#}"))?;
 
         Ok(txid)
     }

crates/tests-e2e/tests/force_close_position.rs

@@ -8,6 +8,10 @@ use tests_e2e::wait_until;
 #[tokio::test]
 #[ignore = "need to be run with 'just e2e' command"]
 async fn can_force_close_position() {
+    // TODO(holzeis): Adapt the test to be able to force close the DLC channel as well. At the
+    // moment this test is only able to force close the ln channel, as the maturity time will never
+    // be reached to get the oracles attestation to complete the dlc channel closure.
+
     let test = setup::TestSetup::new_with_open_position().await;
     let coordinator = &test.coordinator;
     let bitcoin = &test.bitcoind;