Skip to content

Conversation

@Antraxmin
Copy link
Contributor

@AntraxminAntraxmin commented Aug 15, 2025

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.
@AntraxminAntraxmin 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

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
ContributorAuthor

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

@donbarbos
Copy link
Contributor

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

@ZeroIntensity
Copy link
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
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