Skip to content

Add TEST_ENVIRONMENT option and set PYTHONSAFEPATH=1 in the test env #2388

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.

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

Merged
merged 5 commits into from
May 20, 2025

Conversation

joerick
Copy link
Contributor

@joerick joerick commented May 10, 2025

See #2358 for discussion.

Here's a draft of this solution. See what you think @henryiii.

Closes #2358

@henryiii
Copy link
Contributor

This looks good to me.

@henryiii henryiii force-pushed the test-pythonsafepath branch from 45ade11 to 28c589b Compare May 14, 2025 19:34
@joerick joerick added CI: PyPy Run the integration test suite with PyPy included CI: GraalPy Run the integration test suite with GraalPy included labels May 16, 2025
@joerick joerick marked this pull request as ready for review May 16, 2025 16:32
@henryiii henryiii force-pushed the test-pythonsafepath branch from 441f642 to 6f10ebe Compare May 19, 2025 02:37
@henryiii henryiii force-pushed the test-pythonsafepath branch from 6f10ebe to 64b009e Compare May 19, 2025 03:12
@joerick joerick merged commit fcdbed8 into main May 20, 2025
27 checks passed
@joerick joerick deleted the test-pythonsafepath branch May 20, 2025 12:41
@joerick joerick restored the test-pythonsafepath branch May 20, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: GraalPy Run the integration test suite with GraalPy included CI: PyPy Run the integration test suite with PyPy included
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PYTHONSAFEPATH in tests?
2 participants