Skip to content

bpo-36889: Make StreamServer.close() tests more robust - #13790

Merged
1st1 merged 1 commit into
python:masterfrom
asvetlov:fix-hang
Jun 4, 2019
Merged

bpo-36889: Make StreamServer.close() tests more robust#13790
1st1 merged 1 commit into
python:masterfrom
asvetlov:fix-hang

Conversation

@asvetlov

@asvetlovasvetlov commented Jun 3, 2019

Copy link
Copy Markdown
Contributor

There are reports about hanging stream tests on buildbots.

This PR rewrites tests for StreamServer.close()
It uses more future objects for fine-grained synchronization between client and server parts of tests.

https://bugs.python.org/issue36889

@ambv

ambv commented Jun 3, 2019

Copy link
Copy Markdown
Contributor

@vstinner, look at this.

1st1
1st1 approved these changes Jun 4, 2019
@1st1
1st1 merged commit 35890ab into python:masterJun 4, 2019
@1st1
1st1 deleted the fix-hang branch June 4, 2019 11:37
@1st1
1st1 restored the fix-hang branch June 4, 2019 11:37
@1st1
1st1 deleted the fix-hang branch June 4, 2019 11:37
@1st1
1st1 restored the fix-hang branch June 4, 2019 11:37
@asvetlov
asvetlov deleted the fix-hang branch June 4, 2019 11:38
@asvetlov

Copy link
Copy Markdown
ContributorAuthor

Thanks, @1st1

@vstinner

Copy link
Copy Markdown
Member

This change fixed https://bugs.python.org/issue37142 if I understood correctly.

@asvetlov

Copy link
Copy Markdown
ContributorAuthor

Yes, you are right.
Thanks for the clarification

DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip newstestsTests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@asvetlov@ambv@vstinner@1st1@the-knights-who-say-ni@bedevere-bot