rpc, test: document {previous,next}blockhash as optional #21053

pull theStack wants to merge 1 commits into bitcoin:master from theStack:2021-rpc-test-document-previousnextblockhash-as-optional changing 2 files +10 −4

theStack commented at 4:45 PM on February 1, 2021: member
This PR updates the result help of the following RPCs w.r.t. the previousblockhash and nextblockhash fields:
- getblockheader
- getblock
Also adds trivial tests on genesis block (should not contain "previousblockhash") and best block (should not contain "nextblockhash").

rpc, test: document {previous,next}blockhash as optional

Affects the following RPCs:
- getblockheader
- getblock

Also adds trivial tests on genesis block (should not contain
"previousblockhash") and best block (should not contain
"nextblockhash").

ba7e17e073

fanquake added the label RPC/REST/ZMQ on Feb 1, 2021
jonatack commented at 7:43 PM on February 1, 2021: member

utACK. I see happy path tests for "previousblockhash"; are we missing the same for "nextblockhash"?
luke-jr approved
luke-jr commented at 5:05 PM on February 23, 2021: member

utACK
MarcoFalke merged this on Feb 23, 2021
MarcoFalke closed this on Feb 23, 2021
sidhujag referenced this in commit d3c053b9ba on Feb 23, 2021
Fabcien referenced this in commit 77d52b9d3c on Apr 13, 2022
DrahtBot locked this on Aug 16, 2022

Contributors

Labels