Skip to content
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

gh-121090: Skip test_shebang_command_in_venv if Python is not installed #121091

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

encukou
Copy link
Member

@encukou encukou commented Jun 27, 2024

This skip works on a fresh Windows machine.

@encukou encukou requested a review from zooba June 27, 2024 11:29
@bedevere-app bedevere-app bot added the tests Tests in the Lib/test dir label Jun 27, 2024
@@ -748,6 +748,9 @@ def test_shebang_command_in_venv(self):
with self.script(f'#! /usr/bin/env {stem} arg1') as script:
data = self.run_py([script], expect_returncode=103)

if not is_installed("3"):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The function checks for specific registry keys, so you'll need a 3.x or 3.x-32/3.x-arm64 for this to ever return True.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
2 participants