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.

0 files changed, 0 insertions, 0 deletions