diff options
| author | Gregory Sanders <[email protected]> | 2018-03-28 15:39:26 -0400 |
|---|---|---|
| committer | Gregory Sanders <[email protected]> | 2018-03-28 15:39:26 -0400 |
| commit | f526046ef5d8300eab6f112f56f59fd3b7e11d33 (patch) | |
| tree | c6889f5a3ac60851e3b5652a79d3b97cb14ea90b | |
| parent | feebumper: discard change outputs below discard rate (diff) | |
| download | discoin-f526046ef5d8300eab6f112f56f59fd3b7e11d33.tar.xz discoin-f526046ef5d8300eab6f112f56f59fd3b7e11d33.zip | |
adapt bumpfee change discard test to be more strict and add note on p2sh discrep
| -rwxr-xr-x | test/functional/wallet_bumpfee.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/test/functional/wallet_bumpfee.py b/test/functional/wallet_bumpfee.py index 3e496248f..f036481c3 100755 --- a/test/functional/wallet_bumpfee.py +++ b/test/functional/wallet_bumpfee.py @@ -181,7 +181,10 @@ def test_dust_to_fee(rbf_node, dest_address): # the bumped tx sets fee=49,900, but it converts to 50,000 rbfid = spend_one_input(rbf_node, dest_address) fulltx = rbf_node.getrawtransaction(rbfid, 1) - bumped_tx = rbf_node.bumpfee(rbfid, {"totalFee": 49900}) + # (32-byte p2sh-pwpkh output size + 148 p2pkh spend estimate) * 10k(discard_rate) / 1000 = 1800 + # P2SH outputs are slightly "over-discarding" due to the IsDust calculation assuming it will + # be spent as a P2PKH. + bumped_tx = rbf_node.bumpfee(rbfid, {"totalFee": 50000-1800}) full_bumped_tx = rbf_node.getrawtransaction(bumped_tx["txid"], 1) assert_equal(bumped_tx["fee"], Decimal("0.00050000")) assert_equal(len(fulltx["vout"]), 2) |