bpo-33833: Fix ProactorSocketTransport AssertionError#7632
bpo-33833: Fix ProactorSocketTransport AssertionError#7632twisteroidambassador wants to merge 3 commits into
Conversation
|
Can we have a unittest for this? |
It may require a bit of advanced Mock() magic. The preconditions are somewhat complex, and the exception is raised in a done_callback, so usually it just goes to the log and nowhere else. |
Yeah, let's not add more complex mock tests. But if it's possible to recreate the preconditions with a functional test it would be enormously helpful.
We have |
|
Now we have a unit test and blurb. (I am reminded of this joke when writing the test: Patient: "Doctor, it hurts when I do this." |
https://bugs.python.org/issue33833