Skip to content

bpo-37935: Added tests for os.walk(), glob.iglob() and Path.glob()#15956

Merged
gpshead merged 1 commit intopython:masterfrom
serhiy-storchaka:test-walk-many-open-files
Sep 12, 2019
Merged

bpo-37935: Added tests for os.walk(), glob.iglob() and Path.glob()#15956
gpshead merged 1 commit intopython:masterfrom
serhiy-storchaka:test-walk-many-open-files

Conversation

@serhiy-storchaka
Copy link
Copy Markdown
Member

@serhiy-storchaka serhiy-storchaka commented Sep 11, 2019

Test that they do not keep too many file descriptors open for the host OS in a reasonable test scenario.

See bpo-37935.

that they do not keep many file descriptors open.
See bpo-37935.
@gpshead gpshead self-assigned this Sep 12, 2019
@gpshead gpshead changed the title Added tests for os.walk(), glob.iglob() and Path.glob() bpo-37935: Added tests for os.walk(), glob.iglob() and Path.glob() Sep 12, 2019
@gpshead gpshead merged commit f9dc2ad into python:master Sep 12, 2019
@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @serhiy-storchaka for the PR, and @gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 12, 2019
…ythonGH-15956)

Test that they do not keep too many file descriptors open for the host OS in a reasonable test scenario.

See [bpo-37935](https://bugs.python.org/issue37935).
(cherry picked from commit f9dc2ad)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot
Copy link
Copy Markdown

GH-16043 is a backport of this pull request to the 3.8 branch.

gpshead pushed a commit that referenced this pull request Sep 12, 2019
…H-15956) (GH-16043)

Test that they do not keep too many file descriptors open for the host OS in a reasonable test scenario.

See [bpo-37935](https://bugs.python.org/issue37935).
(cherry picked from commit f9dc2ad)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@serhiy-storchaka serhiy-storchaka deleted the test-walk-many-open-files branch October 7, 2019 16:58
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.

5 participants