HomePhabricator
Diffusion Bitcoin ABC staging 8cdda9f61858

Actually run assumevalid.py.
8cdda9f61858
Actions

Description

Actually run assumevalid.py.

Summary:
assumevalid was merged as part of PR 9484, but was not added to the
test_runner, so is not run even as part of the extended tests.

This commit adds assumevalid to the list of tests in test_runner. It
also clarifies the code in assumevalid considerably.

This is a backport of core's PR10073

Test Plan:

make check
./test/functional/test_runner.py assumevalid.py

Reviewers: #bitcoin_abc, schancel

Reviewed By: #bitcoin_abc, schancel

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

Details

Provenance
John Newbery <john@johnnewbery.com>Authored on Feb 23 2017, 23:00
deadalnixCommitted on Jan 18 2018, 00:03
deadalnixPushed on Jan 18 2018, 00:11
Reviewer
Restricted Project
Differential Revision
D974: Actually run assumevalid.py.
Parents
rSTAGINGd68156cc3ec5: Finished sigHashType migration in interpreter.cpp
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/2573

Event Timeline

deadalnix committed rSTAGING8cdda9f61858: Actually run assumevalid.py. (authored by John Newbery <john@johnnewbery.com>).Jan 18 2018, 00:03

Changes (2)

PathSize
test/
functional/

rSTAGING8cdda9f61858

View Options

test/functional/assumevalid.py

Loading...
View Options

test/functional/test_runner.py

Loading...