spesmilo / electrum / 5866898287493120

63%

master: 63%

Build Type

Pull #10372

CirrusCI

Committed by SomberNight

Commit Message

lnworker: split LNWallet and LNWorker: LNWallet "has an" LNWorker

- LNWallet no longer "is-an" LNWorker, instead LNWallet "has-an" LNWorker
- the motivation is to make the unit tests nicer, and allow writing unit tests for more things
  - I hope this makes it possible to e.g. test lnsweep in the unit tests
  - some stuff we would previously have to write a regtest for, maybe we can write a unit test for, now
- in unit tests, MockLNWallet now
  - inherits LNWallet
  - the Wallet is no longer being mocked

Pull Request Pull Request #10372: lnworker: split LNWallet and LNWorker: LNWallet "has an" LNWorker

Run Details

83 of 170 new or added lines in 6 files covered. (48.82%)

255 existing lines in 6 files now uncovered.

23801 of 38097 relevant lines covered (62.47%)

0.62 hits per line