HomePhabricator

[test] fix intermittent issues in chronik_electrum_blockchain

Description

[test] fix intermittent issues in chronik_electrum_blockchain

Summary:
The test makes the incorrect assumptions that when a chain of 3 transactions involving the same scripthash is broadcast, the Electrum server will send 3 notifications.
In reality the server will queue 3 notifications, but when it computes the status of the indexed transactions for the first notification Chronik may already have indexed 2 or all 3 transactions. The Electrum server only actually sends the notification if the status changes, so if the first notificaton already accounts for the second transaction, it will not send out the second notification.

Fix this by actually waiting for the previous notification before sending the next transaction.

Test Plan: test/functional/test_runner.py --repeat 1000 chronik_electrum_blockchain

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D18131

Details

Provenance
PiRKAuthored on May 23 2025, 11:08
PiRKPushed on May 23 2025, 12:30
Reviewer
Restricted Project
Differential Revision
D18131: [test] fix intermittent issues in chronik_electrum_blockchain
Parents
rABCfcfc6e89405b: [test] add a verbose logger to ChronikElectrumClient
Branches
Unknown
Tags
Unknown