From 1fc3a8e8e7ae4698ac5cd5292a7e7e37097d37ce Mon Sep 17 00:00:00 2001 From: pablomartin4btc Date: Tue, 22 Jul 2025 12:56:17 -0300 Subject: [PATCH] rpc, test: Add EnsureUniqueWalletName tests Co-authored-by: stickies-v --- src/wallet/test/CMakeLists.txt | 1 + src/wallet/test/wallet_rpc_tests.cpp | 41 ++++++++++++++++++++++++++++ 2 files changed, 42 insertions(+) create mode 100644 src/wallet/test/wallet_rpc_tests.cpp diff --git a/src/wallet/test/CMakeLists.txt b/src/wallet/test/CMakeLists.txt index 8564eface55..524c7218f4e 100644 --- a/src/wallet/test/CMakeLists.txt +++ b/src/wallet/test/CMakeLists.txt @@ -19,6 +19,7 @@ target_sources(test_bitcoin scriptpubkeyman_tests.cpp spend_tests.cpp wallet_crypto_tests.cpp + wallet_rpc_tests.cpp wallet_tests.cpp wallet_transaction_tests.cpp walletdb_tests.cpp diff --git a/src/wallet/test/wallet_rpc_tests.cpp b/src/wallet/test/wallet_rpc_tests.cpp new file mode 100644 index 00000000000..fb33d3668da --- /dev/null +++ b/src/wallet/test/wallet_rpc_tests.cpp @@ -0,0 +1,41 @@ +// Copyright (c) 2025-present The Bitcoin Core developers +// Distributed under the MIT software license, see the accompanying +// file COPYING or http://www.opensource.org/licenses/mit-license.php. + +#include +#include +#include +#include + +#include + +#include +#include + +namespace wallet { +static std::string TestWalletName(const std::string& endpoint, std::optional parameter = std::nullopt) +{ + JSONRPCRequest req; + req.URI = endpoint; + return EnsureUniqueWalletName(req, parameter ? &*parameter : nullptr); +} + +BOOST_FIXTURE_TEST_SUITE(wallet_rpc_tests, BasicTestingSetup) + +BOOST_AUTO_TEST_CASE(ensure_unique_wallet_name) +{ + // EnsureUniqueWalletName should only return if exactly one unique wallet name is provided + BOOST_CHECK_EQUAL(TestWalletName("/wallet/foo"), "foo"); + BOOST_CHECK_EQUAL(TestWalletName("/wallet/foo", "foo"), "foo"); + BOOST_CHECK_EQUAL(TestWalletName("/", "foo"), "foo"); + BOOST_CHECK_EQUAL(TestWalletName("/bar", "foo"), "foo"); + + BOOST_CHECK_THROW(TestWalletName("/"), UniValue); + BOOST_CHECK_THROW(TestWalletName("/foo"), UniValue); + BOOST_CHECK_THROW(TestWalletName("/wallet/foo", "bar"), UniValue); + BOOST_CHECK_THROW(TestWalletName("/wallet/foo", "foobar"), UniValue); + BOOST_CHECK_THROW(TestWalletName("/wallet/foobar", "foo"), UniValue); +} + +BOOST_AUTO_TEST_SUITE_END() +} // namespace wallet