From 4f3d16e9d532efd6a6bda283925efeb8c93b51c7 Mon Sep 17 00:00:00 2001 From: historia Date: Mon, 24 Aug 2026 05:04:09 -0400 Subject: feat: q behavior mirrors esc, port settings reordered --- app/tests/test_tui.py | 31 ++++++++++++++++++++----------- 1 file changed, 20 insertions(+), 11 deletions(-) (limited to 'app/tests/test_tui.py') 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): -- cgit v1.2.3