diff options
| author | historia <historiavg@proton.me> | 2026-08-24 05:04:09 -0400 |
|---|---|---|
| committer | historia <historiavg@proton.me> | 2026-08-24 05:04:09 -0400 |
| commit | 4f3d16e9d532efd6a6bda283925efeb8c93b51c7 (patch) | |
| tree | bd05c90a6fa47ee17bc521fde5519d19bab6a3d0 /app/tests | |
| parent | 0512a932bbf3b87619b6a250f87eb7904d22d1f7 (diff) | |
| download | tts-audiobook-generator-4f3d16e9d532efd6a6bda283925efeb8c93b51c7.tar.gz | |
feat: q behavior mirrors esc, port settings reordered
Diffstat (limited to 'app/tests')
| -rw-r--r-- | app/tests/test_hub.py | 8 | ||||
| -rw-r--r-- | app/tests/test_tui.py | 31 |
2 files changed, 24 insertions, 15 deletions
diff --git a/app/tests/test_hub.py b/app/tests/test_hub.py index 61f0da1..a009e3c 100644 --- a/app/tests/test_hub.py +++ b/app/tests/test_hub.py @@ -646,8 +646,8 @@ class SettingsTests(unittest.TestCase): hub._settings_menu(None) self.assertEqual([f["key"] for f in captured["fields"]], ["audio_format", "audio_bitrate", "language", - "chunk_size", "qwen_custom_port", "qwen_clone_port", - "faster_port", "audiocpp_port"]) + "chunk_size", "audiocpp_port", "faster_port", + "qwen_custom_port", "qwen_clone_port"]) kinds = {f["key"]: f["kind"] for f in captured["fields"]} self.assertEqual(kinds["audio_format"], "choice") self.assertEqual(kinds["audio_bitrate"], "text") @@ -658,9 +658,9 @@ class SettingsTests(unittest.TestCase): # The ports section note hangs off the first port field so it # renders between the output settings and the ports. notes = {f["key"]: f.get("note") for f in captured["fields"]} - self.assertTrue(notes["qwen_custom_port"]) + self.assertTrue(notes["audiocpp_port"]) self.assertIsNone(notes["audio_format"]) - self.assertIsNone(notes["audiocpp_port"]) + self.assertIsNone(notes["qwen_custom_port"]) self.assertEqual(applied, [{"audio_format": "ogg", "audio_bitrate": "192k", "language": "English", diff --git a/app/tests/test_tui.py b/app/tests/test_tui.py index e7fd976..d121c9a 100644 --- a/app/tests/test_tui.py +++ b/app/tests/test_tui.py @@ -215,11 +215,17 @@ class MenuTests(TuiTestCase): tui.menu(screen, "Pick", self.OPTIONS, back_value=marker), marker) - def test_q_still_aborts_with_back_value(self): + def test_q_returns_back_value_like_esc(self): marker = object() screen = FakeScreen(keys=[ord("q")]) + self.assertIs( + tui.menu(screen, "Pick", self.OPTIONS, back_value=marker), + marker) + + def test_q_aborts_without_back_value(self): + screen = FakeScreen(keys=[ord("q")]) with self.assertRaises(tui.WizardCancelled): - tui.menu(screen, "Pick", self.OPTIONS, back_value=marker) + tui.menu(screen, "Pick", self.OPTIONS) class MenuTableTests(TuiTestCase): @@ -459,11 +465,12 @@ class FormTests(TuiTestCase): tui.form(screen, "Settings", self._fields(), back_value=marker), marker) - def test_q_still_aborts_with_back_value(self): + def test_q_returns_back_value_like_esc(self): marker = object() screen = FakeScreen(keys=[ord("q")]) - with self.assertRaises(tui.WizardCancelled): - tui.form(screen, "Settings", self._fields(), back_value=marker) + self.assertIs( + tui.form(screen, "Settings", self._fields(), back_value=marker), + marker) def test_choice_field_picks_another_value(self): # Enter opens the choice menu on 'm4b' (index 1); Down moves to @@ -646,12 +653,13 @@ class BrowseDirectoryTests(TuiTestCase): back_value=marker), marker) - def test_q_still_aborts_with_back_value(self): + def test_q_returns_back_value_like_esc(self): marker = object() screen = FakeScreen(keys=[ord("q")]) - with self.assertRaises(tui.WizardCancelled): + self.assertIs( tui.browse_directory(screen, "Pick", start=self.root, - back_value=marker) + back_value=marker), + marker) class CheckboxTreeTests(TuiTestCase): @@ -717,12 +725,13 @@ class CheckboxTreeTests(TuiTestCase): back_value=marker), marker) - def test_q_still_aborts_with_back_value(self): + def test_q_returns_back_value_like_esc(self): marker = object() screen = FakeScreen(keys=[ord("q")]) - with self.assertRaises(tui.WizardCancelled): + self.assertIs( tui.checkbox_tree(screen, "Pick models", self.FAMILIES, - back_value=marker) + back_value=marker), + marker) class SuspendTests(TuiTestCase): |