test: Fix unregister_all_during_call cleanup #18563

pull ryanofsky wants to merge 1 commits into bitcoin:master from ryanofsky:pr/valfix changing 1 files +2 −5
  1. ryanofsky commented at 2:18 pm on April 8, 2020: member

    Use TestingSetup fixture to fix unregister_all_during_call test not calling UnregisterBackgroundSignalScheduler, which could trigger an assert in RegisterBackgroundSignalScheduler when called in later tests

    Failure reported by fanquake #18551 (comment)

  2. Fix unregister_all_during_call cleanup
    Use TestingSetup fixture to fix unregister_all_during_call test not calling
    UnregisterBackgroundSignalScheduler, which could trigger an assert in
    RegisterBackgroundSignalScheduler when called in later tests
    
    Failure reported by fanquake <fanquake@gmail.com>
    https://github.com/bitcoin/bitcoin/pull/18551#issuecomment-610974251
    13d2a33537
  3. MarcoFalke commented at 2:26 pm on April 8, 2020: member
    ACK 13d2a33537a403ac47a989be92109d3214375b6a if appveyor unit tests pass
  4. fanquake added the label Tests on Apr 8, 2020
  5. MarcoFalke merged this on Apr 8, 2020
  6. MarcoFalke closed this on Apr 8, 2020

  7. deadalnix referenced this in commit 6f74431345 on Jun 20, 2020
  8. DrahtBot locked this on Feb 15, 2022

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-10-04 22:12 UTC

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