Skip to content

gh-115421: Update the list of installed test subdirectories. - #115422

Merged
Yhg1s merged 2 commits into
python:mainfrom
Yhg1s:testdirs
Feb 13, 2024
Merged

Yhg1s merged 2 commits into
python:mainfrom
Yhg1s:testdirs

Conversation

@Yhg1s

@Yhg1s Yhg1s commented Feb 13, 2024 •

Copy link
Copy Markdown
Member

Update the list of installed test subdirectories with all newly added subdirectories of Lib/test, so that the tests in those directories are properly installed. Also sort the list alphabetically so it's easy to update/check in the future.

subdirectories of Lib/test, so that the tests in those directories are
properly installed.

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but I didn't check manually if the list is exhaustive or not.

@Yhg1s
Yhg1s merged commit 518af37 into python:main Feb 13, 2024
fsc-eriker pushed a commit to fsc-eriker/cpython that referenced this pull request Feb 14, 2024
…ython#115422)

Update the list of installed test subdirectories with all newly added
subdirectories of Lib/test, so that the tests in those directories are
properly installed.
encukou added a commit that referenced this pull request Mar 11, 2024
…H-115813)

This backports:
- GH-115813
- GH-115422

Unlike on the main branch, new directories are added to the end,
so they're a bit easier to patch out if a redistributor needs to do so.

On main & 3.12, there's a special case for `idlelib/idle_test`; on
3.11 TESTSUBDIRS has several more entries that are not in `test/`.
This backport ignores all of them (including idlelib).
(The alternative would be list them, as additions to TEST_HOME_DIR.
But that's probably too invasive; people might split stdlib up in
surprising ways.)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@Yhg1s
Yhg1s deleted the testdirs branch August 13, 2026 12:52
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