Roman Zeyde
Roman Zeyde
This issue also covers transaction signature flows.
Re-opening since we want to keep the confirmation layout open for a few seconds (#5097), even if the client sends a message immediately after the response (currently, it closes the...
`coverage` doesn't ignore optimized out statements (e.g. using `if utils.USE_POWER_MANAGER` guard): [src_apps_common_lock_manager_py.html](https://github.com/user-attachments/files/22737523/src_apps_common_lock_manager_py.html)
Thinking about it, we should probably send the `Cancelled` error as part of the recovery workflow - so the homescreen will start only after THP ACK has been successfully recieved...
Reproduction steps for https://github.com/trezor/trezor-suite/issues/22878 (tested with recent Suite Desktop on Linux[^1] and recent FW[^2]) ``` $ TREZOR_DISABLE_ANIMATION=0 BOOTLOADER_DEVEL=1 BOOTLOADER_QA=1 TREZOR_MODEL=T3W1 QUIET_MODE=1 PYOPT=0 make -C core build_firmware upload $ trezorctl -v...
Tested on T3W1 with with Protocol v1 (using `THP=0`) on dc7deec (with the DEBUG level logging disabled): ``` $ BOOTLOADER_DEVEL=1 BOOTLOADER_QA=1 QUIET_MODE=1 BITCOIN_ONLY=0 TREZOR_MODEL=T3W1 PYOPT=0 THP=0 make -C core clean...
Longest T3W1 device tests: ``` Oct 31 03:25:41 1121.44s call tests/device_tests/bitcoin/test_signtx.py::test_lots_of_inputs Oct 31 03:25:41 281.99s call tests/device_tests/reset_recovery/test_reset_recovery_slip39_advanced.py::test_reset_recovery Oct 31 03:25:41 245.92s call tests/device_tests/webauthn/test_msg_webauthn.py::test_add_remove Oct 31 03:25:41 170.57s call tests/device_tests/reset_recovery/test_reset_recovery_slip39_basic.py::test_reset_recovery Oct...
Running `pytest -v tests/device_tests/bitcoin/test_signtx.py::test_lots_of_inputs --durations 0` on https://github.com/trezor/trezor-firmware/compare/romanz/no-acks (without DEBUG logs): ``` 813.54s call tests/device_tests/bitcoin/test_signtx.py::test_lots_of_inputs 13.38s setup tests/device_tests/bitcoin/test_signtx.py::test_lots_of_inputs 0.53s teardown tests/device_tests/bitcoin/test_signtx.py::test_lots_of_inputs = 1 passed in 827.92s (0:13:47) = ``` So...
Rebased over recent `main`, to include: - #6022 - #6024 - #5997
Debug log (INFO level) ``` 7.010 trezor.ui.layouts.common INFO ButtonRequest sent: thp_pairing_request 7.012 trezor.wire.thp.channel INFO [USBIF] (cid: 3a02) write message: ButtonRequest 7.066 trezor.ui.layouts.common INFO ButtonRequest acked: thp_pairing_request 7.075 trezor.wire.thp.channel INFO [USBIF]...