gh-117482: Fix the Slot Wrapper Inheritance Tests by ericsnowcurrently · Pull Request #122248 · python/cpython · GitHub
Skip to content

gh-117482: Fix the Slot Wrapper Inheritance Tests - #122248

Merged
ericsnowcurrently merged 6 commits into
python:mainfrom
ericsnowcurrently:fix-slot-wrapper-inheritance-tests
Jul 29, 2024
Merged

ericsnowcurrently merged 6 commits into
python:mainfrom
ericsnowcurrently:fix-slot-wrapper-inheritance-tests

Conversation

@ericsnowcurrently

@ericsnowcurrently ericsnowcurrently commented Jul 24, 2024

Copy link
Copy Markdown
Member

The tests were only checking cases where the slot wrapper was present in the initial case. They were missing when the slot wrapper was added in the additional initializations. This fixes that.

I'll be backporting this to 3.13 and 3.12 manually.

@ericsnowcurrently

Copy link
Copy Markdown
Member Author

@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @ericsnowcurrently for commit 60ad88d 🤖

The command will test the builders whose names match following regular expression: iOS

The builders matched are:

  • iOS ARM64 Simulator PR

@ericsnowcurrently
ericsnowcurrently merged commit 490e0ad into python:main Jul 29, 2024
@ericsnowcurrently
ericsnowcurrently deleted the fix-slot-wrapper-inheritance-tests branch July 29, 2024 16:23
ericsnowcurrently added a commit that referenced this pull request Jul 29, 2024
The tests were only checking cases where the slot wrapper was present in the initial case.  They were missing when the slot wrapper was added in the additional initializations.  This fixes that.

(cherry-picked from commit 490e0ad, AKA gh-122248)
ericsnowcurrently added a commit that referenced this pull request Jul 29, 2024
The tests were only checking cases where the slot wrapper was present in the initial case.  They were missing when the slot wrapper was added in the additional initializations.  This fixes that.

(cherry-picked from commit 490e0ad, AKA gh-122248)
@bedevere-bot

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants