-
Notifications
You must be signed in to change notification settings - Fork 37.7k
test: Mockwallet #19800
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: Mockwallet #19800
Conversation
Requested by @jnewbery in #19315 (comment) |
There was a problem hiding this 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.
faa2aa2
to
fa8b892
Compare
fa8b892
to
faf4f21
Compare
nowallet build passed: https://travis-ci.org/github/bitcoin/bitcoin/jobs/720986253#L3238 |
There was a problem hiding this 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.
faf4f21
to
fa170e5
Compare
concept ACK!! very nice :) |
Concept ACK |
Would be good to make progress here, so that the mockwallet can be used in new tests |
There was a problem hiding this 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.
fa170e5
to
fae0de3
Compare
Force pushed second commit to address @jnewbery's feedback |
utACK fae0de3 |
fae0de3
to
fa188c9
Compare
utACK fa188c9 |
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
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
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.