Skip to content

Conversation

Antraxmin
Copy link
Contributor

@Antraxmin Antraxmin 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.
@Antraxmin Antraxmin requested a review from FFY00 as a code owner August 15, 2025 07:23
@bedevere-app bedevere-app bot added awaiting review tests Tests in the Lib/test dir labels Aug 15, 2025
@bedevere-app
Copy link

bedevere-app bot commented Aug 15, 2025

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
Contributor Author

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

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.

4 participants