Enable mempool consistency checks in unit tests #8222

pull sipa wants to merge 1 commits into bitcoin:master from sipa:unittestmempool changing 1 files +1 −0

sipa commented at 5:15 PM on June 18, 2016: member

This helped me discovered #8220.

I can't measure any slowdown for running the unit tests.
Enable mempool consistency checks in unit tests 3775ff9ea7
petertodd commented at 9:19 PM on June 18, 2016: contributor

Concept ACK
MarcoFalke added the label Tests on Jun 19, 2016
MarcoFalke commented at 8:57 PM on June 19, 2016: member

utACK 3775ff9
gmaxwell commented at 9:14 PM on June 19, 2016: contributor

Tested ACK. Just a very slight reservation that perhaps a consistency check being enabled someday hides a bug.
laanwj commented at 8:18 AM on June 20, 2016: member

Sounds like a good idea to me, concept ACK.

Just a very slight reservation that perhaps a consistency check being enabled someday hides a bug.

This just adds checks I, hope? It doesn't ever correct consistency?
laanwj merged this on Jun 22, 2016
laanwj closed this on Jun 22, 2016
laanwj referenced this in commit 961893f26e on Jun 22, 2016
codablock referenced this in commit 2972994086 on Sep 16, 2017
codablock referenced this in commit 9e9a4a8714 on Sep 19, 2017
codablock referenced this in commit f93589be59 on Dec 27, 2017
codablock referenced this in commit e51f505449 on Dec 28, 2017
andvgal referenced this in commit 7d732cf202 on Jan 6, 2019
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels