From d4f4dedc3b84162e235d2dd892a40e5e6e834d54 Mon Sep 17 00:00:00 2001 From: konsti Date: Mon, 16 Sep 2024 13:28:14 +0200 Subject: [PATCH] Ignore activate venv in tests (#7423) --- crates/uv/tests/common/mod.rs | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/crates/uv/tests/common/mod.rs b/crates/uv/tests/common/mod.rs index 949bdcef2..54d9840cc 100644 --- a/crates/uv/tests/common/mod.rs +++ b/crates/uv/tests/common/mod.rs @@ -419,13 +419,15 @@ impl TestContext { /// * Don't wrap text output based on the terminal we're in, the test output doesn't get printed /// but snapshotted to a string. /// * Use a fake `HOME` to avoid accidentally changing the developer's machine. - /// * Hide other Python python with `UV_PYTHON_INSTALL_DIR` and installed interpreters with - /// `UV_TEST_PYTHON_PATH`. + /// * Hide other Pythons with `UV_PYTHON_INSTALL_DIR` and installed interpreters with + /// `UV_TEST_PYTHON_PATH` and an active venv (if applicable) by removing `VIRTUAL_ENV`. /// * Increase the stack size to avoid stack overflows on windows due to large async functions. pub fn add_shared_args(&self, command: &mut Command, activate_venv: bool) { command .arg("--cache-dir") .arg(self.cache_dir.path()) + // When running the tests in a venv, ignore that venv, otherwise we'll capture warnings. + .env_remove("VIRTUAL_ENV") .env("UV_NO_WRAP", "1") .env("HOME", self.home_dir.as_os_str()) .env("UV_PYTHON_INSTALL_DIR", "")