fix(json): Allow elapsed_s to be optional in schema

Author: epage

crates/libtest-json/src/event.rs

@@ -20,6 +20,10 @@ pub enum Event {
         run: bool,
     },
     DiscoverComplete {
+        #[cfg_attr(
+            feature = "serde",
+            serde(default, skip_serializing_if = "Option::is_none")
+        )]
         elapsed_s: Option<Elapsed>,
     },
     SuiteStart,
@@ -44,9 +48,17 @@ pub enum Event {
             serde(default, skip_serializing_if = "Option::is_none")
         )]
         message: Option<String>,
+        #[cfg_attr(
+            feature = "serde",
+            serde(default, skip_serializing_if = "Option::is_none")
+        )]
         elapsed_s: Option<Elapsed>,
     },
     SuiteComplete {
+        #[cfg_attr(
+            feature = "serde",
+            serde(default, skip_serializing_if = "Option::is_none")
+        )]
         elapsed_s: Option<Elapsed>,
     },
 }

crates/libtest-json/tests/roundtrip.rs

@@ -47,7 +47,7 @@ fn discover_case() {
 fn discover_complete() {
     t(
         libtest_json::Event::DiscoverComplete { elapsed_s: None },
-        str![[r#"{"event":"discover-complete","elapsed_s":null}"#]],
+        str![[r#"{"event":"discover-complete"}"#]],
     );
 
     t(
@@ -86,7 +86,7 @@ fn case_complete() {
             message: None,
             elapsed_s: None,
         },
-        str![[r#"{"event":"case-complete","name":"Hello\tworld!","elapsed_s":null}"#]],
+        str![[r#"{"event":"case-complete","name":"Hello\tworld!"}"#]],
     );
 
     t(
@@ -107,7 +107,7 @@ fn case_complete() {
 fn suite_complete() {
     t(
         libtest_json::Event::SuiteComplete { elapsed_s: None },
-        str![[r#"{"event":"suite-complete","elapsed_s":null}"#]],
+        str![[r#"{"event":"suite-complete"}"#]],
     );
 
     t(