Skip to content

gh-131178: Add tests for site command-line interface - #137807

Closed
Antraxmin wants to merge 1 commit into
python:mainfrom
Antraxmin:gh-131178
Closed

gh-131178: Add tests for site command-line interface#137807
Antraxmin wants to merge 1 commit into
python:mainfrom
Antraxmin:gh-131178

Conversation

@Antraxmin

@AntraxminAntraxmin commented Aug 15, 2025

Copy link
Copy Markdown
Contributor

Add comprehensive CLI tests for the site module covering:

  • Basic usage with no arguments
  • --user-site option
  • --user-base option
  • Combined --user-base --user-site options
  • Invalid option handling

All tests use subprocess to verify actual CLI behavior.

Add comprehensive CLI tests for the site module covering:
- Basic usage with no arguments
- --user-site option
- --user-base option
- Combined --user-base --user-site options
- Invalid option handling
All tests use subprocess to verify actual CLI behavior.
@Antraxmin
Antraxmin requested a review from FFY00 as a code ownerAugust 15, 2025 07:23
@bedevere-appbedevere-appBot added awaiting review tests Tests in the Lib/test dir labels Aug 15, 2025
@bedevere-app

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@Antraxmin

Copy link
Copy Markdown
ContributorAuthor

@corona10 Hello. I'm a participant in the 2025 pycon KR sprint. Please review this PR!

@donbarbos

Copy link
Copy Markdown
Contributor

Thank you, but unfortunately we already have a similar PR #133582

@ZeroIntensity

Copy link
Copy Markdown
Member

Yeah, and since the tests on this one are failing, I think we should merge that one instead. If there's anything that this PR tests for that #133582 doesn't, feel free to add it in a follow-up.

@donbarbos

Copy link
Copy Markdown
Contributor

Let's close this PR since #133582 has already been merged

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting reviewskip newssprinttestsTests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Antraxmin@donbarbos@ZeroIntensity@corona10