diff options
| author | fanquake <fanquake@gmail.com> | 2024-01-31 09:59:41 +0000 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2024-01-31 09:59:50 +0000 |
| commit | 11b436a66af3ceaebb0f907878715f331516a0bc (patch) | |
| tree | 3cb69439c08e521e13345851abda430d447791b1 /contrib | |
| parent | cad2df24b396be403f13f372ec48ea14a90d7f06 (diff) | |
| parent | 26ad2aeb29dd0875e8509917ddaa586997e977d2 (diff) | |
| download | bitcoin-11b436a66af3ceaebb0f907878715f331516a0bc.tar.xz | |
Merge bitcoin/bitcoin#29343: test: fix wallet_import_rescan unrounded minimum amount
26ad2aeb29dd0875e8509917ddaa586997e977d2 test: fix wallet_import_rescan unrounded minimum amount (stickies-v)
Pull request description:
Addresses https://github.com/bitcoin/bitcoin/pull/29283#discussion_r1468842089.
Fixes a `JSONRPCException: Invalid amount (-3)` exception by ensuring the amount sent to `sendtoaddress` is rounded to 8 decimals.
See https://cirrus-ci.com/task/5562947183837184?logs=ci#L2559
Note: since `round` can also round down, `min_amount` is not _exactly_ guaranteed, but this is not a problem for the current usage. I've added a docstring to highlight this.
ACKs for top commit:
sr-gi:
ACK [26ad2ae](https://github.com/bitcoin/bitcoin/pull/29343/commits/26ad2aeb29dd0875e8509917ddaa586997e977d2)
Tree-SHA512: 82ce16447f30535f17fa73336f7e4f74639e33215a228294b9b8005b8050a760b90a3726de279cce98c7e439f09104172b74072be3a300dbd461bf0c3f54b954
Diffstat (limited to 'contrib')
0 files changed, 0 insertions, 0 deletions