What changed, and why it matters
This commit only updates a test script that navigates the Ledger device's on-screen dashboard. It adds extra button presses to account for a new 'App settings' menu item that appears before 'App info'. There is no change to the actual Bitcoin app code, cryptographic logic, or transaction signing behavior, so this commit does not introduce or fix a security issue by itself.
No security action required; treat as a routine test-maintenance commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies tests_mainnet/test_dashboard.py to adjust the Speculos-based UI automation for the dashboard. Because a new ‘App settings’ screen was added upstream, the test now presses right once more to reach ‘App info’ and presses left once more to return to ‘Bitcoin’. This is purely a test synchronization change with no functional code impact.
Changed components
tests_mainnet/test_dashboard.pyInspect captured patch +4 / −0
diff --git a/tests_mainnet/test_dashboard.py b/tests_mainnet/test_dashboard.py
index a6a6a54..8ced1b6 100644
--- a/tests_mainnet/test_dashboard.py
+++ b/tests_mainnet/test_dashboard.py
@@ -12,12 +12,16 @@ def test_dashboard(comm: SpeculosClient, is_speculos: bool, app_version: str, mo
if model in ["stax", "flex", "apex_p"]:
pytest.skip("No dashboard test for stax, flex and apex_p")
+ comm.press_and_release("right")
+ comm.wait_for_text_event("App settings")
+
comm.press_and_release("right")
comm.wait_for_text_event("App info")
comm.press_and_release("right")
comm.wait_for_text_event("Quit app")
+ comm.press_and_release("left")
comm.press_and_release("left")
comm.press_and_release("left")
comm.wait_for_text_event("Bitcoin")
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.