The current tx_pool target uses the default mempool, making the target non-deterministic. This PR replaces the active chainstate's mempool (i.e. the node's default mempool) with the already present mocked mempool in the target.
fuzz: Actually use mocked mempool in tx_pool target #26930
pull dergoegge wants to merge 1 commits into bitcoin:master from dergoegge:2023-01-determ-fuzz-tx-pool changing 1 files +3 −1-
dergoegge commented at 11:23 AM on January 20, 2023: member
-
[fuzz] Actually use mocked mempool in tx_pool target 9ab62d71fb
-
DrahtBot commented at 11:23 AM on January 20, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK fanquake If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Tests on Jan 20, 2023
-
maflcko commented at 12:18 PM on January 20, 2023: member
lgtm
Looks like the bug was introduced in f1f10c0514fe81318c8b064f9dad0e2f9a2cd037
- maflcko approved
- fanquake approved
-
fanquake commented at 5:54 PM on January 23, 2023: member
ACK 9ab62d71fb1a54430ff5071bdb1120a414061288
- maflcko merged this on Jan 24, 2023
- maflcko closed this on Jan 24, 2023
- sidhujag referenced this in commit 53e36aaca8 on Jan 24, 2023
- bitcoin locked this on Jan 24, 2024
