Skip to content

bpo-37142: Make stream tests more robust again. - #13804

Merged
vstinner merged 1 commit into
python:masterfrom
asvetlov:hangs
Jun 4, 2019
Merged

bpo-37142: Make stream tests more robust again.#13804
vstinner merged 1 commit into
python:masterfrom
asvetlov:hangs

Conversation

@asvetlov

@asvetlovasvetlov commented Jun 4, 2019

Copy link
Copy Markdown
Contributor

Make test_stream_server_close() implementation following test_stream_server_abort().
Add explicit timeout for tests that can hang.

https://bugs.python.org/issue37142

 Make test_stream_server_close() implementation following test_stream_server_abort().
Add explicit timeout for tests that can hang.
@asvetlov

Copy link
Copy Markdown
ContributorAuthor

Sorry, I forgot apply changes done by #13790 to test_stream_server_close itself.

Also, the PR adds the explicit 1min timeout for tests to fail fast

@vstinner
vstinner merged commit 8d56109 into python:masterJun 4, 2019
@vstinner

Copy link
Copy Markdown
Member

LGTM. I merged your PR @asvetlov, thanks.

@asvetlov

Copy link
Copy Markdown
ContributorAuthor

Thanks, @vstinner

@asvetlov
asvetlov deleted the hangs branch June 4, 2019 13:45
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
Make test_stream_server_close() implementation following test_stream_server_abort().
Add explicit timeout for tests that can hang.
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.

4 participants

@asvetlov@vstinner@the-knights-who-say-ni@bedevere-bot