Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: use MiniWallet for rpc_scanblocks.py #26307

Merged

Conversation

theStack
Copy link
Contributor

This is a small follow-up for #23549 which introduced scanblocks. Since that RPC doesn't need the wallet, we can switch the functional test to use MiniWallet.

@theStack theStack force-pushed the 202210-test-use_MiniWallet_for_scanblocks branch from 298a02b to ae3626e Compare October 13, 2022 20:08
@DrahtBot DrahtBot added the Tests label Oct 13, 2022
@maflcko
Copy link
Member

maflcko commented Oct 14, 2022

review ACK ae3626e

@maflcko maflcko merged commit 3f1f5f6 into bitcoin:master Oct 14, 2022
@theStack theStack deleted the 202210-test-use_MiniWallet_for_scanblocks branch October 14, 2022 11:54
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 23, 2022
ae3626e test: use MiniWallet for rpc_scanblocks.py (Sebastian Falbesoner)

Pull request description:

  This is a small follow-up for bitcoin#23549 which introduced `scanblocks`. Since that RPC doesn't need the wallet, we can switch the functional test to use MiniWallet.

ACKs for top commit:
  MarcoFalke:
    review ACK ae3626e

Tree-SHA512: e0b0088103e059b29719299c58fd5173b1cff58cb73025a9a33ad493cd0ac50ba25a5f790e471c00a09b4dca80f3c011174ca4e0c8f34746c39831f5823dc8ba
@bitcoin bitcoin locked and limited conversation to collaborators Oct 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants