Test some transaction creation with non-empty fee estimator #18245

1. 
   instagibbs commented at 6:15 pm on March 2, 2020: member

   These tests give additional coverage to RPCs that use conf_target in a meaningful way that doesn’t just return min rates.

   Inspired by https://github.com/bitcoin/bitcoin/issues/18240

2. assert_fee_amount should round down f996ff8bbe
3. 0 < conf_target < 1008 for RPC 64eca91294
4. Test that wallet is using correct conf_target feerates when prescribed 31bdc33baa
5. 
   instagibbs commented at 6:18 pm on March 2, 2020: member

   Probably a good idea to stick this test somewhere else, but that would require either replication of this test(filling out the estimator?) or refactoring it to use it as a library.
6. DrahtBot added the label Tests on Mar 2, 2020
7. 
   instagibbs commented at 7:11 pm on March 2, 2020: member

   one build failing:

   0    assert_fee_amount(tx2_info["fee"], tx2_info["vsize"], med_fee)
   1  File "/home/travis/build/bitcoin/bitcoin/build/bitcoin-x86_64-pc-linux-gnu/test/functional/test_framework/util.py", line 39, in assert_fee_amount
   2    raise AssertionError("Fee of %s BTC too low! (Should be %s BTC)" % (str(fee), str(target_fee)))
   3AssertionError: Fee of 0.00008981 BTC too low! (Should be 0.00012702 BTC)
   

   Will investigate

8. 
   DrahtBot commented at 11:50 pm on March 20, 2020: member

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #13990 (Allow fee estimation to work with lower fees by ajtowns)

   If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

9. 
   instagibbs commented at 3:24 pm on August 10, 2020: member

   not taking this further, someone else can pick it up
10. instagibbs closed this on Aug 10, 2020

    ---

11. 
    jonatack commented at 4:52 pm on August 10, 2020: member

    @instagibbs I wrote/am writing conf_target tests as part of follow-ups to #11413 and will look at pulling these commits in.
12. DrahtBot locked this on Feb 15, 2022
13. adamjonas added the label Up for grabs on Aug 2, 2022

Contributors
instagibbs DrahtBot jonatack

Labels
Tests Up for grabs