Skip to content

[3.7] bpo-35602: Make sure the transport is always closed in SelectorEventLoopUnixSockSendfileTests (GH-11338)#11358

Merged
miss-islington merged 1 commit intopython:3.7from
miss-islington:backport-d51324a-3.7
Dec 29, 2018
Merged

[3.7] bpo-35602: Make sure the transport is always closed in SelectorEventLoopUnixSockSendfileTests (GH-11338)#11358
miss-islington merged 1 commit intopython:3.7from
miss-islington:backport-d51324a-3.7

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Dec 29, 2018

There is a race condition in SelectorEventLoopUnixSockSendfileTests that causes the prepare() method return a non connected server protocol, making the cleanup() method skips the correct handling of the transport. This commit makes prepare() always return a connected server protocol that can always be cleaned up correctly.
(cherry picked from commit d51324a)

Co-authored-by: Pablo Galindo Pablogsal@gmail.com

https://bugs.python.org/issue35602

…oopUnixSockSendfileTests (pythonGH-11338)

There is a race condition in SelectorEventLoopUnixSockSendfileTests that causes the prepare() method return a non connected server protocol, making the cleanup() method skips the correct handling of the transport. This commit makes prepare() always return a connected server protocol that can always be cleaned up correctly.
(cherry picked from commit d51324a)

Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
@miss-islington
Copy link
Copy Markdown
Contributor Author

@pablogsal: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit fe06646 into python:3.7 Dec 29, 2018
@miss-islington miss-islington deleted the backport-d51324a-3.7 branch December 29, 2018 19:38
@miss-islington
Copy link
Copy Markdown
Contributor Author

@pablogsal: Status check is done, and it's a success ✅ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants