test: Run mempool_compatibility.py with MiniWallet #26640
pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2212-test-mini-wallet-mempool-🚌 changing 1 files +1 −7-
maflcko commented at 11:17 am on December 5, 2022: memberBy using the already existing miniwallet, the test can be run even when no wallet is compiled.
-
DrahtBot commented at 11:17 am on December 5, 2022: contributor
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
Reviews
See the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #20892 (tests: Run both descriptor and legacy tests within a single test invocation by achow101)
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.
-
DrahtBot added the label Tests on Dec 5, 2022
-
fanquake requested review from glozow on Dec 5, 2022
-
test: Run mempool_compatibility.py with MiniWallet fa43f60a0c
-
maflcko force-pushed on Dec 5, 2022
-
maflcko commented at 12:14 pm on December 5, 2022: memberIt doesn’t matter if the txs are descendants/ancestors of each other, so I just did that to make the diff even smaller.
-
glozow commented at 1:46 pm on December 5, 2022: member
ACK fa43f60a0c24880bf4802c74890644ae785bec7d
- AFAICT code in test does effectively the same thing as before
- test still runs when I disable wallet
- test still passes normally, fails if I delete certain line of
LoadMempool
orDumpMempool
-
achow101 commented at 5:37 pm on December 5, 2022: memberACK fa43f60a0c24880bf4802c74890644ae785bec7d
-
achow101 merged this on Dec 5, 2022
-
achow101 closed this on Dec 5, 2022
-
maflcko deleted the branch on Dec 6, 2022
-
bitcoin locked this on Dec 6, 2023
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-24 12:12 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-24 12:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me