diff options
author | John Newbery <john@johnnewbery.com> | 2021-09-27 16:40:25 +0100 |
---|---|---|
committer | John Newbery <john@johnnewbery.com> | 2021-11-03 12:04:49 +0000 |
commit | 5759fd12b8d5937e9187fa33489a95b1d8e6d1e5 (patch) | |
tree | 3b8232fea2b531065197a560f621699749f2e4f2 | |
parent | 497c9e29640858bb3beb20089c2d4f9e133c7e42 (diff) |
[test] Don't set bypass_limits to true in txvalidation_tests.cpp
AcceptToMemoryPool() is called for an invalid coinbase transaction, so
setting bypass_limits to true or false has no impact on the test.
The only way that changing bypass_limits from true to false could change
the result would be to change the outcome to INVALID(TX_MEMPOOL_POLICY).
Since the ATMP call in this test results in INVALID(TX_CONSENSUS) both
before and after this change, there is no change in behavior.
-rw-r--r-- | src/test/txvalidation_tests.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/txvalidation_tests.cpp b/src/test/txvalidation_tests.cpp index ade9e210f2..a966c30720 100644 --- a/src/test/txvalidation_tests.cpp +++ b/src/test/txvalidation_tests.cpp @@ -38,7 +38,7 @@ BOOST_FIXTURE_TEST_CASE(tx_mempool_reject_coinbase, TestChain100Setup) unsigned int initialPoolSize = m_node.mempool->size(); const MempoolAcceptResult result = AcceptToMemoryPool(m_node.chainman->ActiveChainstate(), *m_node.mempool, MakeTransactionRef(coinbaseTx), - true /* bypass_limits */); + false /* bypass_limits */); BOOST_CHECK(result.m_result_type == MempoolAcceptResult::ResultType::INVALID); |