Skip to content
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

GH-110109: Test pure functionality of pathlib.Path user subclasses #112242

Merged
merged 1 commit into from
Nov 18, 2023

Conversation

barneygale
Copy link
Contributor

@barneygale barneygale commented Nov 18, 2023

Add PurePathTest as a superclass of PathTest, and therefore also PathSubclassTest. This adds coverage of pure functionality in user subclasses of pathlib.Path.

Remove PosixPathAsPureTest and WindowsPathAsPureTest, as they now duplicate PosixPathTest and WindowsPathTest.

This makes the MROs of test unit classes match the MROs of pathlib classes.

…asses

Add `PurePathTest` as a superclass of `PathTest`, and therefore also
`PathSubclassTest`. This adds coverage of pure functionality in user
subclasses of `pathlib.Path`.

Remove `PosixPathAsPureTest` and `WindowsPathAsPureTest`, as they
now duplicate `PosixPathTest` and `WindowsPathTest`.

This makes the MROs of test unit classes match the MROs of pathlib
classes.
@barneygale barneygale added tests Tests in the Lib/test dir skip news topic-pathlib labels Nov 18, 2023
@bedevere-app bedevere-app bot mentioned this pull request Nov 18, 2023
8 tasks
@barneygale barneygale merged commit 607b5e3 into python:main Nov 18, 2023
29 checks passed
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…asses (python#112242)

Add `PurePathTest` as a superclass of `PathTest`, and therefore also
`PathSubclassTest`. This adds coverage of pure functionality in user
subclasses of `pathlib.Path`.

Remove `PosixPathAsPureTest` and `WindowsPathAsPureTest`, as they
now duplicate `PosixPathTest` and `WindowsPathTest`.

This makes the MROs of test unit classes match the MROs of pathlib
classes.
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
…asses (python#112242)

Add `PurePathTest` as a superclass of `PathTest`, and therefore also
`PathSubclassTest`. This adds coverage of pure functionality in user
subclasses of `pathlib.Path`.

Remove `PosixPathAsPureTest` and `WindowsPathAsPureTest`, as they
now duplicate `PosixPathTest` and `WindowsPathTest`.

This makes the MROs of test unit classes match the MROs of pathlib
classes.
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 topic-pathlib
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant