FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

pathlib tests: create `walk()` test hierarchy without using class under test by barneygale · Pull Request #128338 · python/cpython · GitHub

Repository navigation

pathlib tests: create walk() test hierarchy without using class under test - #128338

Merged
barneygale merged 2 commits into
python:mainfrom
barneygale:walk-tests-setup
Jan 4, 2025
Merged

barneygale merged 2 commits into
python:mainfrom
barneygale:walk-tests-setup

Conversation

Copy link
Copy Markdown
Contributor

In the tests for pathlib.Path.walk(), avoid using the path class under test (self.cls) in test setup. Instead we use os functions in test_pathlib, and direct manipulation of DummyPath internal data in test_pathlib_abc.

…er test

In the tests for `pathlib.Path.walk()`, avoid using the path class under
test (`self.cls`) in test setup. Instead we use `os` functions in
`test_pathlib`, and direct manipulation of `DummyPath` internal data in
`test_pathlib_abc`.
barneygale enabled auto-merge (squash) January 4, 2025 15:37
barneygale merged commit fd94c6a into python:main Jan 4, 2025
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 6, 2025
…er test (python#128338)

In the tests for `pathlib.Path.walk()`, avoid using the path class under
test (`self.cls`) in test setup. Instead we use `os` functions in
`test_pathlib`, and direct manipulation of `DummyPath` internal data in
`test_pathlib_abc`.
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
…er test (python#128338)

In the tests for `pathlib.Path.walk()`, avoid using the path class under
test (`self.cls`) in test setup. Instead we use `os` functions in
`test_pathlib`, and direct manipulation of `DummyPath` internal data in
`test_pathlib_abc`.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL