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-130614: Add test suites for readable and writable pathlib paths #130648

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

barneygale
Copy link
Contributor

@barneygale barneygale commented Feb 27, 2025

Add test.test_pathlib.test_read, which tests ReadablePath subclasses. Tests are run against ReadableZipPath, ReadableLocalPath, and Path. This test suite is mostly incomplete.

Add test.test_pathlib.test_write, which tests WritablePath subclasses. Tests are run against WritableZipPath, WritableLocalPath, and Path. This test suite is mostly complete.

Add test.test_pathlib.support.zip_path, which provides the *ZipPath classes mentioned above. It also provides a ZipPathGround class that's used by tests to make assertions about the contents of the zip file.

Add test.test_pathlib.support.local_path, which provides the *LocalPath classes mentioned above. They're stripped-back versions of pathlib.Path, basically. It also provides LocalPathGround class that's used by tests to access the local filesystem.

…paths

Add `test.test_pathlib.test_read`, which tests `ReadablePath` subclasses.
Tests are run against `ReadableZipPath`, `ReadableLocalPath`, and `Path`.
This test suite is mostly incomplete.

Add `test.test_pathlib.test_write`, which tests `WritablePath` subclasses.
Tests are run against `WritableZipPath`, `WritableLocalPath`, and `Path`.
This test suite is mostly complete.

Add `test.test_pathlib.support.zip_path`, which provides the `*ZipPath`
classes mentioned above. It also provides a `ZipPathGround` class that's
used by tests to make assertions about the contents of the zip file.

Add `test.test_pathlib.support.local_path`, which provides the `*LocalPath`
classes mentioned above. They're stripped-back versions of `pathlib.Path`,
basically. It also provides `LocalPathGround` class that's used by tests to
access the local filesystem.
@barneygale
Copy link
Contributor Author

@srinivasreddy here's a framework for copy tests - see test_copy.py. There a bunch more copying tests in test_pathlib_abc that will need to be ported to the new framework, if you'd like to have a go :)

@srinivasreddy
Copy link
Contributor

srinivasreddy commented Mar 3, 2025

@barneygale You have made significant progress, i do not think it would be right for me to start working on this. I will find something else to work on. Thanks 🙏🏾

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

Successfully merging this pull request may close these issues.

2 participants