Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

gh-110088: Fix asyncio test_prompt_cancellation()#110157

Merged
vstinner merged 1 commit into
python:mainpython/cpython:mainfrom
vstinner:asyncio_timeouts2vstinner/cpython:asyncio_timeouts2Copy head branch name to clipboard
Sep 30, 2023
Merged

gh-110088: Fix asyncio test_prompt_cancellation()#110157
vstinner merged 1 commit into
python:mainpython/cpython:mainfrom
vstinner:asyncio_timeouts2vstinner/cpython:asyncio_timeouts2Copy head branch name to clipboard

Conversation

@vstinner

@vstinner vstinner commented Sep 30, 2023

Copy link
Copy Markdown
Member

Don't measure the CI performance: don't test the maximum elapsed time. The check failed on a slow CI.

Don't measure the CI performance: don't test the maximum elapsed
time. The check failed on a slow CI.
@vstinner

Copy link
Copy Markdown
Member Author

The test failed on GHA macOS:

FAIL: test_prompt_cancellation (test.test_asyncio.test_events.KqueueEventLoopTests.test_prompt_cancellation)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/runner/work/cpython/cpython/Lib/test/test_asyncio/test_events.py", line 1701, in test_prompt_cancellation
    self.assertLess(elapsed, 0.1)
AssertionError: 0.15447740299987345 not less than 0.1

build: https://github.com/python/cpython/actions/runs/6364342183/job/17280567179?pr=110148

@vstinner vstinner merged commit c62b49e into python:main Sep 30, 2023
@vstinner vstinner deleted the asyncio_timeouts2 branch September 30, 2023 20:40
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 30, 2023
)

Don't measure the CI performance: don't test the maximum elapsed
time. The check failed on a slow CI.
(cherry picked from commit c62b49e)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-app

bedevere-app Bot commented Sep 30, 2023

Copy link
Copy Markdown

GH-110158 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Sep 30, 2023
@bedevere-app

bedevere-app Bot commented Sep 30, 2023

Copy link
Copy Markdown

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

@bedevere-app bedevere-app Bot removed the needs backport to 3.11 only security fixes label Sep 30, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 30, 2023
)

Don't measure the CI performance: don't test the maximum elapsed
time. The check failed on a slow CI.
(cherry picked from commit c62b49e)

Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner added a commit that referenced this pull request Sep 30, 2023
…110159)

gh-110088: Fix asyncio test_prompt_cancellation() (GH-110157)

Don't measure the CI performance: don't test the maximum elapsed
time. The check failed on a slow CI.
(cherry picked from commit c62b49e)

Co-authored-by: Victor Stinner <vstinner@python.org>
Yhg1s pushed a commit that referenced this pull request Oct 2, 2023
…110158)

gh-110088: Fix asyncio test_prompt_cancellation() (GH-110157)

Don't measure the CI performance: don't test the maximum elapsed
time. The check failed on a slow CI.
(cherry picked from commit c62b49e)

Co-authored-by: Victor Stinner <vstinner@python.org>
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
Don't measure the CI performance: don't test the maximum elapsed
time. The check failed on a slow CI.
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 topic-asyncio

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.