Skip to content

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 <tir.karthi@gmail.com>
@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