fix(schema)!: Remove mode from CaseComplete

This is less a state of an individual Case and more of the operation as
a whole and is a bit redundant.

Author: epage

crates/libtest-json/event.schema.json

@@ -133,9 +133,6 @@
         "name": {
           "type": "string"
         },
-        "mode": {
-          "$ref": "#/$defs/RunMode"
-        },
         "status": {
           "description": "`None` means success",
           "anyOf": [

crates/libtest-json/src/event.rs

@@ -54,11 +54,6 @@ pub enum Event {
     },
     CaseComplete {
         name: String,
-        #[cfg_attr(
-            feature = "serde",
-            serde(default, skip_serializing_if = "RunMode::is_default")
-        )]
-        mode: RunMode,
         /// `None` means success
         #[cfg_attr(
             feature = "serde",
@@ -184,7 +179,6 @@ impl Event {
             }
             Self::CaseComplete {
                 name,
-                mode,
                 status,
                 message,
                 elapsed_s,
@@ -198,13 +192,6 @@ impl Event {
                 buffer.keyval_sep().unwrap();
                 buffer.value(name).unwrap();
 
-                if !mode.is_default() {
-                    buffer.val_sep().unwrap();
-                    buffer.key("mode").unwrap();
-                    buffer.keyval_sep().unwrap();
-                    buffer.value(mode.as_str()).unwrap();
-                }
-
                 if let Some(status) = status {
                     buffer.val_sep().unwrap();
                     buffer.key("status").unwrap();

crates/libtest-json/tests/roundtrip.rs

@@ -105,7 +105,6 @@ fn case_complete() {
     t(
         libtest_json::Event::CaseComplete {
             name: "Hello\tworld!".to_owned(),
-            mode: libtest_json::RunMode::Test,
             status: None,
             message: None,
             elapsed_s: None,
@@ -116,13 +115,12 @@ fn case_complete() {
     t(
         libtest_json::Event::CaseComplete {
             name: "Hello\tworld!".to_owned(),
-            mode: libtest_json::RunMode::Bench,
             status: Some(libtest_json::RunStatus::Ignored),
             message: Some("This\tfailed".to_owned()),
             elapsed_s: Some(libtest_json::Elapsed(Default::default())),
         },
         str![[
-            r#"{"event":"case_complete","name":"Hello\tworld!","mode":"bench","status":"ignored","message":"This\tfailed","elapsed_s":"0"}"#
+            r#"{"event":"case_complete","name":"Hello\tworld!","status":"ignored","message":"This\tfailed","elapsed_s":"0"}"#
         ]],
     );
 }

crates/libtest2-harness/src/harness.rs

@@ -440,7 +440,6 @@ fn run_case(
     let message = err.and_then(|e| e.cause().map(|c| c.to_string()));
     notifier.notify(notify::Event::CaseComplete {
         name: case.name().to_owned(),
-        mode: RunMode::Test,
         status,
         message,
         elapsed_s: Some(notify::Elapsed(start.elapsed())),