diff options
| author | Patrick Lodder <[email protected]> | 2021-08-16 23:00:43 +0200 |
|---|---|---|
| committer | GitHub <[email protected]> | 2021-08-16 23:00:43 +0200 |
| commit | 9de15dd687243d2b0c11228209e40c63282591c2 (patch) | |
| tree | bf1977b415cc973a80cbc0191163a1455706e2b2 /src/amount.cpp | |
| parent | Merge pull request #2456 from rnicoll/1.14.4-intellij (diff) | |
| parent | qa: fix bumpfee now that paytxfee works (diff) | |
| download | discoin-9de15dd687243d2b0c11228209e40c63282591c2.tar.xz discoin-9de15dd687243d2b0c11228209e40c63282591c2.zip | |
Merge pull request #2446 from patricklodder/1.14.4-mintxfee-tests
Implements, tests and assures wallet operator ease-of-sovereignty and making
sure that user-defined fee logic persists through versions in the future by
fixing and testing -paytxfee
1. There was an override of ANY user-defined value to CWallet::GetMinimumFee
- Former logic: always override any value with either -mintxfee or
-mintxrelayfee, whichever is highest
- Proposed logic in this pull request:
- if the user specifies a value, only override when it is lower than
-mintxfee or -mintxrelayfee - this works because we set any default
-mintxfee to be the same as -paytxfee, unless the user explicitly
sets a -mintxfee.
- if no value has been specified, use the rate from -mintxfee or
-mintxrelayfee, whichever is highest
2. Test that the interaction between the wallet parameters -paytxfee and
-mintxfee function as intended. This has to be done using rpc tests rather
than unit tests because it tests the actual parameters passed to the
executables.
3. Undoing the override exposed a misconfiguration in the bumpfee.py test,
where fees were explicitly set higher, yet ignored in subsequent bumps.
Diffstat (limited to 'src/amount.cpp')
0 files changed, 0 insertions, 0 deletions