aboutsummaryrefslogtreecommitdiff
path: root/src/test/amount_tests.cpp
diff options
context:
space:
mode:
authorRoss Nicoll <[email protected]>2021-08-09 23:27:38 +0100
committerRoss Nicoll <[email protected]>2021-08-17 08:50:42 +0100
commitce6e3d8137128e99b2f7f99f79327b8b440df0a2 (patch)
treeefcc20108c599f774d5df680eed7c91e7476b14b /src/test/amount_tests.cpp
parentMerge pull request #2457 from ReverseControl/1.14.4-suggested-changes-for-2297 (diff)
downloaddiscoin-ce6e3d8137128e99b2f7f99f79327b8b440df0a2.tar.xz
discoin-ce6e3d8137128e99b2f7f99f79327b8b440df0a2.zip
Remove relay-only rounding
Remove rounding of transaction sizes when calculating fee minimums for relaying, to simplify fee logic.
Diffstat (limited to 'src/test/amount_tests.cpp')
-rw-r--r--src/test/amount_tests.cpp10
1 files changed, 9 insertions, 1 deletions
diff --git a/src/test/amount_tests.cpp b/src/test/amount_tests.cpp
index 47e73ed38..80a6c3a6f 100644
--- a/src/test/amount_tests.cpp
+++ b/src/test/amount_tests.cpp
@@ -19,7 +19,7 @@ BOOST_AUTO_TEST_CASE(GetFeeTest)
BOOST_CHECK_EQUAL(feeRate.GetFee(1e5), 0);
feeRate = CFeeRate(1000);
- // Must always just return the arg
+ // Wallet fees are rounded up
BOOST_CHECK_EQUAL(feeRate.GetFee(0), 0);
BOOST_CHECK_EQUAL(feeRate.GetFee(1), 1000);
BOOST_CHECK_EQUAL(feeRate.GetFee(121), 1000);
@@ -27,6 +27,14 @@ BOOST_AUTO_TEST_CASE(GetFeeTest)
BOOST_CHECK_EQUAL(feeRate.GetFee(1e3), 1000);
BOOST_CHECK_EQUAL(feeRate.GetFee(9e3), 9000);
+ // Relay fees must always just return the arg
+ BOOST_CHECK_EQUAL(feeRate.GetRelayFee(0), 0);
+ BOOST_CHECK_EQUAL(feeRate.GetRelayFee(1), 1);
+ BOOST_CHECK_EQUAL(feeRate.GetRelayFee(121), 121);
+ BOOST_CHECK_EQUAL(feeRate.GetRelayFee(999), 999);
+ BOOST_CHECK_EQUAL(feeRate.GetRelayFee(1e3), 1e3);
+ BOOST_CHECK_EQUAL(feeRate.GetRelayFee(9e3), 9e3);
+
feeRate = CFeeRate(-1000);
// Must always just return -1 * arg
BOOST_CHECK_EQUAL(feeRate.GetFee(0), 0);