test: getblock on header throws #27237

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2303-test-block-header-throw-📞 changing 1 files +6 −0

maflcko commented at 9:58 AM on March 10, 2023: member

This adds a test to check that getblock throws when only the block header is present.
DrahtBot commented at 9:58 AM on March 10, 2023: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Reviews

See the guideline for information on the review process. A summary of reviews will appear here.
maflcko renamed this:
~~2303 test block header throw 📞~~
test: getblock on header throws
on Mar 10, 2023
DrahtBot renamed this:
~~test: getblock on header throws~~
test: getblock on header throws
on Mar 10, 2023
DrahtBot added the label Tests on Mar 10, 2023
fanquake commented at 1:41 PM on March 23, 2023: member

#18933 is now in.
test: getblock on header throws fa29c12a08
maflcko marked this as ready for review on Mar 23, 2023
maflcko force-pushed on Mar 23, 2023
maflcko commented at 7:22 PM on March 23, 2023: member

Not sure if it makes sense to add a test only. Maybe the error message can be improved along? Closing, let's continue discussion in #https://github.com/bitcoin/bitcoin/issues/20978
maflcko closed this on Mar 23, 2023
maflcko deleted the branch on Mar 23, 2023
bitcoin locked this on Mar 22, 2024

Contributors

Labels