refactor

- fmt
- test

Author: Byron

crates/but/src/branch/mod.rs

@@ -48,12 +48,17 @@ pub enum Subcommands {
     },
 }
 
-pub async fn handle(cmd: &Option<Subcommands>, project: &Project, _json: bool) -> anyhow::Result<()> {
+pub async fn handle(
+    cmd: &Option<Subcommands>,
+    project: &Project,
+    _json: bool,
+) -> anyhow::Result<()> {
     match cmd {
         None => {
-            // Default to list when no subcommand is provided
-            list::list(project, false).await
+            let local = false;
+            list::list(project, local).await
         }
+        Some(Subcommands::List { local }) => list::list(project, *local).await,
         Some(Subcommands::New {
             branch_name,
             anchor,
@@ -145,7 +150,6 @@ pub async fn handle(cmd: &Option<Subcommands>, project: &Project, _json: bool) -
             println!("Branch '{}' not found in any stack", branch_name);
             Ok(())
         }
-        Some(Subcommands::List { local }) => list::list(project, *local).await,
         Some(Subcommands::Unapply { branch_name, force }) => {
             let stacks = but_api::workspace::stacks(
                 project.id,

crates/but/tests/but/journey.rs

@@ -118,6 +118,18 @@ fn from_workspace() -> anyhow::Result<()> {
         "snapshots/from-workspace/status01-verbose.stdout.term.svg"
     ]);
 
+    // List is the default
+    env.but("branch")
+        .assert()
+        .success()
+        .stdout_eq(file!["snapshots/from-workspace/branch01.stdout.term.svg"]);
+
+    // But list is also explicit.
+    env.but("branch list")
+        .assert()
+        .success()
+        .stdout_eq(file!["snapshots/from-workspace/branch01.stdout.term.svg"]);
+
     // TODO: more operations on the repository!
     Ok(())
 }