Skip to content

bpo-26704: Add test for double patching instance methods#11085

Merged
cjw296 merged 1 commit intopython:masterfrom
asottile:bpo-26704
Dec 12, 2018
Merged

bpo-26704: Add test for double patching instance methods#11085
cjw296 merged 1 commit intopython:masterfrom
asottile:bpo-26704

Conversation

@asottile
Copy link
Copy Markdown
Contributor

@asottile asottile commented Dec 10, 2018

(cannot be backported past python3.7 as the bugfix was not backported to python3.6)

https://bugs.python.org/issue26704

Copy link
Copy Markdown
Member

@tirkarthi tirkarthi left a comment

Choose a reason for hiding this comment

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

LGTM. I will wait for someone else to look at this and also the patch in bpo to see if I am missing anything. Thanks for the PR 👍

Comment thread Lib/unittest/test/testmock/testwith.py Outdated
Comment thread Misc/NEWS.d/next/Tests/2018-12-10-13-18-37.bpo-26704.DBAN4c.rst Outdated
@asottile
Copy link
Copy Markdown
Contributor Author

Ahhhhh this would have been a perfect use of github's new "suggested changes" feature 😆. I'll get to these when I'm at a computer instead :)

@asottile
Copy link
Copy Markdown
Contributor Author

fixed!

@cjw296
Copy link
Copy Markdown
Contributor

cjw296 commented Dec 12, 2018

Nice work!

@cjw296 cjw296 merged commit 5a718e9 into python:master Dec 12, 2018
@bedevere-bot
Copy link
Copy Markdown

@cjw296: Please replace # with GH- in the commit message next time. Thanks!

@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @asottile for the PR, and @cjw296 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 12, 2018
(cherry picked from commit 5a718e9)

Co-authored-by: Anthony Sottile <asottile@umich.edu>
@bedevere-bot
Copy link
Copy Markdown

GH-11126 is a backport of this pull request to the 3.7 branch.

@asottile asottile deleted the bpo-26704 branch December 12, 2018 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants