HomePhabricator

test: Use compressed keys in TestChain100Setup

Description

test: Use compressed keys in TestChain100Setup

Summary:
coinbaseKey.MakeNewKey(true); creates a compressed key and there is no reason for the deterministic setup to use uncompressed ones.

Note that there is no test covering the assumeutxo hash at height 210, so upstream didn't bother updating it. I updated it assuming that when this snapshot eventually exists in tests it will just have 100 more standard blocks on top of the previous 110.

This is a partial backport of core#21592
https://github.com/bitcoin/bitcoin/pull/21592/commits/fa732bccb3cf9bc2cdc444975286df0e799917a2

Test Plan: ninja && ninja check&& test/functional/test_runner.py feature_deterministic_chain_setup

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

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

Details

Provenance
MarcoFalke <falke.marco@gmail.com>Authored on Apr 4 2021, 09:52
PiRKCommitted on Oct 19 2022, 17:18
PiRKPushed on Oct 19 2022, 17:18
Reviewer
Restricted Project
Differential Revision
D12305: test: Use compressed keys in TestChain100Setup
Parents
rABCe6ed010a5e22: addrman: Avoid crash on corrupt data, Force Check after deserialize
Branches
Unknown
Tags
Unknown