Uh oh!
There was an error while loading. Please reload this page.
GH-95736: fix IsolatedAsyncioTestCase to set event loop before calling setup functions - #95898
Conversation
Uh oh!
There was an error while loading. Please reload this page.
gvanrossum
commented
Aug 13, 2022
This is waiting for a resolution of #95900 (comment). |
Uh oh!
There was an error while loading. Please reload this page.
gvanrossum
left a comment
There was a problem hiding this comment.
Okay this is what I had in mind so LGTM from that perspective, but we need to get a better understanding of what Thomas is asking about the event loop policy.
gvanrossum
commented
Aug 16, 2022
So we've established that in tests deriving from It makes me wonder if the @kumaraditya303 Can you shorten the test per my remarks previously in this issue? I'd also like to invite @serhiy-storchaka to review this, since he did some work on |
Uh oh!
There was an error while loading. Please reload this page.
serhiy-storchaka
left a comment
There was a problem hiding this comment.
In general the solution LGTM, but I have a lot of questions about testing code.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Done, FTR the previous test was mostly copied from #95736 |
miss-islington
commented
Aug 16, 2022
Thanks @kumaraditya303 for the PR, and @gvanrossum for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11. |
…re calling setup functions (pythonGH-95898) (cherry picked from commit 9d51599) Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
bedevere-bot
commented
Aug 16, 2022
GH-96027 is a backport of this pull request to the 3.11 branch. |
bedevere-bot
commented
Aug 16, 2022
|
bedevere-bot
commented
Aug 16, 2022
|
bedevere-bot
commented
Aug 16, 2022
|
gvanrossum
commented
Aug 16, 2022
Urm, the WebAssembly buildbot failures are actually due to our test. We need to fix this before we can backport. Apparently by forcing a call to |
gvanrossum
commented
Aug 16, 2022
Rest of the discussion in the corresponding issue, starting at #95736 (comment) |
…er before calling setup functions (pythonGH-95898) (cherry picked from commit 9d51599) Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
…er before calling setup functions (pythonGH-95898) (cherry picked from commit 9d51599) Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com> Co-authored-by: Serhiy Storchaka storchaka@gmail.com
…er before calling setup functions (pythonGH-95898) (cherry picked from commit 9d51599) Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com> Co-authored-by: Serhiy Storchaka storchaka@gmail.com
…er before calling setup functions (pythonGH-95898) (cherry picked from commit 9d51599) Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com> Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
…er before calling setup functions (pythonGH-95898) (cherry picked from commit 9d51599) Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com> Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
IsolatedAsyncioTestCasedoes not callasyncio.set_event_loopbeforesetUpanymore,asyncio.Runner+PidFdChildWatcherleaves zombie processes #95736