Try less relative paths for running venv executables

Author: benbrandt

tests/bindings.rs

@@ -143,13 +143,13 @@ where
         .assert()
         .success();
 
-    Command::new(venv_executable_path("pip"))
+    Command::new(path.join(".venv/bin/pip"))
         .current_dir(path)
         .args(["install", "mypy"])
         .assert()
         .success();
 
-    Command::new(venv_executable_path("mypy"))
+    Command::new(path.join(".venv/bin/mypy"))
         .current_dir(path)
         .args(args)
         .assert()
@@ -159,13 +159,6 @@ where
         )
 }
 
-fn venv_executable_path(executable: &str) -> String {
-    format!(
-        "./.venv/bin/{executable}{}",
-        if cfg!(windows) { ".exe" } else { "" }
-    )
-}
-
 fn install_numpy(path: &Path) {
     Command::new("curl")
         .current_dir(path)

tests/componentize.rs

@@ -195,13 +195,13 @@ fn sandbox_example() -> anyhow::Result<()> {
         .assert()
         .success();
 
-    Command::new(venv_executable_path("pip"))
+    Command::new(path.join(".venv/bin/pip"))
         .current_dir(&path)
         .args(["install", "wasmtime"])
         .assert()
         .success();
 
-    Command::new(venv_executable_path("python"))
+    Command::new(path.join(".venv/bin/python"))
         .current_dir(&path)
         .args([
             "-m",
@@ -213,7 +213,7 @@ fn sandbox_example() -> anyhow::Result<()> {
         .assert()
         .success();
 
-    Command::new(venv_executable_path("python"))
+    Command::new(path.join(".venv/bin/python"))
         .current_dir(&path)
         .args(["host.py", "2 + 2"])
         .assert()
@@ -298,10 +298,3 @@ fn install_numpy(path: &Path) {
         .assert()
         .success();
 }
-
-fn venv_executable_path(executable: &str) -> String {
-    format!(
-        "./.venv/bin/{executable}{}",
-        if cfg!(windows) { ".exe" } else { "" }
-    )
-}