test: clean cwd in unit tests (#2211)

Author: jdx

src/cli/env.rs

@@ -72,6 +72,7 @@ mod tests {
     use std::env;
 
     use pretty_assertions::assert_str_eq;
+    use test_log::test;
 
     use crate::cli::tests::grep;
     use crate::dirs;

src/cli/plugins/ls_remote.rs

@@ -67,8 +67,13 @@ Examples:
 
 #[cfg(test)]
 mod tests {
+    use test_log::test;
+
+    use crate::test::reset;
+
     #[test]
     fn test_plugin_list_remote() {
+        reset();
         let stdout = assert_cli!("plugin", "ls-remote");
         assert!(stdout.contains("tiny"));
     }

src/file.rs

@@ -269,7 +269,8 @@ pub fn is_executable(path: &Path) -> bool {
 }
 
 #[cfg(unix)]
-pub fn make_executable(path: &Path) -> Result<()> {
+pub fn make_executable<P: AsRef<Path>>(path: P) -> Result<()> {
+    let path = path.as_ref();
     let mut perms = path.metadata()?.permissions();
     perms.set_mode(perms.mode() | 0o111);
     fs::set_permissions(path, perms)

src/plugins/core/bun.rs

@@ -87,7 +87,7 @@ impl BunPlugin {
                 .join("bun"),
             self.bun_bin(&ctx.tv),
         )?;
-        file::make_executable(&self.bun_bin(&ctx.tv))?;
+        file::make_executable(self.bun_bin(&ctx.tv))?;
         file::make_symlink(Path::new("./bun"), &ctx.tv.install_path().join("bin/bunx"))?;
         Ok(())
     }

src/plugins/core/deno.rs

@@ -87,7 +87,7 @@ impl DenoPlugin {
         file::create_dir_all(tv.install_path().join("bin"))?;
         file::unzip(tarball_path, &tv.download_path())?;
         file::rename(tv.download_path().join("deno"), self.deno_bin(tv))?;
-        file::make_executable(&self.deno_bin(tv))?;
+        file::make_executable(self.deno_bin(tv))?;
         Ok(())
     }
 

src/plugins/core/erlang.rs

@@ -66,7 +66,7 @@ impl ErlangPlugin {
             &self.kerl_path(),
             None,
         )?;
-        file::make_executable(&self.kerl_path())?;
+        file::make_executable(self.kerl_path())?;
         Ok(())
     }
 

src/plugins/core/node.rs

@@ -193,7 +193,7 @@ impl NodePlugin {
     fn install_npm_shim(&self, tv: &ToolVersion) -> Result<()> {
         file::remove_file(self.npm_path(tv)).ok();
         file::write(self.npm_path(tv), include_str!("assets/node_npm_shim"))?;
-        file::make_executable(&self.npm_path(tv))?;
+        file::make_executable(self.npm_path(tv))?;
         Ok(())
     }
 

src/test.rs

@@ -76,10 +76,31 @@ pub fn reset() {
     Config::reset();
     forge::reset();
     config_file::reset();
+    file::remove_all(&*env::HOME.join("cwd")).unwrap();
+    file::create_dir_all(&*env::HOME.join("cwd")).unwrap();
     env::set_current_dir(env::HOME.join("cwd")).unwrap();
     env::remove_var("MISE_FAILURE");
     file::remove_all(&*dirs::TRUSTED_CONFIGS).unwrap();
     file::remove_all(&*dirs::TRACKED_CONFIGS).unwrap();
+    file::create_dir_all(".mise/tasks").unwrap();
+    file::write(
+        ".mise/tasks/filetask",
+        indoc! {r#"#!/usr/bin/env bash
+# mise alias=["ft"]
+# mise description="This is a test build script"
+# mise depends=["lint", "test"]
+# mise sources=[".test-tool-versions"]
+# mise outputs=["$MISE_PROJECT_ROOT/test/test-build-output.txt"]
+# mise env={TEST_BUILDSCRIPT_ENV_VAR = "VALID"}
+
+set -euxo pipefail
+cd "$MISE_PROJECT_ROOT" || exit 1
+echo "running test-build script"
+echo "TEST_BUILDSCRIPT_ENV_VAR: $TEST_BUILDSCRIPT_ENV_VAR" > test-build-output.txt
+    "#},
+    )
+    .unwrap();
+    file::make_executable(".mise/tasks/filetask").unwrap();
     file::write(
         env::HOME.join(".test-tool-versions"),
         indoc! {r#"

test/.gitignore

@@ -2,5 +2,5 @@ data/
 !data/plugins
 cache/
 state/
-cwd/man/
+cwd/
 test-build-output.txt