test: add decodescript RPC test for P2TR output script #23540

pull theStack wants to merge 3 commits into bitcoin:master from theStack:202111-test-improve_decodescript_tests changing 1 files +46 −19

theStack commented at 12:05 AM on November 18, 2021: member

This PR adds a functional sub-test for calling decodescript with a P2TR / segwit v1 output script (OP_1 <32-bytes push>), expecting to return "witness_v1_taproot" as type result.

In the first two commits, the test rpc_decodescript.py is also improved by adding logging (plus getting rid of the enumerations) and also adding missing checks type result checks for all other output script types.

test: add logging to rpc_decodescript.py

Also remove the enumerations ("1)", "2)"...) from the test
cases as those potentially hinder maintainability; e.g. if a
new case in inserted in-between, all the remaining
enumerations would need to be adapted.

0d43525c61

test: check for decodescript RPC 'type' results 099c6957de
fanquake added the label Tests on Nov 18, 2021
test: add decodescript RPC test for P2TR output type 83f6c0f9ef
theStack force-pushed on Nov 18, 2021
MarcoFalke commented at 7:02 AM on November 18, 2021: member

ACK 83f6c0f9ef3a1962be02db479ddf87cb22b8c14b
MarcoFalke merged this on Nov 18, 2021
MarcoFalke closed this on Nov 18, 2021
sidhujag referenced this in commit 2af6963775 on Nov 19, 2021
sidhujag referenced this in commit 114bc9b371 on Nov 19, 2021
theStack deleted the branch on Nov 30, 2021
DrahtBot locked this on Nov 30, 2022

Contributors

theStack

MarcoFalke

Labels

Tests