Skip to content

Commit 958e61e

Browse files
committed
fix: robustness in venvs_site_packages_libs_test for Windows paths
1 parent fd6cda6 commit 958e61e

1 file changed

Lines changed: 10 additions & 3 deletions

File tree

  • tests/venv_site_packages_libs

tests/venv_site_packages_libs/bin.py

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,14 +112,21 @@ def test_whl_with_data_included(self):
112112
venv_root = Path(self.venv)
113113

114114
is_windows = sys.platform == "win32"
115-
bin_dir_name = "Scripts" if is_windows else "bin"
116-
include_dir_name = "Include" if is_windows else "include"
115+
116+
# On Windows, rules_python usually uses Scripts, but some environments or
117+
# configurations might use bin.
118+
if is_windows:
119+
bin_dir_name = "Scripts" if (venv_root / "Scripts").exists() else "bin"
120+
include_dir_name = "Include" if (venv_root / "Include").exists() else "include"
121+
else:
122+
bin_dir_name = "bin"
123+
include_dir_name = "include"
117124

118125
# data
119126
data_data_file = venv_root / "data" / "whl_with_data" / "data_data_file.txt"
120127
self.assertTrue(
121128
data_data_file.exists(),
122-
f"Expected {data_data_file} to exist. venv_root contents: {list(venv_root.iterdir())}. os.name={os.name}, sys.platform={sys.platform}",
129+
f"Expected {data_data_file} to exist. venv_root contents: {list(venv_root.iterdir()) if venv_root.exists() else 'N/A'}. os.name={os.name}, sys.platform={sys.platform}",
123130
)
124131

125132
# scripts

0 commit comments

Comments
 (0)