Movatterモバイル変換


[0]ホーム

URL:


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

Sign up
Appearance settings

[3.12] gh-110088, gh-109878: Fix test_asyncio timeouts (#110092)#110098

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
vstinner merged 1 commit intopython:3.12fromvstinner:asyncio_timeout12
Oct 2, 2023

Conversation

@vstinner
Copy link
Member

@vstinnervstinner commentedSep 29, 2023
edited by bedevere-appbot
Loading

Fix test_asyncio timeouts: don't measure the maximum duration, a test should not measure a CI performance. Only measure the minimum duration when a task has a timeout or delay. Add CLOCK_RES to test_asyncio.utils.

(cherry picked from commitdb0a258)

Copy link
Contributor

@willingcwillingc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

This looks to be a nice clean-up and healthy improvement for asyncio tests. +1 from me. Thanks@vstinner

vstinner reacted with heart emoji
…10092)Fix test_asyncio timeouts: don't measure the maximum duration, a testshould not measure a CI performance. Only measure the minimumduration when a task has a timeout or delay. Add CLOCK_RES totest_asyncio.utils.(cherry picked from commitdb0a258)
@vstinner
Copy link
MemberAuthor

PR rebased on 3.12 to fix a merge conflict with merged commit150bd30.

@vstinnervstinnerenabled auto-merge (squash)October 2, 2023 17:22
@vstinnervstinner merged commit0745ab8 intopython:3.12Oct 2, 2023
@vstinnervstinner deleted the asyncio_timeout12 branchOctober 2, 2023 19:35
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@willingcwillingcwillingc approved these changes

@1st11st1Awaiting requested review from 1st11st1 is a code owner

@asvetlovasvetlovAwaiting requested review from asvetlovasvetlov is a code owner

@gvanrossumgvanrossumAwaiting requested review from gvanrossumgvanrossum is a code owner

@kumaraditya303kumaraditya303Awaiting requested review from kumaraditya303kumaraditya303 is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vstinner@willingc

[8]ページ先頭

©2009-2025 Movatter.jp