foster/bitcoin-bitcoin-core
0
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2025-02-02 09:46:52 -05:00
Code Issues Activity

test: relax bumpfee dust_to_fee txsize an extra vbyte

Browse source 
and add explanatory documentation for the reasoning.
This commit is contained in:
Jon Atack 2020-04-03 17:12:08 +02:00
parent f0d6487e29
commit 25e03ba1ff
No known key found for this signature in database
GPG key ID: 4F5721B3D0E3921D
1 changed files with 12 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
test/functional/wallet_bumpfee.py
View file

@ -253,12 +253,18 @@ def test_dust_to_fee(self, rbf_node, dest_address):
self.log.info('Test that bumped output that is dust is dropped to fee') self.log.info('Test that bumped output that is dust is dropped to fee')
rbfid = spend_one_input(rbf_node, dest_address) rbfid = spend_one_input(rbf_node, dest_address)
fulltx = rbf_node.getrawtransaction(rbfid, 1) fulltx = rbf_node.getrawtransaction(rbfid, 1)
# size of transaction (p2wpkh, 1 input, 2 outputs): 141 vbytes # The DER formatting used by Bitcoin to serialize ECDSA signatures means that signatures can have a
assert_equal(fulltx["vsize"], 141) # variable size of 70-72 bytes (or possibly even less), with most being 71 or 72 bytes. The signature
# bump with fee_rate of 0.00350000 BTC per 1000 vbytes # in the witness is divided by 4 for the vsize, so this variance can take the weight across a 4-byte
# expected bump fee of 141 vbytes * fee_rate 0.00350000 BTC / 1000 vbytes = 0.00049350 BTC # boundary. Thus expected transaction size (p2wpkh, 1 input, 2 outputs) is 140-141 vbytes, usually 141.
# but dust is dropped, so actual bump fee is 0.00050000 if not 140 <= fulltx["vsize"] <= 141:
bumped_tx = rbf_node.bumpfee(rbfid, {"fee_rate": 0.0035}) print("Error: Invalid tx vsize of {} (140-141 expected), full tx: {}".format(fulltx["vsize"], fulltx))
raise AssertionError
# Bump with fee_rate of 0.00350250 BTC per 1000 vbytes to create dust.
# Expected fee is 141 vbytes * fee_rate 0.00350250 BTC / 1000 vbytes = 0.00049385 BTC.
# or occasionally 140 vbytes * fee_rate 0.00350250 BTC / 1000 vbytes = 0.00049035 BTC.
# Dust should be dropped to the fee, so actual bump fee is 0.00050000 BTC.
bumped_tx = rbf_node.bumpfee(rbfid, {"fee_rate": 0.00350250})
full_bumped_tx = rbf_node.getrawtransaction(bumped_tx["txid"], 1) full_bumped_tx = rbf_node.getrawtransaction(bumped_tx["txid"], 1)
assert_equal(bumped_tx["fee"], Decimal("0.00050000")) assert_equal(bumped_tx["fee"], Decimal("0.00050000"))
assert_equal(len(fulltx["vout"]), 2) assert_equal(len(fulltx["vout"]), 2)