gh-149879: Fix test_ssl on Cygwin#150419
Conversation
|
I'm not sure about the |
|
@picnixz: I updated the PR to only ignore |
There was a problem hiding this comment.
Thanks. I wish we had a more cleaner way to know what to ignore in general. Can you add the references (as a comment on the PR) where you found about it? or is an empiric finding? No need to add it inside the sources but just so that we know where to look at in the future...
Alternatively, you can maybe add a regression test to exercise the error code?
Without this change,
I don't see an existing test for |
|
Merged. Thanks for the review @picnixz. |
|
With this change, |
|
test_asyncio.test_sock_lowlevel: test_sock_client_racing() failed. It's unrelated to this change. |
Uh oh!
There was an error while loading. Please reload this page.