spesmilo / electrum / 5615904193511424

63%

push

CirrusCI

Refactor trampoline forwarding and hold invoices.

 - maybe_fulfill_htlc returns a forwarding callback that
   covers both cases.
 - previously, the callback of hold invoices was called as a
   side-effect of lnworker.check_mpp_status.
 - the same data structures (lnworker.trampoline_forwardings,
   lnworker.trampoline_forwarding_errors) are used for both
   trampoline forwardings and hold invoices.
 - maybe_fulfill_htlc still recursively calls itself to perform
   checks on trampoline onion. This is ugly, but ugliness is now
   contained to that method.

47 of 47 new or added lines in 2 files covered. (100.0%)

18394 of 31127 relevant lines covered (59.09%)

3.54 hits per line