Retropex/bitcoin
1
0
Fork 0
mirror of https://github.com/Retropex/bitcoin.git synced 2025-06-02 15:32:34 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

[wallet] Return object from upgradewallet RPC

Browse Source
This commit is contained in:
Sishir Giri 2020-10-18 18:01:42 -07:00 committed by John Newbery
parent 867dbeba5f
commit 2ead31fb1b
2 changed files with 13 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/wallet/rpcwallet.cpp
View File

@ -4443,7 +4443,12 @@ static RPCHelpMan upgradewallet()
{ {
{"version", RPCArg::Type::NUM, /* default */ strprintf("%d", FEATURE_LATEST), "The version number to upgrade to. Default is the latest wallet version"} {"version", RPCArg::Type::NUM, /* default */ strprintf("%d", FEATURE_LATEST), "The version number to upgrade to. Default is the latest wallet version"}
}, },
RPCResults{}, RPCResult{
RPCResult::Type::OBJ, "", "",
{
{RPCResult::Type::STR, "error", /* optional */ true, "Error message (if there is one)"}
},
},
RPCExamples{ RPCExamples{
HelpExampleCli("upgradewallet", "169900") HelpExampleCli("upgradewallet", "169900")
+ HelpExampleRpc("upgradewallet", "169900") + HelpExampleRpc("upgradewallet", "169900")
@ -4468,7 +4473,11 @@ static RPCHelpMan upgradewallet()
if (!pwallet->UpgradeWallet(version, error, warnings)) { if (!pwallet->UpgradeWallet(version, error, warnings)) {
throw JSONRPCError(RPC_WALLET_ERROR, error.original); throw JSONRPCError(RPC_WALLET_ERROR, error.original);
} }
return error.original; UniValue obj(UniValue::VOBJ);
if (!error.empty()) {
obj.pushKV("error", error.original);
}
return obj;
}, },
}; };
} }

4
test/functional/wallet_upgradewallet.py
View File

@ -107,7 +107,7 @@ class UpgradeWalletTest(BitcoinTestFramework):
# calling upgradewallet without version arguments # calling upgradewallet without version arguments
# should return nothing if successful # should return nothing if successful
assert_equal(wallet.upgradewallet(), "") assert_equal(wallet.upgradewallet(), {})
new_version = wallet.getwalletinfo()["walletversion"] new_version = wallet.getwalletinfo()["walletversion"]
# upgraded wallet version should be greater than older one # upgraded wallet version should be greater than older one
assert_greater_than(new_version, old_version) assert_greater_than(new_version, old_version)
@ -130,7 +130,7 @@ class UpgradeWalletTest(BitcoinTestFramework):
assert_equal('hdseedid' in wallet.getwalletinfo(), False) assert_equal('hdseedid' in wallet.getwalletinfo(), False)
# calling upgradewallet with explicit version number # calling upgradewallet with explicit version number
# should return nothing if successful # should return nothing if successful
assert_equal(wallet.upgradewallet(169900), "") assert_equal(wallet.upgradewallet(169900), {})
new_version = wallet.getwalletinfo()["walletversion"] new_version = wallet.getwalletinfo()["walletversion"]
# upgraded wallet should have version 169900 # upgraded wallet should have version 169900
assert_equal(new_version, 169900) assert_equal(new_version, 169900)