feat(update_backlog)!: move CLI to verb-criteria model

Author: ErichDonGubler

moz-webgpu-cts/src/main.rs

@@ -127,12 +127,14 @@ enum Subcommand {
         #[clap(value_enum, long, default_value_t = Default::default())]
         on_zero_item: OnZeroItem,
     },
-    /// Identify and promote tests that are ready to come out of the `backlog` implementation
-    /// status.
+    /// Set `implementation-status` properties to `backlog` or `implementing` based on a
+    /// given `criteria`, allowing changes only in the permitted `direction`.
     UpdateBacklog {
-        /// The mode to use for updating tests.
+        /// Direction(s) permitted for `implementation-status` changes.
+        direction: UpdateBacklogDirection,
+        /// The criteria to use to determine whether something is ready to come out of `backlog`.
         #[clap(subcommand)]
-        preset: UpdateBacklogSubcommand,
+        criteria: UpdateBacklogCriteria,
     },
     /// Dump all metadata as JSON. Do so at your own risk; no guarantees are made about the
     /// schema of this JSON, for now.
@@ -289,8 +291,9 @@ enum OnZeroItem {
     Hide,
 }
 
-#[derive(Clone, Copy, Debug)]
+#[derive(Clone, Copy, Debug, ValueEnum)]
 enum UpdateBacklogDirection {
+    /// Allows promotions from `backlog` to `implementing`.
     Promote,
 }
 
@@ -302,15 +305,10 @@ impl UpdateBacklogDirection {
     }
 }
 
-#[derive(Clone, Copy, Debug)]
-enum UpdateBacklogCriteria {
-    PermaPassing { only_across_all_platforms: bool },
-}
-
 #[derive(Clone, Copy, Debug, Parser)]
-enum UpdateBacklogSubcommand {
-    /// Remove tests that expect only `PASS` outcomes on all platforms from `backlog`.
-    PromotePermaPassing {
+enum UpdateBacklogCriteria {
+    /// Determine status based on `PASS` outcomes on all platforms from `backlog`.
+    PermaPassing {
         #[clap(long)]
         only_across_all_platforms: bool,
     },
@@ -1003,17 +1001,10 @@ fn run(cli: Cli) -> ExitCode {
             println!("Full analysis: {analysis:#?}");
             ExitCode::SUCCESS
         }
-        Subcommand::UpdateBacklog { preset } => {
-            let (direction, criteria) = match preset {
-                UpdateBacklogSubcommand::PromotePermaPassing {
-                    only_across_all_platforms,
-                } => (
-                    UpdateBacklogDirection::Promote,
-                    UpdateBacklogCriteria::PermaPassing {
-                        only_across_all_platforms,
-                    },
-                ),
-            };
+        Subcommand::UpdateBacklog {
+            direction,
+            criteria,
+        } => {
             let mut files = {
                 let mut found_parse_err = false;
                 let extracted = read_and_parse_all_metadata(browser, &checkout)