[tests] Add node methods to test framework #10171

pull jnewbery wants to merge 3 commits into bitcoin:master from jnewbery:add_node_methods_to_test_framework changing 2 files +169 −125
  1. jnewbery commented at 3:21 am on April 8, 2017: member

    First step towards #10082:

    • moves initialize_chain() and initialize_chain_clean() in their entireties to be methods of BitcoinTestFramework. At the moment, these are only ever called by BitcoinTestFramework, so it makes sense to move them to be class methods.
    • add wrapper methods for start / stop nodes functions to BitcoinTestFramework class
    • [trivial] restructures ordering of members of BitcoinTestFramework for clarity.
  2. fanquake added the label Tests on Apr 8, 2017
  3. JeremyRubin commented at 3:15 pm on April 10, 2017: contributor
    looks reasonable. Have you been able to figure out why Travis is timing out?
  4. jnewbery force-pushed on Apr 10, 2017
  5. jnewbery commented at 8:56 pm on April 10, 2017: member

    Have you been able to figure out why Travis is timing out?

    Because I messed up the merge :( Should be fixed now.

  6. jnewbery force-pushed on Apr 11, 2017
  7. jimmysong commented at 9:06 pm on April 17, 2017: contributor

    ACK 2558b9a25047768b955863baa858f13e1f631ef1

    Makes groking test_framework simpler.

    Edit: tested locally.

  8. jimmysong approved
  9. move initialize_chain() and initialize_chain_clean() to be methods of BitcoinTestFramework b111324acd
  10. Add start and stop node methods to BitcoinTestFramework b7dd44c528
  11. jnewbery force-pushed on May 2, 2017
  12. jnewbery commented at 7:13 pm on May 2, 2017: member
    rebased
  13. sdaftuar commented at 8:37 pm on May 5, 2017: member

    ACK 6291f47

    Perhaps the commit message in the last commit should be fixed, though? “Reorganize BitcoinTestFramework class”

  14. Reorganize BitcoinTestFramework class 4550049903
  15. jnewbery force-pushed on May 5, 2017
  16. jnewbery commented at 10:06 pm on May 5, 2017: member
  17. sdaftuar commented at 9:17 am on May 6, 2017: member
    re-ACK 4550049
  18. MarcoFalke merged this on May 7, 2017
  19. MarcoFalke closed this on May 7, 2017

  20. MarcoFalke referenced this in commit fff72de5bf on May 7, 2017
  21. jnewbery deleted the branch on May 7, 2017
  22. PastaPastaPasta referenced this in commit bc4113d7ac on Jun 10, 2019
  23. PastaPastaPasta referenced this in commit 330d85ec57 on Jun 10, 2019
  24. PastaPastaPasta referenced this in commit ee6217cc0f on Jun 10, 2019
  25. PastaPastaPasta referenced this in commit bfeb05a29d on Jun 10, 2019
  26. PastaPastaPasta referenced this in commit 15064f3b8f on Jun 10, 2019
  27. PastaPastaPasta referenced this in commit 252bb8cfd0 on Jun 10, 2019
  28. PastaPastaPasta referenced this in commit 36d2b6bc9d on Jun 11, 2019
  29. PastaPastaPasta referenced this in commit b13fbf6854 on Jun 11, 2019
  30. PastaPastaPasta referenced this in commit 0eade2d5ee on Jun 15, 2019
  31. PastaPastaPasta referenced this in commit 815dbf8ea4 on Jun 19, 2019
  32. PastaPastaPasta referenced this in commit c7877690c7 on Jun 19, 2019
  33. PastaPastaPasta referenced this in commit c7c9875562 on Jun 19, 2019
  34. PastaPastaPasta referenced this in commit 5534e3899c on Jun 19, 2019
  35. PastaPastaPasta referenced this in commit cdc1095c0a on Jun 19, 2019
  36. PastaPastaPasta referenced this in commit 7773cf9a87 on Jun 19, 2019
  37. PastaPastaPasta referenced this in commit e459589292 on Jun 19, 2019
  38. PastaPastaPasta referenced this in commit 5892f88195 on Jun 20, 2019
  39. barrystyle referenced this in commit 6fd931d375 on Jan 22, 2020
  40. MarcoFalke locked this on Sep 8, 2021

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-12-19 03:12 UTC

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