diff options
author | Sebastian Falbesoner <sebastian.falbesoner@gmail.com> | 2023-03-24 17:46:15 +0100 |
---|---|---|
committer | Sebastian Falbesoner <sebastian.falbesoner@gmail.com> | 2023-03-24 17:54:09 +0100 |
commit | afc2dd54848fa70ed408ae259420ff8648f21efc (patch) | |
tree | 94f710b6f86c5f2039470e0b72abe83b7b2b7f07 /test/functional/wallet_hd.py | |
parent | e352f5ab6b60ec1cc549997275e945238508cdee (diff) | |
download | bitcoin-afc2dd54848fa70ed408ae259420ff8648f21efc.tar.xz |
test: various `converttopsbt` check cleanups in rpc_psbt.py
In the functional test rpc_psbt.py, some comments around the
`converttopsbt` RPC checks are wrong or outdated and can be
removed:
> Error could be either "TX decode failed" (segwit inputs causes
> parsing to fail) or "Inputs must not have scriptSigs and
> scriptWitnesses"
Decoding a valid TX with at least one input always succeeds with the
heuristic, i.e. this comment is not right and we can assert for the
error string "Inputs must not have scriptSigs and scriptWitnesses"
on the calls below.
> We must set iswitness=True because the serialized transaction has
> inputs and is therefore a witness transaction
This is also unneeded (and confusing, w.r.t. "is therefore a witness
transaction"?), for a TX with one input there is no need to set the
`iswitness` parameter. For sake of completeness, we still keep one
variant where iswitness is explicitly set to true.
Lastly, there is a superflous `converttopsbt` call on the raw tx which
is the same as just about ~10 lines above, so it can be removed.
Diffstat (limited to 'test/functional/wallet_hd.py')
0 files changed, 0 insertions, 0 deletions