diff options
author | Jon Atack <jon@atack.com> | 2020-11-09 15:00:31 +0100 |
---|---|---|
committer | Jon Atack <jon@atack.com> | 2020-11-11 15:56:01 +0100 |
commit | 410e471fa42d3db04e8879c71f8c824dcc151a83 (patch) | |
tree | 640921f4d4e7802571c06f1803264197a793179c | |
parent | a0d495747320c79b27a83c216dcc526ac8df8f24 (diff) | |
download | bitcoin-410e471fa42d3db04e8879c71f8c824dcc151a83.tar.xz |
wallet: remove redundant bumpfee fee_rate checks
SetFeeEstimateMode() handles these checks now and provides a more actionable
error message.
-rw-r--r-- | src/wallet/rpcwallet.cpp | 12 | ||||
-rwxr-xr-x | test/functional/wallet_bumpfee.py | 2 |
2 files changed, 1 insertions, 13 deletions
diff --git a/src/wallet/rpcwallet.cpp b/src/wallet/rpcwallet.cpp index 238d37ae98..73aed65439 100644 --- a/src/wallet/rpcwallet.cpp +++ b/src/wallet/rpcwallet.cpp @@ -3474,18 +3474,6 @@ static RPCHelpMan bumpfee_helper(std::string method_name) auto conf_target = options.exists("confTarget") ? options["confTarget"] : options["conf_target"]; - if (!conf_target.isNull()) { - if (options.exists("fee_rate")) { - throw JSONRPCError(RPC_INVALID_PARAMETER, "Cannot specify both conf_target and fee_rate. Please provide either a confirmation target in blocks for automatic fee estimation, or an explicit fee rate."); - } - } else if (options.exists("fee_rate")) { - CFeeRate fee_rate(AmountFromValue(options["fee_rate"])); - if (fee_rate <= CFeeRate(0)) { - throw JSONRPCError(RPC_INVALID_PARAMETER, strprintf("Invalid fee_rate %s (must be greater than 0)", fee_rate.ToString())); - } - coin_control.m_feerate = fee_rate; - } - if (options.exists("replaceable")) { coin_control.m_signal_bip125_rbf = options["replaceable"].get_bool(); } diff --git a/test/functional/wallet_bumpfee.py b/test/functional/wallet_bumpfee.py index 5f76c78198..b9726ccfc3 100755 --- a/test/functional/wallet_bumpfee.py +++ b/test/functional/wallet_bumpfee.py @@ -111,7 +111,7 @@ class BumpFeeTest(BitcoinTestFramework): rbf_node.bumpfee, rbfid, {"fee_rate": INSUFFICIENT}) self.log.info("Test invalid fee rate settings") - assert_raises_rpc_error(-8, "Invalid fee_rate 0.00000000 BTC/kB (must be greater than 0)", + assert_raises_rpc_error(-8, "Insufficient total fee 0.00, must be at least 0.00001704 (oldFee 0.00000999 + incrementalFee 0.00000705)", rbf_node.bumpfee, rbfid, {"fee_rate": 0}) assert_raises_rpc_error(-4, "Specified or calculated fee 0.141 is too high (cannot be higher than -maxtxfee 0.10", rbf_node.bumpfee, rbfid, {"fee_rate": TOO_HIGH}) |