refactor: use derive(Default) on {Test,Subtest}Outcome

Author: ErichDonGubler

moz-webgpu-cts/src/wpt/metadata.rs

@@ -1241,11 +1241,12 @@ pub(crate) const CRASH: &str = "CRASH";
 pub(crate) const OK: &str = "OK";
 pub(crate) const ERROR: &str = "ERROR";
 
-#[derive(Debug, Deserialize, EnumSetType, Hash, Serialize)]
+#[derive(Debug, Default, Deserialize, EnumSetType, Hash, Serialize)]
 #[enumset(serialize_repr = "list")]
 #[enumset(serialize_deny_unknown)]
 #[serde(rename_all = "UPPERCASE")]
 pub enum TestOutcome {
+    #[default]
     Ok,
     Pass,
     Fail,
@@ -1255,12 +1256,6 @@ pub enum TestOutcome {
     Skip,
 }
 
-impl Default for TestOutcome {
-    fn default() -> Self {
-        Self::Ok
-    }
-}
-
 impl Display for TestOutcome {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         write!(
@@ -1310,23 +1305,18 @@ impl<'a> Properties<'a> for TestProps<TestOutcome> {
     }
 }
 
-#[derive(Debug, Deserialize, EnumSetType, Hash, Serialize)]
+#[derive(Debug, Default, Deserialize, EnumSetType, Hash, Serialize)]
 #[enumset(serialize_repr = "list")]
 #[enumset(serialize_deny_unknown)]
 #[serde(rename_all = "UPPERCASE")]
 pub enum SubtestOutcome {
+    #[default]
     Pass,
     Fail,
     Timeout,
     NotRun,
 }
 
-impl Default for SubtestOutcome {
-    fn default() -> Self {
-        Self::Pass
-    }
-}
-
 impl Display for SubtestOutcome {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         write!(