From b8addc87f804789368612bf8e1be31a2bf28efe1 Mon Sep 17 00:00:00 2001 From: cirakg Date: Sun, 25 Oct 2026 23:51:52 +0100 Subject: test(ui): second UI scenario, and the fixes it found A second UI-test build (uitest-extra.jungle) covers the legacy PIN and the forced new PIN, Change and Reset PIN, a queued Require PIN, UPC-A, Code 39, the code error screen, the encoding progress, auto-lock on a barcode, the light theme and the glance without a code; the main flow adds the Steam and Library details and the menu's text screens. Fixed from what it found: Entry IDs tripped the watchdog with 110 entries; auto-lock didn't close a locked barcode; a queued Require PIN showed "Confirm in menu"; locked barcodes lacked the padlock; the PIN keypad's bottom row, the Reset PIN question, text-screen bottoms, the glance title and the light-theme warning colour. Goldens approved on all seven watches; only the Forerunner 70 was re-verified after the version bump. Co-Authored-By: Claude Opus 5.5 --- source/ui/DebugMenu.mc | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'source/ui/DebugMenu.mc') diff --git a/source/ui/DebugMenu.mc b/source/ui/DebugMenu.mc index f1ab407..452cb1d 100644 --- a/source/ui/DebugMenu.mc +++ b/source/ui/DebugMenu.mc @@ -40,6 +40,8 @@ module DebugMenu { } } + const BENCH_HEADERS = 20; + // The running benchmark (kept so its timer isn't collected). var bench as BenchRunner? = null; @@ -74,12 +76,15 @@ module DebugMenu { } else if (_step == 2) { _lines.add("PIN derivation: " + Kdf.calibrate(1000, Kdf.ITERATIONS_PER_STEP) + " iterations per second"); } else if (_step == 3) { + // At most BENCH_HEADERS: all 110 of the UI test's entries in + // one event tripped the watchdog. var vault = AppState.theVault(); var ids = vault.ids(); - for (var i = 0; i < ids.size(); i++) { + var n = ids.size() < BENCH_HEADERS ? ids.size() : BENCH_HEADERS; + for (var i = 0; i < n; i++) { vault.header(ids[i]); } - _lines.add("Headers: " + ids.size() + " in " + (System.getTimer() - t) + " ms"); + _lines.add("Headers: " + n + " in " + (System.getTimer() - t) + " ms"); } else if (_step == 4) { SelfTest.run(); _lines.add("Self-test: " + (System.getTimer() - t) + " ms"); -- cgit v1.2.3