thePR0M3TH3AN/seedPass
1
0
Fork 0
mirror of https://github.com/PR0M3TH3AN/SeedPass.git synced 2025-09-09 15:58:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #559 from PR0M3TH3AN/qechl7-codex/update-show_entry_details_by_index-method

Browse Source 
Add pause between entry details and action menu
This commit is contained in:
thePR0M3TH3AN
2025-07-14 22:36:03 -04:00
committed by GitHub
parent bdf83fabd8 4d559d0339
commit b8e6ae3e36
2 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/password_manager/manager.py
View File

@@ -1792,7 +1792,7 @@ class PasswordManager:
pause() pause()
def show_entry_details_by_index(self, index: int) -> None: 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: try:
entry = self.entry_manager.retrieve_entry(index) entry = self.entry_manager.retrieve_entry(index)
if not entry: if not entry:
@@ -1808,6 +1808,7 @@ class PasswordManager:
) )
self.display_entry_details(index) self.display_entry_details(index)
pause()
self._entry_actions_menu(index, entry) self._entry_actions_menu(index, entry)
except Exception as e: except Exception as e:
logging.error(f"Failed to display entry details: {e}", exc_info=True) logging.error(f"Failed to display entry details: {e}", exc_info=True)

12
src/tests/test_manager_list_entries.py
View File

@@ -116,15 +116,21 @@ def test_show_entry_details_by_index(monkeypatch):
"password_manager.manager.clear_header_with_notification", "password_manager.manager.clear_header_with_notification",
lambda *a, **k: header_calls.append(True), 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( monkeypatch.setattr(
pm, pm,
"_entry_actions_menu", "_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) monkeypatch.setattr("password_manager.manager.pause", lambda *a, **k: None)
pm.show_entry_details_by_index(index) pm.show_entry_details_by_index(index)
assert len(header_calls) == 1 assert len(header_calls) == 1
assert len(action_calls) == 1 assert call_order == ["display", "actions"]