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

gh-98703: add tests for closing _ProactorSocketTransport with proactor event loop #98730

Merged
merged 2 commits into from Oct 27, 2022

Conversation

kumaraditya303
Copy link
Contributor

@kumaraditya303 kumaraditya303 commented Oct 26, 2022

@bedevere-bot bedevere-bot added awaiting review tests Tests in the Lib/test dir labels Oct 26, 2022
@kumaraditya303 kumaraditya303 changed the title add tests for closing asyncio.StreamWriter with proactor event loop add tests for closing _ProactorSocketTransport with proactor event loop Oct 26, 2022
@Fidget-Spinner Fidget-Spinner changed the title add tests for closing _ProactorSocketTransport with proactor event loop gh-98703: add tests for closing _ProactorSocketTransport with proactor event loop Oct 26, 2022
@Fidget-Spinner
Copy link
Member

Fidget-Spinner commented Oct 26, 2022

Skip news because it shares the same news item as #98704.

@kumaraditya303 kumaraditya303 marked this pull request as ready for review Oct 27, 2022
@Fidget-Spinner
Copy link
Member

Fidget-Spinner commented Oct 27, 2022

These are great tests. It seems that they failed on every platform prior to the fix :)!

@Fidget-Spinner Fidget-Spinner merged commit 96ae80f into python:main Oct 27, 2022
13 checks passed
@miss-islington
Copy link
Contributor

miss-islington commented Oct 27, 2022

Thanks @kumaraditya303 for the PR, and @Fidget-Spinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒🤖

@bedevere-bot
Copy link

bedevere-bot commented Oct 27, 2022

GH-98755 is a backport of this pull request to the 3.11 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 27, 2022
… proactor event loop (pythonGH-98730)

(cherry picked from commit 96ae80f)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
@bedevere-bot
Copy link

bedevere-bot commented Oct 27, 2022

GH-98756 is a backport of this pull request to the 3.10 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 27, 2022
… proactor event loop (pythonGH-98730)

(cherry picked from commit 96ae80f)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Oct 27, 2022
…tor event loop (GH-98730)

(cherry picked from commit 96ae80f)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Oct 27, 2022
…tor event loop (GH-98730)

(cherry picked from commit 96ae80f)

Co-authored-by: Kumar Aditya <59607654+kumaraditya303@users.noreply.github.com>
@kumaraditya303 kumaraditya303 deleted the proactor-test branch Oct 27, 2022
gvanrossum pushed a commit to gvanrossum/cpython that referenced this pull request Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
expert-asyncio skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants