-
Notifications
You must be signed in to change notification settings - Fork 37.7k
qa: Add further tests to wallet_balance #15758
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
Conversation
fa61f27
to
fabc1c6
Compare
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
Unless there are objections, this will be merged tomorrow |
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.
looks good. A few comments inline.
fa8d8eb
to
fa79a78
Compare
utACK fa79a78 |
fa79a78 test: Add reorg test to wallet_balance (MarcoFalke) fad03cd test: Check that wallet txs not in the mempool are untrusted (MarcoFalke) fa19531 test: Add getunconfirmedbalance test with conflicts (MarcoFalke) fa464e8 test: Add wallet_balance test for watchonly (MarcoFalke) Pull request description: Second commit can be reviewed with `--ignore-all-space` ACKs for commit fa79a7: jnewbery: utACK fa79a78 Tree-SHA512: ec4919a3c93b6dcb35d58e7c65bdffe7f4c8cb87b9287f3679631c1823ef5bd72789f233def94e60c1ab332711601751645566f5997ce250af55b328ed60e917
Github-Pull: bitcoin#15758 Rebased-From: fad03cd
Github-Pull: bitcoin#15758 Rebased-From: fa79a78
Summary: bitcoin/bitcoin@fa464e8 --- Partial backport of Core [[bitcoin/bitcoin#15758 | PR15758]] Test Plan: ./test/functional/test_runner.py wallet_balance Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Subscribers: deadalnix Differential Revision: https://reviews.bitcoinabc.org/D6440
Summary: bitcoin/bitcoin@fa19531 --- Depends on D6440 Partial backport of Core [[bitcoin/bitcoin#15758 | PR15758]] Test Plan: test_runner.py wallet_balance Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D6441
…ntrusted Summary: bitcoin/bitcoin@fad03cd --- Depends on D6441 Partial backport of Core [[bitcoin/bitcoin#15758 | PR15758]] Test Plan: test_runner.py wallet_balance Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D6442
Summary: bitcoin/bitcoin@fa79a78 --- Depends on D6442 Concludes backport of Core [[bitcoin/bitcoin#15758 | PR15758]] Test Plan: test_runner.py wallet_balance Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D6443
Second commit can be reviewed with
--ignore-all-space