better interactive nav

Author: ozankasikci

src/commands/interactive/command.rs

@@ -516,6 +516,11 @@ where
         match self.focus {
             Focus::Worktrees => {
                 if self.worktrees.is_empty() {
+                    if !super::GLOBAL_ACTIONS.is_empty() {
+                        self.focus = Focus::GlobalActions;
+                        self.global_action_selected =
+                            super::GLOBAL_ACTIONS.len().saturating_sub(1);
+                    }
                     return;
                 }
                 if matches!(self.selected, Some(0)) && !super::GLOBAL_ACTIONS.is_empty() {

src/commands/interactive/tests.rs

@@ -318,7 +318,7 @@ fn up_from_top_moves_to_global_actions() -> Result<()> {
 
     let result = command.run(|_| Ok(()), |_, _| Ok(()))?;
 
-    assert_eq!(result, Some(String::from(super::REPO_ROOT_SELECTION)));
+    assert_eq!(result, Some(Selection::RepoRoot));
 
     Ok(())
 }
@@ -351,3 +351,26 @@ fn up_from_top_after_tabbing_picks_last_global_action() -> Result<()> {
 
     Ok(())
 }
+
+#[test]
+fn up_with_no_worktrees_moves_to_global_actions() -> Result<()> {
+    let backend = TestBackend::new(40, 12);
+    let terminal = Terminal::new(backend)?;
+    let events = StubEvents::new(vec![key(KeyCode::Up), key(KeyCode::Enter)]);
+
+    let worktrees = Vec::new();
+    let command = InteractiveCommand::new(
+        terminal,
+        events,
+        PathBuf::from("/tmp/worktrees"),
+        worktrees,
+        vec![String::from("main")],
+        Some(String::from("main")),
+    );
+
+    let result = command.run(|_| Ok(()), |_, _| Ok(()))?;
+
+    assert_eq!(result, Some(Selection::RepoRoot));
+
+    Ok(())
+}