diff options
author | MarcoFalke <falke.marco@gmail.com> | 2019-12-03 10:25:23 -0500 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2019-12-03 10:25:34 -0500 |
commit | 2b6575d9895c7da2ec64f2cb04be150f91a600f8 (patch) | |
tree | 3abc866339d67f425e17b73a21673b0967589bc4 /test | |
parent | 35eda631ed3bd23d4a41761a85a96f925d4a6337 (diff) | |
parent | 02afb0c550dc8529918460c845d1da3adf236eed (diff) | |
download | bitcoin-2b6575d9895c7da2ec64f2cb04be150f91a600f8.tar.xz |
Merge #17643: wallet: Fix origfee return for bumpfee with feerate arg
02afb0c550dc8529918460c845d1da3adf236eed Fix origfee return for bumpfee with feerate arg (Gregory Sanders)
Pull request description:
fixes https://github.com/bitcoin/bitcoin/issues/17642 and adds a simple test that would have caught it
ACKs for top commit:
achow101:
ACK 02afb0c550dc8529918460c845d1da3adf236eed
Tree-SHA512: 303e392e05407f204dffe360689b5bb5dc77fd462dd0e489bc0b6c8f94f89ab7fe2bd8cb47e4dc6dc5c23a619826d15f3bf6b02b2c8e96402fbb51953c462e2d
Diffstat (limited to 'test')
-rwxr-xr-x | test/functional/wallet_bumpfee.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/test/functional/wallet_bumpfee.py b/test/functional/wallet_bumpfee.py index 9d6aa36c35..0c08655833 100755 --- a/test/functional/wallet_bumpfee.py +++ b/test/functional/wallet_bumpfee.py @@ -101,7 +101,8 @@ def test_simple_bumpfee_succeeds(self, mode, rbf_node, peer_node, dest_address): else: bumped_tx = rbf_node.bumpfee(rbfid) assert_equal(bumped_tx["errors"], []) - assert bumped_tx["fee"] - abs(rbftx["fee"]) > 0 + assert bumped_tx["fee"] > -rbftx["fee"] + assert_equal(bumped_tx["origfee"], -rbftx["fee"]) # check that bumped_tx propagates, original tx was evicted and has a wallet conflict self.sync_mempools((rbf_node, peer_node)) assert bumped_tx["txid"] in rbf_node.getrawmempool() |