• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

spesmilo / electrum / 4775448984420352
61%
master: 63%

Build:
Build:
LAST BUILD BRANCH: qml_manual_fee_feerate_edit
DEFAULT BRANCH: master
Ran 26 Sep 2025 04:01PM UTC
Jobs 1
Files 88
Run time 1min
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

26 Sep 2025 03:53PM UTC coverage: 61.226% (-0.005%) from 61.231%
4775448984420352

Pull #10239

CirrusCI

SomberNight
tests: clear util.callback_mgr between test cases

util.callback_mgr.callbacks was not getting properly cleared between tests.
Every time an Abstract_Wallet or an LNWorker (or many other subclasses of EventListener) is instantiated,
self.register_callbacks() is called in __init__, which puts callbacks into util.callback_mgr.callbacks.
These are only cleaned up if we explicitly call Abstract_Wallet.stop() or LNWorker.stop() later, which we usually do not do in the tests.

As a result, when running multiple unit tests in a row, lots of objects created in a given testcase are never GC-ed and leak into subsequent tests. This is not only a memory leak, but wastes compute too: when events are triggered and cbs get called, these old objects also have their cbs called.

After running all (~1061) unit tests, I observe util.callback_mgr.callbacks had 30 events with a total of 3156 callbacks stored.

On my laptop, running all unit tests previously took ~115 sec, and now it takes ~73 sec.
Pull Request #10239: tests: clear util.callback_mgr between test cases

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

6 existing lines in 2 files now uncovered.

22778 of 37203 relevant lines covered (61.23%)

0.61 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
1
37.14
-0.57% electrum/lnwatcher.py
5
87.4
-1.29% electrum/lnrouter.py
Jobs
ID Job ID Ran Files Coverage
1 4775448984420352.1 26 Sep 2025 04:01PM UTC 88
61.23
Source Files on build 4775448984420352
  • Tree
  • List 88
  • Changed 4
  • Source Changed 0
  • Coverage Changed 4
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Build #47754489...
  • Pull Request #10239
  • PR Base - master (#45996457...)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc