tests: allow qemu tests to run longer
What the developer wrote
Authored by Jon Griffiths
tests: allow qemu tests to run longer
What changed, and why it matters
This commit only changes a test script. It increases the maximum allowed runtime from 60 minutes to 90 minutes when tests are run inside a QEMU emulator. There is no change to the actual Jade wallet firmware or any security-sensitive code.
No security action needed. Treat as routine test maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies a single line in test_jade.py’s check_stuck() helper. The watchdog timeout is now conditional: 90 minutes when args.qemu is true, otherwise 60 minutes. This is purely a test infrastructure tweak to accommodate slower QEMU-based test execution. No production code, cryptographic logic, or attack surface is affected.
Changed components
test_jade.py (test infrastructure only)Inspect captured patch +1 / −1
diff --git a/test_jade.py b/test_jade.py
index 2bb043e..37a790b 100644
--- a/test_jade.py
+++ b/test_jade.py
@@ -4211,7 +4211,7 @@ def test_ble_connection_fails(info):
def check_stuck():
# NOTE: belt'n'braces - serial/ble reads/writes should timeout before this does
- timeout = 60 # minutes
+ timeout = 90 if args.qemu else 60 # minutes
time.sleep(60 * timeout)
logger.error(f'tests got caught running longer than {timeout} minutes, terminating')
logger.handlers[0].flush()
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.