From: Matt Corallo <649246+TheBlueMatt@users.noreply.github.com> Date: Thu, 29 Sep 2022 22:02:07 +0000 (+0000) Subject: Merge pull request #1106 from TheBlueMatt/2021-10-no-perm-err-broadcast X-Git-Tag: v0.0.112~24 X-Git-Url: http://git.bitcoin.ninja/?a=commitdiff_plain;h=7544030bb63fee6484fc178bb2ac8f382fe3b5b1;p=rust-lightning Merge pull request #1106 from TheBlueMatt/2021-10-no-perm-err-broadcast Do not broadcast commitment txn on Permanent mon update failure --- 7544030bb63fee6484fc178bb2ac8f382fe3b5b1 diff --cc lightning-invoice/src/payment.rs index ee85fab0b,e8e296294..cadb595e7 --- a/lightning-invoice/src/payment.rs +++ b/lightning-invoice/src/payment.rs @@@ -1695,9 -1731,9 +1695,9 @@@ mod tests invoice_payer.pay_invoice(&invoice_to_pay).unwrap(); let inflight_map = invoice_payer.create_inflight_map(); - // Only the second path, which failed with `MonitorUpdateFailed` should be added to our + // Only the second path, which failed with `MonitorUpdateInProgress` should be added to our // inflight map because retries are disabled. - assert_eq!(inflight_map.len(), 2); + assert_eq!(inflight_map.0.len(), 2); } #[test]