test: remove unused create_confirmed_utxos helper #25374

pull theStack wants to merge 1 commits into bitcoin:master from theStack:202206-test-remove_unused_create_confirmed_utxos_helper changing 1 files +0 −33
  1. theStack commented at 11:15 pm on June 14, 2022: member
    After more and more non-wallet tests have been converted to use MiniWallet (#25087, #24839, #24749 etc.), the create_confirmed_utxos helper is now not used anymore and can be removed. An alternative would be to create a MiniWallet version of create_confirmed_utxos, but it seems that it’s not worth it, considering that would be only two lines (calling MiniWallet’s send_self_transfer_multi with a subsequent generate call), see comment #24839 (review).
  2. test: remove unused `create_confirmed_utxos` helper
    Confirmed UTXOs in functional tests can simply be created by using
    MiniWallet's `send_self_transfer_multi` method with a subsequent
    `generate` call to mine a block.
    42b2fdfd5f
  3. DrahtBot added the label Tests on Jun 14, 2022
  4. MarcoFalke commented at 6:20 am on June 15, 2022: member
    cr ACK 42b2fdfd5f9d854fe05a248278fc309a6a9fa6bc
  5. MarcoFalke merged this on Jun 15, 2022
  6. MarcoFalke closed this on Jun 15, 2022

  7. theStack deleted the branch on Jun 15, 2022
  8. sidhujag referenced this in commit 1630aae6a4 on Jun 15, 2022
  9. DrahtBot locked this on Jun 15, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-23 00:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me