Page MenuHomePhabricator

Tests for PSBT

Authored by deadalnix on Nov 2 2019, 01:48.


Group Reviewers
Restricted Project
rSTAGING2633916fbb90: Tests for PSBT
rABC2633916fbb90: Tests for PSBT

Added functional tests for PSBT that test the RPCs. Also added all
of the BIP 174 test vectors (except for the updater tests) in the
functional tests.

Added a Unit test for the BIP 174 updater test vector.

This is a partial backport of Core PR13557 :

Depends on D4352

Test Plan
make check
./test/functionnal/ rpc_psbt

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

deadalnix created this revision.Nov 2 2019, 01:48
Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 2 2019, 01:48
Fabien requested changes to this revision.Nov 4 2019, 13:33
Fabien added a subscriber: Fabien.
Fabien added inline comments.
7 ↗(On Diff #13870)

Nit: move librairies to their own section.

12 ↗(On Diff #13870)


16 ↗(On Diff #13870)

Nit: put above and sort.

41 ↗(On Diff #13870)

p2pkh and p2sh

109 ↗(On Diff #13870)

Is this test failing ?

This revision now requires changes to proceed.Nov 4 2019, 13:33
deadalnix added inline comments.Nov 9 2019, 16:40
109 ↗(On Diff #13870)

ha yes, this is failing due to some segwit stuff, but we don't have segwit. Removing.

deadalnix updated this revision to Diff 14026.Nov 9 2019, 16:46

Update as per comments

Fabien accepted this revision.Nov 9 2019, 17:24
This revision is now accepted and ready to land.Nov 9 2019, 17:24
This revision was automatically updated to reflect the committed changes.