Skip to content

Commit 6945853

Browse files
committed
test: use NoLimits() in MempoolIndexingTest
The (100, 1000000, 1000, 1000000) limits are arbitrarily high and don't restrict anything, they are just meant to calculate ancestors properly. Using NoLimits() makes this intent more clear and simplifies the code.
1 parent 3a86f24 commit 6945853

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/test/mempool_tests.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ BOOST_AUTO_TEST_CASE(MempoolIndexingTest)
203203

204204
CTxMemPool::setEntries setAncestorsCalculated;
205205
std::string dummy;
206-
BOOST_CHECK_EQUAL(pool.CalculateMemPoolAncestors(entry.Fee(2000000LL).FromTx(tx7), setAncestorsCalculated, 100, 1000000, 1000, 1000000, dummy), true);
206+
BOOST_CHECK_EQUAL(pool.CalculateMemPoolAncestors(entry.Fee(2'000'000LL).FromTx(tx7), setAncestorsCalculated, CTxMemPool::Limits::NoLimits(), dummy), true);
207207
BOOST_CHECK(setAncestorsCalculated == setAncestors);
208208

209209
pool.addUnchecked(entry.FromTx(tx7), setAncestors);
@@ -261,7 +261,7 @@ BOOST_AUTO_TEST_CASE(MempoolIndexingTest)
261261
tx10.vout[0].nValue = 10 * COIN;
262262

263263
setAncestorsCalculated.clear();
264-
BOOST_CHECK_EQUAL(pool.CalculateMemPoolAncestors(entry.Fee(200000LL).Time(4).FromTx(tx10), setAncestorsCalculated, 100, 1000000, 1000, 1000000, dummy), true);
264+
BOOST_CHECK_EQUAL(pool.CalculateMemPoolAncestors(entry.Fee(200'000LL).Time(4).FromTx(tx10), setAncestorsCalculated, CTxMemPool::Limits::NoLimits(), dummy), true);
265265
BOOST_CHECK(setAncestorsCalculated == setAncestors);
266266

267267
pool.addUnchecked(entry.FromTx(tx10), setAncestors);

0 commit comments

Comments
 (0)