Page MenuHomePhabricator

[Tests] Cleanup extra instances of create_transaction

Authored by nakihito on Jan 2 2020, 21:48.



Partial backport of Core PR13669

Changes to omitted because they caused the
test to fail the assert in create_transaction() due to divergences in our codebases.

Depends on D4820

Test Plan
test/functional/ abc-mempool-accept-txn abc-mempool-coherence-on-activations abc-minimaldata abc-replay-protection abc-schnorr abc-schnorrmultisig feature_block feature_cltv feature_dersig p2p_invalid_block p2p_invalid_tx p2p_unrequested_blocks

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Owners added a reviewer: Restricted Owners Package.Jan 2 2020, 21:48
nakihito requested review of this revision.Jan 3 2020, 20:19
nakihito edited the summary of this revision. (Show Details)
nakihito added inline comments.
93 ↗(On Diff #15118)

This goes by a different name in our codebase, but is equivalent to Core's create_transaction() here:

This revision is now accepted and ready to land.Jan 4 2020, 06:35