test: adjust chainstate tests to use recognized snapshot base
ClosedPublic
Actions

Authored by PiRK on Wed, Mar 5, 14:43.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC43c4d6926372: test: adjust chainstate tests to use recognized snapshot base

Summary

In future commits, loading the block index while making use of a
snapshot is contingent on the snapshot being recognized by chainparams.

Ensure all existing unittests that use snapshots use a recognized
snapshot (at height 110).

Co-authored-by: Ryan Ofsky <ryan@ofsky.org>

This is a partial backport of core#27596
https://github.com/bitcoin/bitcoin/pull/27596/commits/49ef778158c43859946a592e11ec34fe1b93a5b6

Test Plan

ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.Wed, Mar 5, 14:43

Herald added a reviewer: Restricted Project. · View Herald TranscriptWed, Mar 5, 14:43

PiRK requested review of this revision.Wed, Mar 5, 14:43

teamcity edited the summary of this revision. (Show Details)Wed, Mar 5, 14:43

Harbormaster completed remote builds in B32596: Diff 52933.Wed, Mar 5, 14:54

Fabien accepted this revision.Thu, Mar 6, 08:44

This revision is now accepted and ready to land.Thu, Mar 6, 08:44

Closed by commit rABC43c4d6926372: test: adjust chainstate tests to use recognized snapshot base (authored by James O'Beirne <james.obeirne@pm.me>, committed by PiRK). · Explain WhyThu, Mar 6, 09:35

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC43c4d6926372: test: adjust chainstate tests to use recognized snapshot base.

Revision Contents
Changeset List

Path

Size

src/

test/

validation_chainstatemanager_tests.cpp

106 lines

validation.cpp

4 lines

Diff 52944

View Options

src/test/validation_chainstatemanager_tests.cpp