Merge pull request #13023 from Byron/fix-test-isolation

isolate editor env in reword CLI tests

Author: Byron

crates/but-testsupport/src/prepare_cmd_env.rs

@@ -54,6 +54,9 @@ fn updates() -> Vec<EnvOp> {
         EnvOp::Remove("GIT_COMMON_DIR"),
         EnvOp::Remove("GIT_ASKPASS"),
         EnvOp::Remove("SSH_ASKPASS"),
+        EnvOp::Remove("GIT_EDITOR"),
+        EnvOp::Remove("VISUAL"),
+        EnvOp::Remove("EDITOR"),
     ]
     .into_iter()
     .chain(

crates/but/tests/but/command/reword.rs

@@ -89,7 +89,7 @@ fn reword_branch_from_editor_trims_trailing_newlines_in_confirmation_output() ->
         "#!/usr/bin/env bash\nprintf 'renamed-branch\\n\\n' > \"$1\"\n",
     );
     env.but("reword branch-to-rename-123")
-        .env("EDITOR", "bash editor.sh")
+        .env("GIT_EDITOR", "bash editor.sh")
         .assert()
         .success()
         .stdout_eq(str![[r#"