Remove CARGO_TERM_COLOR rather than set it to "never"

Author: smoelius

cargo-dylint/tests/ci.rs

@@ -8,7 +8,7 @@ use regex::Regex;
 use semver::{Op, Version};
 use similar_asserts::SimpleDiff;
 use std::{
-    env::{set_current_dir, set_var, var},
+    env::{remove_var, set_current_dir, var},
     ffi::OsStr,
     fmt::Write as _,
     fs::{read_dir, read_to_string, write},
@@ -26,7 +26,7 @@ static DESCRIPTION_REGEX: LazyLock<Regex> =
 fn initialize() {
     set_current_dir("..").unwrap();
     unsafe {
-        set_var(env::CARGO_TERM_COLOR, "never");
+        remove_var(env::CARGO_TERM_COLOR);
     }
 }
 

cargo-dylint/tests/integration/depinfo_dylint_libs.rs

@@ -1,12 +1,12 @@
 use assert_cmd::prelude::*;
 use dylint_internal::env;
 use predicates::prelude::*;
-use std::{env::set_var, process::Command};
+use std::{env::remove_var, process::Command};
 
 #[ctor::ctor]
 fn initialize() {
     unsafe {
-        set_var(env::CARGO_TERM_COLOR, "never");
+        remove_var(env::CARGO_TERM_COLOR);
     }
 }
 

cargo-dylint/tests/integration/library_packages.rs

@@ -1,7 +1,7 @@
 use assert_cmd::prelude::*;
 use dylint_internal::{CommandExt, env, packaging::isolate};
 use predicates::prelude::*;
-use std::{env::set_var, fs::OpenOptions, io::Write};
+use std::{env::remove_var, fs::OpenOptions, io::Write};
 use tempfile::tempdir;
 
 // smoelius: "Separate lints into categories" commit
@@ -10,7 +10,7 @@ const REV: &str = "402fc24351c60a3c474e786fd76aa66aa8638d55";
 #[ctor::ctor]
 fn initialize() {
     unsafe {
-        set_var(env::CARGO_TERM_COLOR, "never");
+        remove_var(env::CARGO_TERM_COLOR);
     }
 }
 

dylint-link/src/main.rs

@@ -377,7 +377,7 @@ linker = "false"
 
         std::process::Command::new("cargo")
             .env(env::CARGO_HOME, cargo_home.path())
-            .env(env::CARGO_TERM_COLOR, "never")
+            .env_remove(env::CARGO_TERM_COLOR)
             .current_dir(&package)
             .arg("build")
             .assert()

utils/testing/src/lib.rs

@@ -322,7 +322,7 @@ fn rustc_flags(metadata: &Metadata, package: &Package, target: &Target) -> Resul
         // say "Building `package` examples".
         dylint_internal::cargo::build(&format!("`{}` examples", package.name))
             .build()
-            .envs([(env::CARGO_TERM_COLOR, "never")])
+            .env_remove(env::CARGO_TERM_COLOR)
             .args([
                 "--manifest-path",
                 package.manifest_path.as_ref(),