feat(json): Abstract away writing json (#97)

Author: epage

Cargo.lock

@@ -26,6 +26,7 @@ pre-release-replacements = [
 [features]
 default = []
 serde = ["dep:serde"]
+json = ["dep:serde_json", "serde"]
 unstable-schema = ["serde", "dep:schemars", "dep:serde_json"]
 
 [dependencies]

crates/libtest-json/Cargo.toml

@@ -32,6 +32,13 @@ pub enum Event {
     },
 }
 
+impl Event {
+    #[cfg(feature = "json")]
+    pub fn to_jsonline(&self) -> String {
+        serde_json::to_string(self).expect("always valid json")
+    }
+}
+
 #[derive(Copy, Clone, Default, Debug, PartialEq, Eq)]
 #[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
 #[cfg_attr(feature = "serde", derive(serde::Serialize))]

crates/libtest-json/src/event.rs

@@ -26,7 +26,7 @@ pre-release-replacements = [
 [features]
 default = []
 color = ["dep:anstream", "dep:anstyle"]
-json = ["libtest-json/serde", "dep:serde", "dep:serde_json"]
+json = ["libtest-json/json"]
 threads = []
 
 [dependencies]
@@ -35,8 +35,6 @@ 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 }
 libtest-json = { version = "0.1.0", path = "../libtest-json" }
 
 [dev-dependencies]

crates/libtest2-harness/Cargo.toml

@@ -13,7 +13,7 @@ impl<W: std::io::Write> JsonNotifier<W> {
 
 impl<W: std::io::Write> super::Notifier for JsonNotifier<W> {
     fn notify(&mut self, event: Event) -> std::io::Result<()> {
-        let event = serde_json::to_string(&event)?;
+        let event = event.to_jsonline();
         writeln!(self.writer, "{event}")?;
         Ok(())
     }