mirror of
https://github.com/bitcoin/bitcoin.git
synced 2025-06-30 18:51:15 +02:00
rpc: update min package size error message in submitpackage
Currently, the only allowed package topology has a min size of 2. Update the error message to reflect that.
This commit is contained in:
@ -861,9 +861,9 @@ static RPCHelpMan submitpackage()
|
|||||||
[&](const RPCHelpMan& self, const JSONRPCRequest& request) -> UniValue
|
[&](const RPCHelpMan& self, const JSONRPCRequest& request) -> UniValue
|
||||||
{
|
{
|
||||||
const UniValue raw_transactions = request.params[0].get_array();
|
const UniValue raw_transactions = request.params[0].get_array();
|
||||||
if (raw_transactions.size() < 1 || raw_transactions.size() > MAX_PACKAGE_COUNT) {
|
if (raw_transactions.size() < 2 || raw_transactions.size() > MAX_PACKAGE_COUNT) {
|
||||||
throw JSONRPCError(RPC_INVALID_PARAMETER,
|
throw JSONRPCError(RPC_INVALID_PARAMETER,
|
||||||
"Array must contain between 1 and " + ToString(MAX_PACKAGE_COUNT) + " transactions.");
|
"Array must contain between 2 and " + ToString(MAX_PACKAGE_COUNT) + " transactions.");
|
||||||
}
|
}
|
||||||
|
|
||||||
std::vector<CTransactionRef> txns;
|
std::vector<CTransactionRef> txns;
|
||||||
|
@ -343,10 +343,10 @@ class RPCPackagesTest(BitcoinTestFramework):
|
|||||||
assert_raises_rpc_error(-25, "package topology disallowed", node.submitpackage, chain_hex)
|
assert_raises_rpc_error(-25, "package topology disallowed", node.submitpackage, chain_hex)
|
||||||
assert_equal(legacy_pool, node.getrawmempool())
|
assert_equal(legacy_pool, node.getrawmempool())
|
||||||
|
|
||||||
assert_raises_rpc_error(-8, f"Array must contain between 1 and {MAX_PACKAGE_COUNT} transactions.", node.submitpackage, [])
|
assert_raises_rpc_error(-8, f"Array must contain between 2 and {MAX_PACKAGE_COUNT} transactions.", node.submitpackage, [])
|
||||||
assert_raises_rpc_error(-25, "package topology disallowed", node.submitpackage, [chain_hex[0]] * 1)
|
assert_raises_rpc_error(-8, f"Array must contain between 2 and {MAX_PACKAGE_COUNT} transactions.", node.submitpackage, [chain_hex[0]] * 1)
|
||||||
assert_raises_rpc_error(
|
assert_raises_rpc_error(
|
||||||
-8, f"Array must contain between 1 and {MAX_PACKAGE_COUNT} transactions.",
|
-8, f"Array must contain between 2 and {MAX_PACKAGE_COUNT} transactions.",
|
||||||
node.submitpackage, [chain_hex[0]] * (MAX_PACKAGE_COUNT + 1)
|
node.submitpackage, [chain_hex[0]] * (MAX_PACKAGE_COUNT + 1)
|
||||||
)
|
)
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user