spesmilo / electrum / 5197209021448192

63%

push

CirrusCI

lnpeer and lnworker cleanup:
 - rename trampoline_forwardings -> final_onion_forwardings,
   because this dict is used for both trampoline and hold invoices
 - remove timeout from hold_invoice_callbacks (redundant with invoice)
 - add test_failure boolean parameter to TestPeer._test_simple_payment,
   in order to test correct propagation of OnionRoutingFailures.
 - maybe_fulfill_htlc: raise an OnionRoutingFailure if we do not have
   the preimage for a payment that does not have a hold invoice callback.
   Without this, the above unit tests stall when we use test_failure=True

17 of 17 new or added lines in 3 files covered. (100.0%)

18493 of 31349 relevant lines covered (58.99%)

2.94 hits per line