fix(harness): Make color support optional

Author: epage

crates/libtest2-harness/Cargo.toml

@@ -25,16 +25,17 @@ pre-release-replacements = [
 
 [features]
 default = []
+color = ["dep:anstream", "dep:anstyle"]
 json = ["dep:serde", "dep:serde_json"]
 junit = []
 threads = []
 
 [dependencies]
-anstream = "0.6.4"
-anstyle = "1.0.10"
 lexarg-parser = { version = "0.1.0", path = "../lexarg-parser" }
 lexarg-error = { version = "0.1.0", path = "../lexarg-error" }
 libtest-lexarg = { version = "0.1.0", path = "../libtest-lexarg" }
+anstream = { version = "0.6.4", optional = true }
+anstyle = { version = "1.0.10", optional = true }
 serde = { version = "1.0.160", features = ["derive"], optional = true }
 serde_json = { version = "1.0.96", optional = true }
 

crates/libtest2-harness/src/harness.rs

@@ -37,6 +37,7 @@ impl Harness {
             std::process::exit(1)
         });
 
+        #[cfg(feature = "color")]
         match opts.color {
             libtest_lexarg::ColorConfig::AutoColor => anstream::ColorChoice::Auto,
             libtest_lexarg::ColorConfig::AlwaysColor => anstream::ColorChoice::Always,
@@ -131,7 +132,10 @@ fn parse<'p>(
 }
 
 fn notifier(opts: &libtest_lexarg::TestOpts) -> std::io::Result<Box<dyn notify::Notifier>> {
+    #[cfg(feature = "color")]
     let stdout = anstream::stdout();
+    #[cfg(not(feature = "color"))]
+    let stdout = std::io::stdout();
     let notifier: Box<dyn notify::Notifier> = match opts.format {
         #[cfg(feature = "json")]
         OutputFormat::Json => Box::new(notify::JsonNotifier::new(stdout)),

crates/libtest2-harness/src/notify/mod.rs

@@ -2,7 +2,10 @@
 mod json;
 #[cfg(feature = "junit")]
 mod junit;
+#[cfg(not(feature = "color"))]
+mod no_style;
 mod pretty;
+#[cfg(feature = "color")]
 mod style;
 mod summary;
 mod terse;
@@ -11,7 +14,10 @@ mod terse;
 pub(crate) use json::*;
 #[cfg(feature = "junit")]
 pub(crate) use junit::*;
+#[cfg(not(feature = "color"))]
+pub(crate) use no_style::*;
 pub(crate) use pretty::*;
+#[cfg(feature = "color")]
 pub(crate) use style::*;
 pub(crate) use summary::*;
 pub(crate) use terse::*;

crates/libtest2-harness/src/notify/no_style.rs

@@ -0,0 +1,11 @@
+pub(crate) struct Style;
+
+impl std::fmt::Display for Style {
+    fn fmt(&self, _formatter: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        Ok(())
+    }
+}
+
+pub(crate) const FAILED: Style = Style;
+pub(crate) const OK: Style = Style;
+pub(crate) const IGNORED: Style = Style;

crates/libtest2-mimic/Cargo.toml

@@ -24,7 +24,8 @@ pre-release-replacements = [
 ]
 
 [features]
-default = ["json", "junit", "threads"]
+default = ["color", "json", "junit", "threads"]
+color = ["libtest2-harness/color"]
 json = ["libtest2-harness/json"]
 junit = ["libtest2-harness/junit"]
 threads = ["libtest2-harness/threads"]

crates/libtest2/Cargo.toml

@@ -24,7 +24,8 @@ pre-release-replacements = [
 ]
 
 [features]
-default = ["json", "junit", "threads"]
+default = ["color", "json", "junit", "threads"]
+color = ["libtest2-harness/color"]
 json = ["libtest2-harness/json"]
 junit = ["libtest2-harness/junit"]
 threads = ["libtest2-harness/threads"]