spesmilo / electrum
62%
master: 62%

LAST BUILD BRANCH: 202512_openalias
DEFAULT BRANCH: master
Repo Added
Files 88
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH fix_channel_removal_assert
branch: fix_channel_removal_assert
CHANGE BRANCH
x

coverage: 62.356% (+0.02%) from 62.34%
5209744434003968

Pull #10346

CirrusCI

f321x 
lnchannel: allow deleting unfunded incoming channels

We tried to delete incoming channels that didn't get funded after
lnutil.CHANNEL_OPENING_TIMEOUT, however an assert prevented this:

```
  3.63 | E | lnwatcher.LNWatcher.[default_wallet-LNW] | Exception in check_onchain_situation: AssertionError()
Traceback (most recent call last):
  File "/home/user/code/electrum-fork/electrum/util.py", line 1233, in wrapper
    return await func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/code/electrum-fork/electrum/lnwatcher.py", line 117, in check_onchain_situation
    await self.update_channel_state(
    ...<5 lines>...
        keep_watching=keep_watching)
  File "/home/user/code/electrum-fork/electrum/lnwatcher.py", line 135, in update_channel_state
    chan.update_onchain_state(
    ~~~~~~~~~~~~~~~~~~~~~~~~~^
        funding_txid=funding_txid,
        ^^^^^^^^^^^^^^^^^^^^^^^^^^
    ...<2 lines>...
        closing_height=closing_height,
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        keep_watching=keep_watching)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/code/electrum-fork/electrum/lnchannel.py", line 341, in update_onchain_state
    self.update_unfunded_state()
    ~~~~~~~~~~~~~~~~~~~~~~~~~~^^
  File "/home/user/code/electrum-fork/electrum/lnchannel.py", line 382, in update_unfunded_state
    self.lnworker.remove_channel(self.channel_id)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^
  File "/home/user/code/electrum-fork/electrum/lnworker.py", line 3244, in remove_channel
    assert chan.can_be_deleted()
           ~~~~~~~~~~~~~~~~~~~^^
AssertionError
```
Pull Request #10346: lnchannel: allow deleting unfunded incoming channels

6 of 7 new or added lines in 1 file covered. (85.71%)

3 existing lines in 3 files now uncovered.

23649 of 37926 relevant lines covered (62.36%)

0.62 hits per line

Relevant lines Covered
37926 RELEVANT LINES 23649 COVERED LINES
0.62 HITS PER LINE
Source Files on fix_channel_removal_assert

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
52097444... fix_channel_removal_assert lnchannel: allow deleting unfunded incoming channels We tried to delete incoming channels that didn't get funded after lnutil.CHANNEL_OPENING_TIMEOUT, however an assert prevented this: ``` 3.63 | E | lnwatcher.LNWatcher.[default_wallet-LNW] | ... Pull #10346 f321x CirrusCI
62.36
See All Builds (12578)