Fix UV_PYTHON_PREFERENCE reading breaks tox when env_site_packages_dir is in set_env

Signed-off-by: Bernát Gábor <[email protected]>

Author: gaborbernat

src/tox_uv/_venv.py

@@ -4,6 +4,7 @@
 
 import json
 import logging
+import os
 import sys
 from abc import ABC
 from functools import cached_property
@@ -73,7 +74,9 @@ def register_config(self) -> None:
         self.conf.add_config(
             keys=["uv_python_preference"],
             of_type=cast("Type[Optional[PythonPreference]]", Optional[PythonPreference]),  # noqa: UP006
-            default=lambda conf, name: self.environment_variables.get("UV_PYTHON_PREFERENCE", "system"),  # noqa: ARG005
+            # use os.environ here instead of self.environment_variables as this value is needed to create the virtual
+            # environment, if environment variables use env_site_packages_dir we would run into a chicken-egg problem.
+            default=lambda conf, name: os.environ.get("UV_PYTHON_PREFERENCE", "system"),  # noqa: ARG005
             desc=(
                 "Whether to prefer using Python installations that are already"
                 " present on the system, or those that are downloaded and"
@@ -240,10 +243,10 @@ def env_python(self) -> Path:
     def env_site_package_dir(self) -> Path:
         if sys.platform == "win32":  # pragma: win32 cover
             return self.venv_dir / "Lib" / "site-packages"
-        else:  # pragma: win32 no cover # noqa: RET505
-            py = self._py_info
-            impl = "pypy" if py.implementation == "pypy" else "python"
-            return self.venv_dir / "lib" / f"{impl}{py.version_dot}" / "site-packages"
+        # pragma: win32 no cover
+        py = self._py_info
+        impl = "pypy" if py.implementation == "pypy" else "python"
+        return self.venv_dir / "lib" / f"{impl}{py.version_dot}" / "site-packages"
 
     def env_version_spec(self) -> str:
         base = self.base_python.version_info

tests/test_tox_uv_venv.py

@@ -71,6 +71,22 @@ def test_uv_venv_preference_override_via_env_var(
     assert got == "only-managed"
 
 
+def test_uv_venv_preference_override_via_env_var_and_set_env_depends_on_py(
+    tox_project: ToxProjectCreator, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    project = tox_project({"tox.ini": "[testenv]\nset_env=A={env_site_packages_dir}"})
+    monkeypatch.setenv("UV_PYTHON_PREFERENCE", "only-managed")
+
+    result = project.run("c", "-k", "set_env")
+    result.assert_success()
+
+    parser = ConfigParser()
+    parser.read_string(result.out)
+    got = parser["testenv:py"]["set_env"]
+
+    assert str(project.path) in got
+
+
 def test_uv_venv_spec(tox_project: ToxProjectCreator) -> None:
     ver = sys.version_info
     project = tox_project({"tox.ini": f"[testenv]\npackage=skip\nbase_python={ver.major}.{ver.minor}"})