Don't test against the mempool min fee information in mempool_limit.py #12512

pull Empact wants to merge 1 commits into bitcoin:master from Empact:fix-min-fee-test changing 1 files +1 −1

Empact commented at 9:35 PM on February 22, 2018: member

Because the right-hand side of this comparison can be influenced externally, e.g. via the -maxmempool argument, the existing mempool state, host memory usage, etc.

Called out by @MarcoFalke here: #12356 (review)

Don't test against the mempool min fee information in mempool_limit.py

Because the right-hand side of this comparison can be influenced
externally, e.g. via the -maxmempool argument, the existing mempool state,
host memory usage, etc.

55f89da1a5

MarcoFalke commented at 9:37 PM on February 22, 2018: member

utACK 55f89da1a5048cf275c37bceef664e7c04831a35
fanquake added the label Tests on Feb 24, 2018
practicalswift commented at 4:32 PM on February 24, 2018: contributor

utACK 55f89da1a5048cf275c37bceef664e7c04831a35
conscott commented at 2:49 PM on February 27, 2018: contributor

Test ACK 55f89da1a5048cf275c37bceef664e7c04831a35
laanwj merged this on Mar 5, 2018
laanwj closed this on Mar 5, 2018
laanwj referenced this in commit 955fd23517 on Mar 5, 2018
Empact deleted the branch on May 16, 2018
PastaPastaPasta referenced this in commit d3725fd130 on Jun 11, 2020
PastaPastaPasta referenced this in commit 31bc883118 on Jun 11, 2020
DrahtBot locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#12356 Fix 'mempool min fee not met' debug output