diff options
| author | MacroFake <falke.marco@gmail.com> | 2022-06-13 22:15:02 +0200 |
|---|---|---|
| committer | MacroFake <falke.marco@gmail.com> | 2022-06-13 22:15:07 +0200 |
| commit | a55606c3bdbfdf660b093bc2a618d537ffae7f26 (patch) | |
| tree | 727fbabeb3bc07697c62008bd40543c5bd98a3f4 /contrib/message-capture/message-capture-parser.py | |
| parent | b91055ea55977aa3967d2808c641b6add015bf70 (diff) | |
| parent | fa779de665227cf15a4056959e9753618cc013ac (diff) | |
| download | bitcoin-a55606c3bdbfdf660b093bc2a618d537ffae7f26.tar.xz | |
Merge bitcoin/bitcoin#25356: test: Remove MiniWallet mempool_valid option
fa779de665227cf15a4056959e9753618cc013ac test: Remove MiniWallet mempool_valid option (MacroFake)
Pull request description:
It seems an unnecessary burden to force MiniWallet call-sites to figure out for each tx whether it is mempool valid or not. The result will only be used for internal sanity checks. So remove the option:
* Replace the vsize sanity check with a call to `get_vsize()`.
* Drop the fee check. Hopefully any future bug here will be caught by code-review or otherwise.
ACKs for top commit:
theStack:
Code-review ACK fa779de665227cf15a4056959e9753618cc013ac
Tree-SHA512: df44a0e116a0b6b15389c80038f9b45e17f186d0e3d7b0925e367fd2cbbcab9a7a6f7add41859ffb5603885f304233a5d28fbd57a4008ebdfe5edbe83bb1d7c3
Diffstat (limited to 'contrib/message-capture/message-capture-parser.py')
0 files changed, 0 insertions, 0 deletions