8000 Use `EnvironmentVarGuard` in tests instead of manually messing with `os.environ` · Issue #128438 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

Use EnvironmentVarGuard in tests instead of manually messing with os.environ #128438

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
WolframAlph opened this issue Jan 3, 2025 · 0 comments
Open

Use EnvironmentVarGuard in tests instead of manually messing with os.environ #128438

WolframAlph opened this issue Jan 3, 2025 · 0 comments
Labels
tests Tests in the Lib/test dir type-feature A feature request or enhancement

Comments

@WolframAlph
Copy link
Contributor
WolframAlph commented Jan 3, 2025 •
edited by bedevere-app bot
Loading

Feature or enhancement

Proposal:

Some tests manually tweak os.environ. There is new way to do that with EnvironmentVarGuard. Identify all places and update.

Has this already been discussed elsewhere?

No response given

Links to previous discussion of this feature:

No response

Linked PRs
@WolframAlph WolframAlph added the type-feature A feature request or enhancement label Jan 3, 2025
@bedevere-app bedevere-app bot mentioned this issue Jan 3, 2025
Open
@WolframAlph WolframAlph changed the title Use EnvironmentVarGuard in test_cmd_line instead of manually messing with os.environ Use EnvironmentVarGuard in tests instead of manually messing with os.environ Jan 3, 2025
@picnixz picnixz added tests Tests in the Lib/test dir and removed type-feature A feature request or enhancement labels Jan 3, 2025
This was referenced Jan 4, 2025
Merged
Merged
gaogaotiantian pushed a commit that referenced this issue Jan 11, 2025
@WolframAlph
gh-128438: Add EnvironmentVarGuard for test_pdb.py (#128522)
0946ed2
picnixz pushed a commit that referenced this issue Jan 15, 2025
@WolframAlph
gh-128438: Use EnvironmentVarGuard for test_{builtin,io,locale}.py (
ae7f621 
#128476)

Modifying locale-related environment variables in `Lib/test/test_builtin.py`,
`Lib/test/test_io.py` and `Lib/test/test_locale.py` is now achieved by using
an `EnvironmentVarGuard` context instead of an explicit `try-finally` block.
@bedevere-app bedevere-app bot mentioned this issue Feb 11, 2025
Merged
Eclips4 pushed a commit that referenced this issue Mar 27, 2025
@Wulian233
gh-128438: Use EnvironmentVarGuard for datetimetester.py to manag…
a096a41 
…e environment varibales (#130002)
@bedevere-app bedevere-app bot mentioned this issue Mar 29, 2025
Merged
@picnixz picnixz added the type-feature A feature request or enhancement label Mar 29, 2025
diegorusso pushed a commit to diegorusso/cpython that referenced this issue Apr 1, 2025
@Wulian233 @diegorusso
pythongh-128438: Use EnvironmentVarGuard for datetimetester.py to…
1aec098 
… manage environment varibales (python#130002)
seehwan pushed a commit to seehwan/cpython that referenced this issue Apr 16, 2025
@Wulian233 @seehwan80
pythongh-128438: Use EnvironmentVarGuard for datetimetester.py to…
e60ecee 
… manage environment varibales (python#130002)
picnixz pushed a commit that referenced this issue Apr 27, 2025
@Wulian233
gh-128438: Use EnvironmentVarGuard in test_zoneinfo.py (#131870)
8b4fd24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tests Tests in the Lib/test dir type-feature A feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@picnixz @gaogaotiantian @WolframAlph
0