feat(json): Specify MessageKind to be in precedence order

Author: epage

crates/libtest-json/event.schema.json

@@ -216,8 +216,8 @@
     "MessageKind": {
       "type": "string",
       "enum": [
-        "ignored",
-        "failed"
+        "failed",
+        "ignored"
       ]
     },
     "CaseMessage": {

crates/libtest-json/src/event.rs

@@ -472,13 +472,14 @@ impl RunMode {
     }
 }
 
-#[derive(Copy, Clone, Debug, PartialEq, Eq)]
+#[derive(Copy, Clone, Debug, PartialEq, Eq, PartialOrd, Ord)]
 #[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
 #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 #[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
 pub enum MessageKind {
-    Ignored,
+    // Highest precedent items for determining test status last
     Failed,
+    Ignored,
 }
 
 impl MessageKind {

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

@@ -145,12 +145,7 @@ impl super::Notifier for Summary {
 fn find_run_status(case_status: &CaseStatus) -> Option<MessageKind> {
     let mut status = None;
     for event in &case_status.messages {
-        status = match (status, event.status) {
-            (None, _) => Some(event.status),
-            (Some(MessageKind::Ignored), _) => status,
-            (_, MessageKind::Ignored) => Some(event.status),
-            (Some(MessageKind::Failed), _) => status,
-        }
+        status = status.max(Some(event.status));
     }
     status
 }