Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Aug 25, 2020

This introduces a minimalistic test wallet, which can be used as a drop in replacement for the Bitcoin Core wallet to create dummy transactions with a given fee rate.

@fanquake fanquake added the Tests label Aug 25, 2020
@maflcko
Copy link
Member Author

maflcko commented Aug 25, 2020

Requested by @jnewbery in #19315 (comment)

Copy link
Contributor

@jnewbery jnewbery left a comment

Choose a reason for hiding this comment

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

Very cool. Strong concept ACK if this allows us to build transactions outside the wallet more easily.

@maflcko maflcko force-pushed the 2008-testMiniWallet branch from faa2aa2 to fa8b892 Compare August 25, 2020 12:39
@bitcoin bitcoin deleted a comment from StanislavKlimov95 Aug 25, 2020
@maflcko maflcko force-pushed the 2008-testMiniWallet branch from fa8b892 to faf4f21 Compare August 25, 2020 12:58
@bitcoin bitcoin deleted a comment from StanislavKlimov95 Aug 25, 2020
@maflcko
Copy link
Member Author

maflcko commented Aug 25, 2020

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

Concept ACK, tested ACK, very nice.

A few style nits below, feel free to ignore.

@maflcko maflcko force-pushed the 2008-testMiniWallet branch from faf4f21 to fa170e5 Compare August 25, 2020 14:22
@amitiuttarwar
Copy link
Contributor

concept ACK!! very nice :)

@theStack
Copy link
Contributor

Concept ACK

@maflcko
Copy link
Member Author

maflcko commented Sep 7, 2020

Would be good to make progress here, so that the mockwallet can be used in new tests

Copy link
Contributor

@jnewbery jnewbery left a comment

Choose a reason for hiding this comment

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

This is great. I've left lots of small style comments. Nothing blocking though.

@maflcko maflcko force-pushed the 2008-testMiniWallet branch from fa170e5 to fae0de3 Compare September 7, 2020 12:46
@maflcko
Copy link
Member Author

maflcko commented Sep 7, 2020

Force pushed second commit to address @jnewbery's feedback

@jnewbery
Copy link
Contributor

jnewbery commented Sep 7, 2020

utACK fae0de3

@maflcko maflcko force-pushed the 2008-testMiniWallet branch from fae0de3 to fa188c9 Compare September 7, 2020 13:06
@jnewbery
Copy link
Contributor

jnewbery commented Sep 7, 2020

utACK fa188c9

@maflcko maflcko merged commit 564e1ab into bitcoin:master Sep 9, 2020
@maflcko maflcko deleted the 2008-testMiniWallet branch September 9, 2020 07:14
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Sep 9, 2020
fa188c9 test: Use MiniWalet in p2p_feefilter (MarcoFalke)
fa39c62 test: inline hashToHex (MarcoFalke)

Pull request description:

  This introduces a minimalistic test wallet, which can be used as a drop in replacement for the Bitcoin Core wallet to create dummy transactions with a given fee rate.

ACKs for top commit:
  jnewbery:
    utACK fa188c9

Tree-SHA512: 0aad9cb14eea4f0055bd6a47cc8c8f82a16941b152598c3bf1e083aae84cca4ffa23f0b854a362a68be1b917deba1b5ec7c0207b63b0805d747ba9a7d1d82efe
Fabcien pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Oct 11, 2021
Summary:
> This introduces a minimalistic test wallet, which can be used as a drop in replacement for the Bitcoin Core wallet to create dummy transactions with a given fee rate.

This is a backport of [[bitcoin/bitcoin#19800 | core#19800]]

It is squashed with the first bugfix commit of core#19922:
bitcoin/bitcoin@fa65a11

Test Plan: `ninja check-functional`

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

Differential Revision: https://reviews.bitcoinabc.org/D10199
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 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.

7 participants