HomePhabricator

Merge #11877: Improve createrawtransaction functional tests

Description

Merge #11877: Improve createrawtransaction functional tests

Summary:
PR11877 backport https://github.com/bitcoin/bitcoin/pull/11877/files
modifications:

  • for "vout must be a number" ABC error message
  • remove stuff related to RBF

88af502 test: Add createrawtransaction functional tests (João Barbosa)
27c6199 test: Add multidict to support dictionary with duplicate key (laanwj) (João Barbosa)
320669a rpc: Validate replaceable type in createrawtransaction (João Barbosa)

Pull request description:

This was motivated by the `Invalid parameter, duplicated address` test.

Credit to @laanwj for `multidict` implementation.

Test Plan: test_runner.py

Reviewers: jasonbcox, #bitcoin_abc, deadalnix, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

Maniphest Tasks: T639

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

Details

Provenance
Wladimir J. van der Laan <laanwj@gmail.com>Authored on Dec 13 2017, 16:12
Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>Committed on May 20 2019, 16:07
markblundebergPushed on May 20 2019, 16:07
Reviewer
Restricted Project
Differential Revision
D3072: Merge #11877: Improve createrawtransaction functional tests
Parents
rABC8f90b867e96b: Merge #12278: Add special error for genesis coinbase to getrawtransaction
Branches
Unknown
Tags
Unknown

Event Timeline

Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com> committed rABC313eca342457: Merge #11877: Improve createrawtransaction functional tests (authored by Wladimir J. van der Laan <laanwj@gmail.com>).May 20 2019, 16:07