feat: running process detection, menu gating

1 files changed, 126 insertions, 24 deletions

diff --git a/tests/test_hub.py b/tests/test_hub.py
index 5f6d992..ce9af43 100644
--- a/tests/test_hub.py
+++ b/tests/test_hub.py
@@ -1,15 +1,14 @@
-"""Tests for the TUI hub (hub.py) menu and helpers.
+"""Tests for the TUI hub (ui/hub.py) menu and helpers.
 
-The hub drives the same curses widgets as tui.py, so these tests reuse the
-fake curses/screen from test_tui to run the menu without a terminal.
+The hub drives the same curses widgets as ui/tui.py, so these tests reuse
+the fake curses/screen from test_tui to run the menu without a terminal.
 """
 
 import unittest
 from pathlib import Path
 from unittest.mock import patch
 
-import hub
-import tui
+from ui import hub, tui
 from tests.test_tui import FakeCurses, FakeScreen
 
 
@@ -37,13 +36,21 @@ class HubHelperTests(unittest.TestCase):
 
     def test_status_mark(self):
         from backends import BackendStatus
-        ready = BackendStatus("k", "l", installed=True, configured=True)
-        half = BackendStatus("k", "l", installed=True, configured=False)
+        running = BackendStatus("k", "l", installed=True, configured=True,
+                                running=True)
+        installed = BackendStatus("k", "l", installed=True,
+                                  configured=False)
         none = BackendStatus("k", "l", installed=False, configured=False)
-        self.assertEqual(hub._status_mark("k", [ready]), "ready")
-        self.assertEqual(hub._status_mark("k", [half]), "installed")
-        self.assertEqual(hub._status_mark("k", [none]), "not set up")
-        self.assertEqual(hub._status_mark("missing", []), "not set up")
+        # running beats installed (a server is up even if not configured);
+        # only a backend that is neither installed nor running is dimmed.
+        self.assertEqual(hub._status_mark(running),
+                         ("running", "ok", "body"))
+        self.assertEqual(hub._status_mark(installed),
+                         ("installed", "warn", "body"))
+        self.assertEqual(hub._status_mark(none),
+                         ("unavailable", "err", "dim"))
+        self.assertEqual(hub._status_mark(None),
+                         ("unavailable", "err", "dim"))
 
 
 class HubMenuTests(unittest.TestCase):
@@ -58,31 +65,126 @@ class HubMenuTests(unittest.TestCase):
         self.addCleanup(self._patcher.stop)
         self.addCleanup(tui._THEME.clear)
 
-    def test_quit_returns_none(self):
-        # Main menu: move to "Quit" (4th option, index 3) and press Enter.
-        screen = FakeScreen(keys=[FakeCurses.KEY_DOWN, FakeCurses.KEY_DOWN,
-                                  FakeCurses.KEY_DOWN, 10])
+    def _none_status(self, key="k", label="l"):
+        from backends import BackendStatus
+        return BackendStatus(key, label, installed=False, configured=False)
+
+    def test_quit_returns_none_when_no_backend(self):
+        # No backends installed/running: menu is [Set up, Quit]. Quit is the
+        # 2nd option (Down once) then Enter.
+        screen = FakeScreen(keys=[FakeCurses.KEY_DOWN, 10])
         with patch.object(hub, "detect_all", return_value=[]):
             result = hub._hub_menu(screen)
         self.assertIsNone(result)
 
-    def test_convert_with_no_ready_backend_offers_setup(self):
-        # Convert -> "Set up a backend..." is the only entry -> Enter selects
-        # it -> setup menu lists 3 backends; press Esc to go back -> convert
-        # returns None -> main menu loops. Then quit (Down x3 + Enter).
+    def test_menu_has_only_setup_and_quit_without_backends(self):
+        # Capture the options handed to tui.menu: with nothing installed or
+        # running, Convert/Configure must be absent.
+        captured = {}
+
+        def fake_menu(stdscr, title, options, **kwargs):
+            captured["options"] = options
+            return "quit"
+
+        screen = FakeScreen()
+        with patch.object(hub.tui, "menu", fake_menu), \
+                patch.object(hub, "detect_all", return_value=[]):
+            hub._hub_menu(screen)
+        labels = [label for label, _ in captured["options"]]
+        self.assertEqual(labels, ["Set up a backend...", "Quit"])
+
+    def test_menu_has_all_four_when_one_installed(self):
+        captured = {}
+
+        def fake_menu(stdscr, title, options, **kwargs):
+            captured["options"] = options
+            captured["rows"] = kwargs.get("table_rows")
+            return "quit"
+
+        screen = FakeScreen()
+        st = self._none_status("qwen", "qwen-tts")
+        st.installed = True
+        with patch.object(hub.tui, "menu", fake_menu), \
+                patch.object(hub, "detect_all", return_value=[st]):
+            hub._hub_menu(screen)
+        labels = [label for label, _ in captured["options"]]
+        self.assertEqual(
+            labels,
+            ["Convert books...", "Set up a backend...",
+             "Configure a backend...", "Quit"])
+        # The status table is passed through, one row per backend.
+        self.assertEqual(captured["rows"],
+                         [("qwen-tts", "installed", "warn", "body")])
+
+    def test_table_dims_name_when_not_installed_and_not_running(self):
+        captured = {}
+
+        def fake_menu(stdscr, title, options, **kwargs):
+            captured["rows"] = kwargs.get("table_rows")
+            return "quit"
+
+        screen = FakeScreen()
+        dead = self._none_status("audiocpp", "audio.cpp")
+        external = self._none_status("qwen", "qwen-tts")
+        external.running = True
+        with patch.object(hub.tui, "menu", fake_menu), \
+                patch.object(hub, "detect_all",
+                             return_value=[dead, external]):
+            hub._hub_menu(screen)
+        # Unusable backend: dim name. Running-but-not-installed stays bright.
+        self.assertEqual(
+            captured["rows"],
+            [("audio.cpp", "unavailable", "err", "dim"),
+             ("qwen-tts", "running", "ok", "body")])
+
+    def test_menu_has_all_four_when_one_running_only(self):
+        # Running but not installed (an external server) still unlocks the
+        # Convert/Configure entries.
+        captured = {}
+
+        def fake_menu(stdscr, title, options, **kwargs):
+            captured["options"] = options
+            return "quit"
+
+        screen = FakeScreen()
+        st = self._none_status("qwen", "qwen-tts")
+        st.running = True
+        with patch.object(hub.tui, "menu", fake_menu), \
+                patch.object(hub, "detect_all", return_value=[st]):
+            hub._hub_menu(screen)
+        labels = [label for label, _ in captured["options"]]
+        self.assertEqual(
+            labels,
+            ["Convert books...", "Set up a backend...",
+             "Configure a backend...", "Quit"])
+
+    def test_convert_with_no_available_backend_offers_setup(self):
+        # One installed-but-not-ready backend → Convert is offered. The
+        # convert menu lists no available backend, so only "Set up a
+        # backend..." is shown; Enter selects it → setup menu lists 3
+        # backends; Esc goes back → convert returns None → main menu loops.
+        # Then quit: main menu now has 4 options, Quit is the 4th (Down x3).
         from backends import BackendInfo, BackendStatus
-        none = BackendStatus("k", "l", installed=False, configured=False)
+        none = BackendStatus("k", "l", installed=True, configured=False)
         infos = [BackendInfo("audiocpp", "audio.cpp", lambda: none,
                              lambda: 0),
-                 BackendInfo("qwen", "Qwen", lambda: none, lambda: 0),
+                 BackendInfo("qwen", "qwen-tts", lambda: none, lambda: 0),
                  BackendInfo("faster", "faster", lambda: none, lambda: 0)]
-        with patch.object(hub, "detect_all", return_value=[none, none, none]), \
+        # installed=True so the main menu shows Convert; but ready/running
+        # is False so the convert menu's available list is empty.
+        statuses = [BackendStatus("audiocpp", "audio.cpp", installed=True,
+                                  configured=False),
+                    BackendStatus("qwen", "qwen-tts", installed=True,
+                                  configured=False),
+                    BackendStatus("faster", "faster", installed=True,
+                                  configured=False)]
+        with patch.object(hub, "detect_all", return_value=statuses), \
                 patch.object(hub, "REGISTRY", infos):
             # Convert(Enter), setup-entry(Enter), Esc on setup menu,
             # back at main menu -> Down x3 -> Enter (Quit).
             screen = FakeScreen(keys=[10, 10, 27,
-                                      FakeCurses.KEY_DOWN, FakeCurses.KEY_DOWN,
-                                      FakeCurses.KEY_DOWN, 10])
+                                       FakeCurses.KEY_DOWN, FakeCurses.KEY_DOWN,
+                                       FakeCurses.KEY_DOWN, 10])
             result = hub._hub_menu(screen)
         self.assertIsNone(result)