Skip to content

bpo-39915: Ensure await_args_list is updated according to the order in which coroutines were awaited #18924

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

Merged
merged 2 commits into from
Mar 11, 2020

Conversation

tirkarthi
Copy link
Member

@tirkarthi tirkarthi commented Mar 11, 2020

await_args_list should be updated with the arguments with which the mock was awaited instead of using call_args which has the last call object.

https://bugs.python.org/issue39915

@tirkarthi tirkarthi requested review from cjw296 and lisroach March 11, 2020 14:37
@tirkarthi tirkarthi changed the title bpo-39915: Ensure await_args_list is updated according to the order in which coroutines are awaited bpo-39915: Ensure await_args_list is updated according to the order in which coroutines were awaited Mar 11, 2020
@tirkarthi tirkarthi added needs backport to 3.8 type-bug An unexpected behavior, bug, or error labels Mar 11, 2020
Copy link
Contributor

@cjw296 cjw296 left a comment

Choose a reason for hiding this comment

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

LGTM!

@cjw296 cjw296 merged commit e553f20 into python:master Mar 11, 2020
@miss-islington
Copy link
Contributor

Thanks @tirkarthi for the PR, and @cjw296 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 11, 2020
…n which coroutines were awaited (pythonGH-18924)

Create call objects with awaited arguments instead of using call_args which has only last call value.
(cherry picked from commit e553f20)

Co-authored-by: Karthikeyan Singaravelan <[email protected]>
@bedevere-bot
Copy link

GH-18927 is a backport of this pull request to the 3.8 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants