p2p_orphan_handling
relied on checking the debug log for orphanage changes. This updates the tests to reduce debug log checking and add checks using tx_in_orphanage()
and getorphantxs
introduced in #30793.
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/31037.
See the guideline for information on the review process.
Type | Reviewers |
---|---|
ACK | glozow, rkrux, danielabrozzoni |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
p2p_orphan_handling now uses tx_in_orphanage
to more directly check for inclusion/exclusion
in the orphanage.
Increases test robustness by adding
checks for orphanage size and presence
of orphans in the orphanage
tACK 9de9c858d5aa412e1c6086e564fbe85984a4f2d5
Successful make and all functional tests. This is my preference as well to use the getorphantxs
RPC for asserting over searching in the debug logs.