From e528b34ed88d9d4652cbc0b22dc2badfeaaa4793 Mon Sep 17 00:00:00 2001 From: Pablo Greco Date: Thu, 24 Nov 2022 20:33:02 +0000 Subject: [PATCH] Fix tests after backport of #22949 --- src/test/miner_tests.cpp | 2 +- test/functional/wallet_keypool.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/miner_tests.cpp b/src/test/miner_tests.cpp index f6bc0ace01..6f817c4e7b 100644 --- a/src/test/miner_tests.cpp +++ b/src/test/miner_tests.cpp @@ -195,7 +195,7 @@ void MinerTestingSetup::TestPackageSelection(const CChainParams& chainparams, co m_node.mempool->addUnchecked(entry.Fee(10000).FromTx(tx)); pblocktemplate = AssemblerForTest(chainparams).CreateNewBlock(scriptPubKey); BOOST_REQUIRE_EQUAL(pblocktemplate->block.vtx.size(), 9U); - BOOST_CHECK(pblocktemplate->block.vtx[8]->GetHash() == hashLowFeeTx2); + BOOST_CHECK(pblocktemplate->block.vtx[6]->GetHash() == hashLowFeeTx2); } // NOTE: These tests rely on CreateNewBlock doing its own self-validation! diff --git a/test/functional/wallet_keypool.py b/test/functional/wallet_keypool.py index 7d1e7162da..66dae49ac1 100755 --- a/test/functional/wallet_keypool.py +++ b/test/functional/wallet_keypool.py @@ -184,7 +184,7 @@ class KeyPoolTest(BitcoinTestFramework): # create a transaction without change at the maximum fee rate, such that the output is still spendable: res = w2.walletcreatefundedpsbt(inputs=[], outputs=[{destination: 0.00025000}], options={"subtractFeeFromOutputs": [0], "feeRate": 0.0008823}) assert_equal("psbt" in res, True) - assert_equal(res["fee"], Decimal("0.00016853")) + assert_equal(res["fee"], Decimal("0.00016852")) # creating a 10,000 sat transaction with a manual change address should be possible res = w2.walletcreatefundedpsbt(inputs=[], outputs=[{destination: 0.00025000}], options={"subtractFeeFromOutputs": [0], "feeRate": 0.00010, "changeAddress": addr.pop()})