Page MenuHomePhabricator

test: Use compressed keys in TestChain100Setup
ClosedPublic

Authored by PiRK on Oct 19 2022, 12:25.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCe5c4f8aebfb9: 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

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable