Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make AsyncResult behave more like in older versions in cross-thread usage #1743

Merged
merged 5 commits into from
Jan 12, 2021

Commits on Jan 11, 2021

  1. Add failing test for #1739

    jamadden committed Jan 11, 2021
    Configuration menu
    Copy the full SHA
    09f3ae7 View commit details
    Browse the repository at this point in the history
  2. Add initial fix for #1739; needs more testing and a change note.

    In addition to the two missing tests mentioned in test__event, I need to see if I can get a semaphore in this situation.
    jamadden committed Jan 11, 2021
    Configuration menu
    Copy the full SHA
    8243351 View commit details
    Browse the repository at this point in the history

Commits on Jan 12, 2021

  1. Configuration menu
    Copy the full SHA
    fb70b4a View commit details
    Browse the repository at this point in the history
  2. Skip test_cross_thread_use during leakchecks.

    The background thread is problematic.
    jamadden committed Jan 12, 2021
    Configuration menu
    Copy the full SHA
    ca2de79 View commit details
    Browse the repository at this point in the history
  3. Debug the test failure in TestSemaphoreMultiThread.test_dueling_threa…

    …ds_with_hub
    
    And fix. Leave a comment about what was discovered. It's somewhat surprising.
    jamadden committed Jan 12, 2021
    Configuration menu
    Copy the full SHA
    7eb9b59 View commit details
    Browse the repository at this point in the history