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

[3.10] bpo-45678: Add more singledispatchmethod tests (GH-29412)#29424

Merged
ambv merged 1 commit into
python:3.10python/cpython:3.10from
miss-islington:backport-32f55d1-3.10miss-islington/cpython:backport-32f55d1-3.10Copy head branch name to clipboard
Nov 5, 2021
Merged

[3.10] bpo-45678: Add more singledispatchmethod tests (GH-29412)#29424
ambv merged 1 commit into
python:3.10python/cpython:3.10from
miss-islington:backport-32f55d1-3.10miss-islington/cpython:backport-32f55d1-3.10Copy head branch name to clipboard

Conversation

@miss-islington

@miss-islington miss-islington commented Nov 5, 2021

Copy link
Copy Markdown
Contributor

In order to fix a bug in the 3.9 branch in GH-29394, more tests were added to
test_functools.py to ensure that singledispatchmethod still correctly
wrapped a target method, even if the target method had already been wrapped by
multiple other decorators. This PR brings the new tests into the 3.11 and 3.10
branches as well.
(cherry picked from commit 32f55d1)

Co-authored-by: Alex Waygood Alex.Waygood@Gmail.com

https://bugs.python.org/issue45678

In order to fix a bug in the 3.9 branch in pythonGH-29394, more tests were added to
``test_functools.py`` to ensure that ``singledispatchmethod`` still correctly
wrapped a target method, even if the target method had already been wrapped by
 multiple other decorators. This PR brings the new tests into the 3.11 and 3.10
branches as well.
(cherry picked from commit 32f55d1)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@bedevere-bot bedevere-bot added the tests Tests in the Lib/test dir label Nov 5, 2021
@ambv ambv merged commit bcb236c into python:3.10 Nov 5, 2021
@miss-islington miss-islington deleted the backport-32f55d1-3.10 branch November 5, 2021 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

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