diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2021-08-19 19:16:04 +0200 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2021-08-19 19:16:09 +0200 |
| commit | 92f3a4b4d0531976c9484f5b5075487e0925b619 (patch) | |
| tree | d5d25c0c923d6b8737cb360c105380193331bc8c /.cirrus.yml | |
| parent | b6a8e68b4e11be3aa5ceff8c483594f8e576faaf (diff) | |
| parent | 8dcbbbea6486e9ab7d5e7397b82585141f9910bf (diff) | |
| download | bitcoin-92f3a4b4d0531976c9484f5b5075487e0925b619.tar.xz | |
Merge bitcoin/bitcoin#22742: test: Use proper target in do_fund_send
8dcbbbea6486e9ab7d5e7397b82585141f9910bf test: fix bug in 22686 (S3RK)
Pull request description:
It seems there is a bug in the test in #22686, the code behaviour itself looks correct.
Instead of verifying the scenario from #22670 with both `upper_bound` and `lower_bound` for the transaction amount, the tests verified `lower_bound` two times. This fix is to properly use function parameter instead of a variable from the scope. The test still passes with both values, so no code changes are required.
ACKs for top commit:
achow101:
ACK 8dcbbbea6486e9ab7d5e7397b82585141f9910bf
Tree-SHA512: 82837b2e00bd075a7b6b7a31031f4d3ba1ab4bd5967e90488f527fcc618aeb3945d6ae3ed66b9eb11616dda3ef376c5001559d2a7a79a759fc6800358bf52537
Diffstat (limited to '.cirrus.yml')
0 files changed, 0 insertions, 0 deletions