[test] add a verbose logger to ChronikElectrumClient
ClosedPublic
Actions

Authored by PiRK on May 22 2025, 20:46.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCfcfc6e89405b: [test] add a verbose logger to ChronikElectrumClient

Summary

This will help troubleshoot test failures related to unexpected notifications and multiple clients.
The logger can use an optional name passed to the ChronikElectrumClient.__init__ method for easier debugging. By default this "name" is a unique number.

Test Plan

ninja check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronikelectrummaxhistory_typo

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33404
Build 66287: Build Diff	build-diff · build-without-wallet · build-clang · build-debug · build-clang-tidy
Build 66286: arc lint + arc unit

Event Timeline

PiRK created this revision.May 22 2025, 20:46

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 22 2025, 20:46

PiRK requested review of this revision.May 22 2025, 20:46

PiRK added inline comments.May 22 2025, 20:49

test/functional/test_framework/jsonrpctools.py
93 ↗	(On Diff #54131)	By prefixing the logger name with "TestFramework." we inherit the settings (logging level, output file...) of the parent logger defined in test_framework.py