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: run rpc-generateblock.py even with wallet disabled #23558

Merged

Conversation

DariusParvin
Copy link
Contributor

@DariusParvin DariusParvin commented Nov 19, 2021

Run rpc_generateblock.py test even when the wallet is disabled, as discussed in #20078.

This PR picked up from where #20808 left off, started by @nginocchio. Since then, there have been many improvements to MiniWallet, making this PR more straightforward.

L23 makes use of MiniWallet.rescan_utxos() to add the pre-mined block utxos (when self.setup_clean_chain is not set to True), rather than generating new blocks during the test.

@DariusParvin DariusParvin force-pushed the rpc-generateblock-to-miniwallet branch from a8acd3e to 42ae0cd Compare November 19, 2021 20:59
@DrahtBot DrahtBot added the Tests label Nov 19, 2021
@DariusParvin DariusParvin force-pushed the rpc-generateblock-to-miniwallet branch from 78a72ab to f38c11b Compare November 20, 2021 07:52
@DariusParvin DariusParvin force-pushed the rpc-generateblock-to-miniwallet branch from a82325d to d94164b Compare November 20, 2021 08:02
@maflcko
Copy link
Member

maflcko commented Nov 20, 2021

Thanks, Concept ACK.

@DariusParvin DariusParvin force-pushed the rpc-generateblock-to-miniwallet branch from 7686983 to 7c24756 Compare November 20, 2021 08:31
@DariusParvin DariusParvin force-pushed the rpc-generateblock-to-miniwallet branch from 7c24756 to 1e86ff7 Compare November 20, 2021 08:58
Copy link
Contributor

@mjdietzx mjdietzx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested ACK 1e86ff7

@maflcko maflcko merged commit 3688313 into bitcoin:master Nov 21, 2021
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Nov 21, 2021
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Nov 23, 2021
@bitcoin bitcoin locked and limited conversation to collaborators Nov 21, 2022
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.

4 participants