[tests] Remove accounts from wallet_importprunedfunds.py #13138

pull jnewbery wants to merge 1 commits into bitcoin:master from jnewbery:remove_account_wallet_importprunedfunds changing 1 files +9 −16
  1. jnewbery commented at 7:25 PM on May 1, 2018: member

    This was split from #13075 to not block review/merge of that PR.

  2. jnewbery renamed this:
    [tests] Remove accounts from wallet_importprunedfunds.py
    [Do not merge] [tests] Remove accounts from wallet_importprunedfunds.py
    on May 1, 2018
  3. fanquake added the label Tests on May 1, 2018
  4. jnewbery cross-referenced this on Jun 11, 2018 from issue Remove wallet 'account' API by jnewbery
  5. MarcoFalke commented at 3:40 PM on June 18, 2018: member

    @jnewbery Mind doing a rebase on master, since the last travis run (https://travis-ci.org/bitcoin/bitcoin/builds/373612454) was about two months ago.

  6. [tests] Remove accounts from wallet_importprunedfunds.py 38040c34e1
  7. jnewbery force-pushed on Jun 18, 2018
  8. jnewbery commented at 7:48 PM on June 18, 2018: member

    Rebased on master. I can no longer repro the OOM locally (tried running the test 100 times), and Travis passes. @MarcoFalke - can you try running this locally?

    I'm removing the [Do not merge] tag, since if this runs reliably it's ready for merge.

  9. jnewbery renamed this:
    [Do not merge] [tests] Remove accounts from wallet_importprunedfunds.py
    [tests] Remove accounts from wallet_importprunedfunds.py
    on Jun 18, 2018
  10. MarcoFalke commented at 8:14 PM on June 18, 2018: member

    @jnewbery Good to hear your tested ack on 0882406 ;)

  11. MarcoFalke commented at 8:19 PM on June 18, 2018: member

    Concept ACK 38040c34e1e71ee4b6b56a018f742c8b03546b75

    Going to take a closer look later

  12. MarcoFalke commented at 5:08 PM on July 14, 2018: member

    utACK 38040c34e1

  13. MarcoFalke merged this on Jul 14, 2018
  14. MarcoFalke closed this on Jul 14, 2018
  15. MarcoFalke referenced this in commit 8803c9132a on Jul 14, 2018
  16. Bushstar cross-referenced this on Jul 16, 2018 from issue commits from bitcoin/master by Bushstar
  17. jasonbcox referenced this in commit 57b370dd0d on Dec 6, 2019
  18. Fuzzbawls cross-referenced this on Jun 25, 2020 from issue [Tracking] Remove wallet 'account' API by Fuzzbawls
  19. PastaPastaPasta referenced this in commit ffe575a9fd on Jul 19, 2020
  20. PastaPastaPasta referenced this in commit 5a1ca94063 on Jul 24, 2020
  21. PastaPastaPasta referenced this in commit f8fb77abc5 on Jul 27, 2020
  22. UdjinM6 referenced this in commit 24cc9b89a2 on Jul 27, 2020
  23. UdjinM6 referenced this in commit 72a56e3118 on Jul 27, 2020
  24. PastaPastaPasta referenced this in commit 3dad55fc4c on Jun 27, 2021
  25. PastaPastaPasta referenced this in commit 9ce89a7114 on Jun 28, 2021
  26. PastaPastaPasta referenced this in commit afd1189c69 on Jun 28, 2021
  27. PastaPastaPasta referenced this in commit d9eb5242e1 on Jun 28, 2021
  28. PastaPastaPasta referenced this in commit 526b4ede3a on Jun 28, 2021
  29. PastaPastaPasta referenced this in commit 15a65ab407 on Jun 29, 2021
  30. bitcoin locked this on Sep 8, 2021
Contributors
jnewberyMarcoFalke
Labels
Tests
Linked (view graph)
#12952 Remove wallet 'account' API#13075 [tests] Remove 'account' API from wallet functional tests#13078 importprunedfunds + removeprunedfunds + listtransactions can cause OOM
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: 2026-05-20 06:55 UTC