HomePhabricator

validation tests: Use existing {Chainstate,Block}Man

Description

validation tests: Use existing {Chainstate,Block}Man

Summary:
Use {Chain,}TestingSetup's existing {Chainstate,Block}Manager and avoid
unnecessarily creating a local one.

This also helps reduce the code diff for a later commit where we change
{Chainstate,Block}Manager's constructor signature.

This is a backport of core#25815

Test Plan: ninja check

Reviewers: #bitcoin_abc, bytesofman

Reviewed By: #bitcoin_abc, bytesofman

Subscribers: bytesofman

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

Details

Provenance
Carl Dong <contact@carldong.me>Authored on Jul 15 2022, 22:31
PiRKCommitted on Thu, Apr 18, 10:02
PiRKPushed on Thu, Apr 18, 10:02
Reviewer
Restricted Project
Differential Revision
D16004: validation tests: Use existing {Chainstate,Block}Man
Parents
rABCa5cf9ab2059c: [Cashtab] Show cursor pointer for extension open in tab button
Branches
Unknown
Tags
Unknown