diff options
Diffstat (limited to 'app/tests/test_runview.py')
| -rw-r--r-- | app/tests/test_runview.py | 81 |
1 files changed, 81 insertions, 0 deletions
diff --git a/app/tests/test_runview.py b/app/tests/test_runview.py index 0d99a6a..7f79053 100644 --- a/app/tests/test_runview.py +++ b/app/tests/test_runview.py @@ -237,6 +237,46 @@ class StateTransitionTests(_FakeTui, unittest.TestCase): self.assertEqual(view.server, "error") self.assertEqual(view.log_tail, ["boom"]) + def test_server_exit_keeps_a_known_crash_hint(self): + view, _ = self.make_view() + view.handle_event({"kind": "exited", "name": "sglomni", + "returncode": 1, "log_tail": ["fp8..."], + "hint": "FP8 needs compute capability 8.9+"}) + self.assertEqual(view.phase, "error") + self.assertEqual(view.boot_hint, + "FP8 needs compute capability 8.9+") + + def test_boot_failure_is_recorded_in_the_dated_log(self): + # A failed boot never reaches the converter, so without this the + # dated log the failure pointers name would stay blank. + with tempfile.TemporaryDirectory() as tmp: + log_path = os.path.join(tmp, "audiobook_test.log") + view, _ = self.make_view(log_path=log_path) + view.handle_event({"kind": "starting", "name": "sglomni", + "log_path": "/tmp/sglomni-server.log"}) + view.handle_event({"kind": "exited", "name": "sglomni", + "returncode": 1, "log_tail": ["boom"], + "hint": "FP8 needs compute capability 8.9+"}) + with open(log_path, encoding="utf-8") as logf: + text = logf.read() + self.assertIn("ERROR - server exited with code 1", text) + self.assertIn("WARNING - hint: FP8 needs compute capability 8.9+", + text) + self.assertIn("the server's own output is in /tmp/sglomni-server.log", + text) + + def test_boot_timeout_is_recorded_without_optional_detail(self): + with tempfile.TemporaryDirectory() as tmp: + log_path = os.path.join(tmp, "audiobook_test.log") + view, _ = self.make_view(log_path=log_path) + view.handle_event({"kind": "timeout", "name": "sglomni", + "seconds": 1200, "log_tail": []}) + with open(log_path, encoding="utf-8") as logf: + text = logf.read() + self.assertIn("ERROR - server did not become ready in time", text) + self.assertNotIn("hint:", text) + self.assertNotIn("the server's own output is in", text) + def test_server_down_during_convert(self): view, _ = self.make_view() view.handle_event({"kind": "book", "index": 1, "total": 1, @@ -397,6 +437,25 @@ class RenderTests(_FakeTui, unittest.TestCase): self.assertIn("not responding", text) self.assertIn("the server is not responding", text) + def test_error_summary_draws_the_boot_hint(self): + view, screen = self.make_view() + view.handle_event({"kind": "exited", "name": "sglomni", + "returncode": 1, "log_tail": [], + "hint": "FP8 needs compute capability 8.9+"}) + view.render() + self.assertIn("FP8 needs compute capability 8.9+", + self._strings(screen)) + + def test_error_screen_names_the_server_log(self): + view, screen = self.make_view() + view.handle_event({"kind": "starting", "name": "sglomni", + "log_path": "/tmp/sglomni-server.log"}) + view.handle_event({"kind": "exited", "name": "sglomni", + "returncode": 1, "log_tail": ["boom"]}) + view.render() + self.assertIn("server log: /tmp/sglomni-server.log", + self._strings(screen)) + def test_summary_screen_after_done(self): view, screen = self.make_view() view.handle_event({"kind": "book", "index": 1, "total": 1, @@ -645,6 +704,28 @@ class RunLoopTests(_FakeTui, unittest.TestCase): self.assertIn("Full details in the log file: /tmp/runs/a.log", mk_notice.call_args[0][0]) + def test_stop_and_exit_boot_failure_names_reason_hint_and_server_log(self): + # A run that dies in the boot phase must not summarize as a bare + # "No books were converted": the reason, the known-crash hint, + # and the server's own log path all land in the summary. + with patch.object(runview.servers, "stop"), \ + patch.object(runview.common, + "record_post_tui_notice") as mk_notice: + view, screen = self.make_view([], stop_and_exit=True, + log_path="/tmp/runs/a.log") + view._queue.put({"kind": "starting", "name": "sglomni", + "log_path": "/tmp/sglomni-server.log"}) + view._queue.put({"kind": "exited", "name": "sglomni", + "returncode": 1, "log_tail": [], + "hint": "FP8 needs compute capability 8.9+"}) + view.run() + text = mk_notice.call_args[0][0] + self.assertIn("No books were converted", text) + self.assertIn("Failure: server exited with code 1", text) + self.assertIn("hint: FP8 needs compute capability 8.9+", text) + self.assertIn("server log: /tmp/sglomni-server.log", text) + self.assertIn("Full details in the log file: /tmp/runs/a.log", text) + class WorkerTests(_FakeTui, unittest.TestCase): """The worker thread's handoff into audiobook.convert.""" |