stub a GITHUB_OUTPUT env var with tmp path

Author: 2bndy5

cpp-linter/src/run.rs

@@ -190,11 +190,19 @@ mod test {
     use super::run_main;
     use std::env;
 
-    #[tokio::test]
-    async fn normal() {
+    /// helper to avoid writing to the same GITHUB_OUTPUT file in parallel-running tests.
+    fn setup_tmp_gh_out_path() -> tempfile::TempDir {
+        let tmp_dir = tempfile::tempdir().unwrap();
+        let gh_out_path = tmp_dir.path().join("gh_out");
         unsafe {
-            env::remove_var("GITHUB_OUTPUT"); // avoid writing to GH_OUT in parallel-running tests
+            env::set_var("GITHUB_OUTPUT", gh_out_path.to_string_lossy().to_string());
         }
+        tmp_dir
+    }
+
+    #[tokio::test]
+    async fn normal() {
+        let tmp_gh_out = setup_tmp_gh_out_path();
         run_main(vec![
             "cpp-linter".to_string(),
             "-l".to_string(),
@@ -205,23 +213,21 @@ mod test {
         ])
         .await
         .unwrap();
+        drop(tmp_gh_out);
     }
 
     #[tokio::test]
     async fn version_command() {
-        unsafe {
-            env::remove_var("GITHUB_OUTPUT"); // avoid writing to GH_OUT in parallel-running tests
-        }
+        let tmp_gh_out = setup_tmp_gh_out_path();
         run_main(vec!["cpp-linter".to_string(), "version".to_string()])
             .await
             .unwrap();
+        drop(tmp_gh_out);
     }
 
     #[tokio::test]
     async fn force_debug_output() {
-        unsafe {
-            env::remove_var("GITHUB_OUTPUT"); // avoid writing to GH_OUT in parallel-running tests
-        }
+        let tmp_gh_out = setup_tmp_gh_out_path();
         run_main(vec![
             "cpp-linter".to_string(),
             "-l".to_string(),
@@ -231,13 +237,12 @@ mod test {
         ])
         .await
         .unwrap();
+        drop(tmp_gh_out);
     }
 
     #[tokio::test]
     async fn no_version_input() {
-        unsafe {
-            env::remove_var("GITHUB_OUTPUT"); // avoid writing to GH_OUT in parallel-running tests
-        }
+        let tmp_gh_out = setup_tmp_gh_out_path();
         run_main(vec![
             "cpp-linter".to_string(),
             "-l".to_string(),
@@ -246,12 +251,13 @@ mod test {
         ])
         .await
         .unwrap();
+        drop(tmp_gh_out);
     }
 
     #[tokio::test]
     async fn pre_commit_env() {
+        let tmp_gh_out = setup_tmp_gh_out_path();
         unsafe {
-            env::remove_var("GITHUB_OUTPUT"); // avoid writing to GH_OUT in parallel-running tests
             env::set_var("PRE_COMMIT", "1");
         }
         run_main(vec![
@@ -262,15 +268,14 @@ mod test {
         ])
         .await
         .unwrap_err();
+        drop(tmp_gh_out);
     }
 
     // Verifies that the system gracefully handles cases where all analysis is disabled.
     // This ensures no diagnostic comments are generated when analysis is explicitly skipped.
     #[tokio::test]
     async fn no_analysis() {
-        unsafe {
-            env::remove_var("GITHUB_OUTPUT"); // avoid writing to GH_OUT in parallel-running tests
-        }
+        let tmp_gh_out = setup_tmp_gh_out_path();
         run_main(vec![
             "cpp-linter".to_string(),
             "-l".to_string(),
@@ -281,19 +286,19 @@ mod test {
         ])
         .await
         .unwrap();
+        drop(tmp_gh_out);
     }
 
     #[tokio::test]
     async fn bad_repo_root() {
-        unsafe {
-            env::remove_var("GITHUB_OUTPUT"); // avoid writing to GH_OUT in parallel-running tests
-        }
+        let tmp_gh_out = setup_tmp_gh_out_path();
         run_main(vec![
             "cpp-linter".to_string(),
             "--repo-root".to_string(),
             "some-non-existent-dir".to_string(),
         ])
         .await
         .unwrap_err();
+        drop(tmp_gh_out);
     }
 }