Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rSTAGINGc453a23e9675: Add outputtype module
rABCc453a23e9675: Add outputtype module

Summary

Moves OutputType into its own module

Move AddAndGetDestinationForScript from wallet to outputype module

Makes AddAndGetDestinationForScript use a generic CKeyStore rather than
the wallet, and makes it always add the script to the keystore, rather
than only adding related (redeem) scripts.

segwit support for createmultisig RPC
[tests] functional test for createmultisig RPC

This is a backport of Core PR13072

Depends on D4423

Test Plan

make check
./test/functional/test_runner.py

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr13072

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 8053
Build 14101: Bitcoin ABC Buildbot (legacy)
Build 14100: arc lint + arc unit

Event Timeline

deadalnix created this revision.Nov 11 2019, 01:03

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 11 2019, 01:03

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8053: Diff 14037.Nov 11 2019, 01:14

What is the purpose of the OutputType for us, intended that we don't do segwit ?

It's not super useful, ut helps backports.

rebase

Build Bitcoin-ABC / Diffs / Diff Testing started.

Harbormaster completed remote builds in B8076: Diff 14076.Nov 12 2019, 16:21

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Fabien requested changes to this revision.Nov 14 2019, 15:58

Fabien added inline comments.

src/outputtype.h
21 ↗	(On Diff #14076)	Despite somewhat documented in a couple places, there is no such option in our codebase.
test/functional/rpc_createmultisig.py
26 ↗	(On Diff #14076)	BTC => BCH
49 ↗	(On Diff #14076)	Nit: spaces around operators
52 ↗	(On Diff #14076)	Dito

This revision now requires changes to proceed.Nov 14 2019, 15:58

Address comments

Build Bitcoin-ABC / Diffs / Diff Testing started.

Harbormaster completed remote builds in B8134: Diff 14160.Nov 16 2019, 13:48

Build Bitcoin-ABC / Diffs / Diff Testing passed.

deadalnix added inline comments.Nov 16 2019, 14:46

test/functional/rpc_createmultisig.py
49 ↗	(On Diff #14076)	I'm not sure what you want me to do here.

Fabien added inline comments.Nov 16 2019, 22:10

test/functional/rpc_createmultisig.py
49 ↗	(On Diff #14076)	It's just style nit, I want you to surround the operators with spaces, e.g. here: `total = 149 * 50 + (height - 149 - 100) * 25` I'll see if there is a way to tell `autopep8` to do it automatically, but I think I did already search for this feature with no success.