mirror of
https://github.com/bitcoin/bitcoin.git
synced 2026-01-19 14:53:43 +01:00
b104d44227test: Remove RPCOverloadWrapper (Ava Chow)4d32c19516test: Replace importpubkey (Ava Chow)fe838dd391test: Replace usage of addmultisigaddress (Ava Chow)d314207779test: Replace usage of importaddress (Ava Chow)fcc457573ftest: Replace importprivkey with wallet_importprivkey (Ava Chow)94c87bbbd0test: Remove unnecessary importprivkey from wallet_createwallet (Ava Chow) Pull request description: `RPCOverloadWrapper` implemented overloads for legacy wallet only RPCs so that the same function call could be used within tests for both legacy wallets and descriptor wallets. With legacy wallets now removed, there is no need to continue to have these overloads. For `importaddress`, `addmultisigaddress`, and `importpubkey`, the uses of these are converted to `importdescriptors`. For `importprivkey`, a new helper function `wallet_imporprivkey` is introduced that does what the overload did. This is mainly to reduce verbosity as `importprivkey` was more widely used throughout the tests. Some tests that used these RPCs are now also no longer relevant and have been removed. ACKs for top commit: Sjors: ACKb104d44227pablomartin4btc: cr ACKb104d44227rkrux: ACKb104d44227w0xlt: ACKb104d44227Tree-SHA512: ded2f73829e2ce28466d4a9738eb382783ad990daee5d1859dbc4d354e6f8eec0c483ed5ecb1287fe0dd24ac332065b733a30d71b126b841bd7cd49e9a094b6d