HomePhabricator

[qa] Split out transaction creation from next_block in functional test

Description

[qa] Split out transaction creation from next_block in functional test

Summary:
This allows the easy creation of a tx to be used in this test without requiring next_block() to be called.
next_block() changes self.tip as a side effect, which is unexpected when only a tx is desired. This forces the test
writer to call tip(...) after generating invalid txs to bring the tip back to a good state. This is a bad code smell.

Test Plan: test_runner.py abc-magnetic-anomaly-activation

Reviewers: deadalnix, schancel, #bitcoin_abc

Reviewed By: schancel, #bitcoin_abc

Subscribers: teamcity

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

Details

Provenance
jasonbcoxAuthored on Oct 16 2018, 18:00
FabienPushed on Oct 18 2018, 17:14
Reviewer
Restricted Project
Differential Revision
D1930: [qa] Split out transaction creation from next_block in functional test
Parents
rSTAGINGa46b1d0e35dd: Add OSX keystroke to clear RPCConsole
Branches
Unknown
Tags
Unknown