diff options
author | Jon Atack <jon@atack.com> | 2021-09-28 22:52:33 +0200 |
---|---|---|
committer | Jon Atack <jon@atack.com> | 2021-09-30 16:38:55 +0200 |
commit | b658d7d5c5339739dc19bf961d84186469a818d5 (patch) | |
tree | b0958a1a9445061a88e10048ad875bb523b6cbcc /test | |
parent | efa227f5df5f5a9669dec5f1d574cf22d3c0903f (diff) | |
download | bitcoin-b658d7d5c5339739dc19bf961d84186469a818d5.tar.xz |
test: update assert_fee_amount() in test_framework/util.py
- update call to round() with satoshi_round() to avoid intermittent test failures
- rename fee_per_kB to feerate_BTC_kvB for precision
- store division result in feerate_BTC_vB
Diffstat (limited to 'test')
-rw-r--r-- | test/functional/test_framework/util.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/test/functional/test_framework/util.py b/test/functional/test_framework/util.py index ea5c641b4a..2b6f045c72 100644 --- a/test/functional/test_framework/util.py +++ b/test/functional/test_framework/util.py @@ -34,13 +34,14 @@ def assert_approx(v, vexp, vspan=0.00001): raise AssertionError("%s > [%s..%s]" % (str(v), str(vexp - vspan), str(vexp + vspan))) -def assert_fee_amount(fee, tx_size, fee_per_kB): - """Assert the fee was in range""" - target_fee = round(tx_size * fee_per_kB / 1000, 8) +def assert_fee_amount(fee, tx_size, feerate_BTC_kvB): + """Assert the fee is in range.""" + feerate_BTC_vB = feerate_BTC_kvB / 1000 + target_fee = satoshi_round(tx_size * feerate_BTC_vB) if fee < target_fee: raise AssertionError("Fee of %s BTC too low! (Should be %s BTC)" % (str(fee), str(target_fee))) # allow the wallet's estimation to be at most 2 bytes off - if fee > (tx_size + 2) * fee_per_kB / 1000: + if fee > (tx_size + 2) * feerate_BTC_vB: raise AssertionError("Fee of %s BTC too high! (Should be %s BTC)" % (str(fee), str(target_fee))) |