Differential D18107

[chronik] Electrum: don't assume notifications are returning in array in tests
ClosedPublic
Actions

Authored by Fabien on Mon, May 19, 08:00.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC18334335d361: [chronik] Electrum: don't assume notifications are returning in array in tests

Summary

This is the pendant of D18105 but for the test framework.

Depends on D18105.

Test Plan

./test/functional/test_runner.py chronik_electrum_*

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik_electrum_test_no_array

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33359
Build 66199: Build Diff	build-without-wallet · build-clang-tidy · build-clang · build-debug · build-diff
Build 66198: arc lint + arc unit

Event Timeline

Fabien created this revision.Mon, May 19, 08:00

Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, May 19, 08:00

Fabien requested review of this revision.Mon, May 19, 08:00

Fabien added a child revision: D18108: [chronik] Electrum: handle unconfirmed parents in history.Mon, May 19, 08:51

PiRK accepted this revision.Mon, May 19, 08:55

This revision is now accepted and ready to land.Mon, May 19, 08:55

This revision was landed with ongoing or failed builds.Mon, May 19, 09:15

Closed by commit rABC18334335d361: [chronik] Electrum: don't assume notifications are returning in array in tests (authored by Fabien). · Explain Why

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC18334335d361: [chronik] Electrum: don't assume notifications are returning in array in tests.

Revision Contents
Changeset List

Path

Size

test/

functional/

chronik_electrum_blockchain.py

2 lines

test_framework/

jsonrpctools.py

6 lines

Diff 54064

View Options

test/functional/chronik_electrum_blockchain.py