test: Run feature_cltv with MiniWallet #21754

1. 
   MarcoFalke commented at 3:18 pm on April 22, 2021: member

   Allows to run the test even with no wallet compiled in
2. MarcoFalke force-pushed on Apr 22, 2021
3. DrahtBot added the label Tests on Apr 22, 2021
4. MarcoFalke force-pushed on Apr 22, 2021
5. 
   practicalswift commented at 6:27 pm on April 22, 2021: contributor

   Concept ACK: more MiniWallet is better :)
6. 
   DrahtBot commented at 9:10 pm on April 22, 2021: member

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

   Conflicts

   No conflicts as of last run.

7. MarcoFalke force-pushed on Apr 23, 2021
8. test: Remove new_tx reference

   new_tx is a reference to tx. To avoid confusion and code-bloat, remove it

   fa5f938cfe
9. test: Hide tx rehash in helper

   This avoids code-bloat and accidentally forgetting to call it

   fa5591d38c
10. test: Run feature_cltv with MiniWallet fa066f1b66
11. MarcoFalke force-pushed on Apr 29, 2021
12. theStack approved
13. 
    theStack commented at 7:00 pm on May 4, 2021: member

    ACK fa066f1b66ab75e6c88e11922bbd085c9d3211de 💽 Nice to see MiniWallet used more and more! The raw_script type introduced will also make it easier to convert other tests to use the internal wallet.
14. MarcoFalke merged this on May 5, 2021
15. MarcoFalke closed this on May 5, 2021

    ---

16. MarcoFalke deleted the branch on May 5, 2021
17. sidhujag referenced this in commit f5a788ba3a on May 5, 2021
18. gwillen referenced this in commit ab1e80eb09 on Jun 1, 2022
19. DrahtBot locked this on Aug 18, 2022

Contributors
MarcoFalke practicalswift DrahtBot theStack

Labels
Tests