From 4d559d03391b85e65704ee537044679c6b11f44f Mon Sep 17 00:00:00 2001 From: thePR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com> Date: Mon, 14 Jul 2025 22:34:04 -0400 Subject: [PATCH] pause after showing entry details --- src/password_manager/manager.py | 3 ++- src/tests/test_manager_list_entries.py | 12 +++++++++--- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/src/password_manager/manager.py b/src/password_manager/manager.py index 516f820..0c2eafb 100644 --- a/src/password_manager/manager.py +++ b/src/password_manager/manager.py @@ -1792,7 +1792,7 @@ class PasswordManager: pause() def show_entry_details_by_index(self, index: int) -> None: - """Display entry details for ``index`` without prompting.""" + """Display details for entry ``index`` and offer actions.""" try: entry = self.entry_manager.retrieve_entry(index) if not entry: @@ -1808,6 +1808,7 @@ class PasswordManager: ) self.display_entry_details(index) + pause() self._entry_actions_menu(index, entry) except Exception as e: logging.error(f"Failed to display entry details: {e}", exc_info=True) diff --git a/src/tests/test_manager_list_entries.py b/src/tests/test_manager_list_entries.py index f309deb..bc3e762 100644 --- a/src/tests/test_manager_list_entries.py +++ b/src/tests/test_manager_list_entries.py @@ -116,15 +116,21 @@ def test_show_entry_details_by_index(monkeypatch): "password_manager.manager.clear_header_with_notification", lambda *a, **k: header_calls.append(True), ) - action_calls = [] + + call_order = [] + monkeypatch.setattr( + pm, + "display_entry_details", + lambda *a, **k: call_order.append("display"), + ) monkeypatch.setattr( pm, "_entry_actions_menu", - lambda *a, **k: action_calls.append(True), + lambda *a, **k: call_order.append("actions"), ) monkeypatch.setattr("password_manager.manager.pause", lambda *a, **k: None) pm.show_entry_details_by_index(index) assert len(header_calls) == 1 - assert len(action_calls) == 1 + assert call_order == ["display", "actions"]