spesmilo / electrum / 4749326195359744

63%

Build Type

push

CirrusCI

Committed by SomberNight

Commit Message

lnworker.suggest_splits: (fix) don't force splitting

lnworker.suggest_splits for non-trampoline case tries to split amts over 5000 sat
but mpp_split.suggest_splits does not return splits where any part is smaller than 10000 sat.
So in practice, without trampoline, invoices between 5k and ~20k sats could not be paid.
This suggests that the logic should not be scattered in multiple places but merged into mpp_split.py...
This commit just does a quick fix though, to try again without splitting if there was no solution.

Run Details

4 of 6 new or added lines in 1 file covered. (66.67%)

16 existing lines in 3 files now uncovered.

19027 of 32082 relevant lines covered (59.31%)

2.96 hits per line