Fix bumpfee rpc "errors" return value

pull/10450/head
Russell Yanofsky 8 years ago
parent 7e96ecf075
commit 1b6602fbe6

@ -2909,7 +2909,7 @@ UniValue bumpfee(const JSONRPCRequest& request)
UniValue errors(UniValue::VARR); UniValue errors(UniValue::VARR);
for (const std::string& err: feeBump.getErrors()) for (const std::string& err: feeBump.getErrors())
errors.push_back(err); errors.push_back(err);
result.push_back(errors); result.push_back(Pair("errors", errors));
return result; return result;
} }

@ -88,6 +88,7 @@ def test_simple_bumpfee_succeeds(rbf_node, peer_node, dest_address):
sync_mempools((rbf_node, peer_node)) sync_mempools((rbf_node, peer_node))
assert rbfid in rbf_node.getrawmempool() and rbfid in peer_node.getrawmempool() assert rbfid in rbf_node.getrawmempool() and rbfid in peer_node.getrawmempool()
bumped_tx = rbf_node.bumpfee(rbfid) bumped_tx = rbf_node.bumpfee(rbfid)
assert_equal(bumped_tx["errors"], [])
assert bumped_tx["fee"] - abs(rbftx["fee"]) > 0 assert bumped_tx["fee"] - abs(rbftx["fee"]) > 0
# check that bumped_tx propogates, original tx was evicted and has a wallet conflict # check that bumped_tx propogates, original tx was evicted and has a wallet conflict
sync_mempools((rbf_node, peer_node)) sync_mempools((rbf_node, peer_node))

Loading…
Cancel
Save