chore: add reset and recovery context tests

Trezor/Trezor firmware

3039c42e View source ↗

Public commit record

What the developer wrote

Authored by Michal Kazda

67/100 · Adequate

chore: add reset and recovery context tests

[no changelog]

✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification! No meaningful explanatory body

The short version

What changed, and why it matters

This commit only updates a list of automated test cases in a text file. It removes two PIN-related tests and adds seventeen tests covering wallet reset and recovery scenarios. There are no code changes to the firmware or any user-facing behavior.

Recommended action

No security action required. Treat as routine test-suite maintenance.

Security signals we found

No strong security signals were identified.

Technical analysis

Evidence from the diff

The diff modifies tests/context_tests.txt, which appears to be a manifest of tests to run in a particular context (likely CI or a test suite selector). It replaces two PIN test references with reset/recovery test references. No source code, logic, or configuration affecting device operation is changed.

Changed components

tests/context_tests.txt

Inspect captured patch +19 / −2

diff --git a/tests/context_tests.txt b/tests/context_tests.txt
index 4e6eaea4..8dc5060a 100644
--- a/tests/context_tests.txt
+++ b/tests/context_tests.txt
@@ -1,2 +1,19 @@
-../tests/device_tests/test_pin.py::test_correct_pin
-../tests/click_tests/test_pin.py::test_pin_short
+../tests/device_tests/reset_recovery/test_recovery_bip39_dryrun.py::test_dry_run
+../tests/device_tests/reset_recovery/test_recovery_bip39_dryrun.py::test_invalid_seed_core
+../tests/device_tests/reset_recovery/test_recovery_bip39_dryrun.py::test_seed_mismatch
+../tests/device_tests/reset_recovery/test_recovery_bip39_t2.py::test_tt_nopin_nopassphrase
+../tests/device_tests/reset_recovery/test_recovery_bip39_t2.py::test_tt_pin_passphrase
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic.py::test_1of1
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic.py::test_abort
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic.py::test_abort_between_shares
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic.py::test_invalid_mnemonic_first_share
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic.py::test_noabort
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic.py::test_same_share
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic.py::test_wrong_nth_word[0]
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic_dryrun.py::test_2of3_dryrun
+../tests/device_tests/reset_recovery/test_recovery_slip39_basic_dryrun.py::test_2of3_invalid_seed_dryrun
+../tests/device_tests/reset_recovery/test_reset_backup.py::test_skip_backup_manual
+../tests/device_tests/reset_recovery/test_reset_slip39_basic.py::test_reset_device_slip39_basic
+../tests/device_tests/reset_recovery/test_reset_bip39_t2.py::test_reset_failed_check
+../tests/device_tests/bitcoin/test_getaddress_show.py::test_show_tt
+../tests/device_tests/bitcoin/test_signtx_segwit_native.py::test_send_both

Risk score

Why this scored 15/100

Our methodology →

Potential impact 0/30

Exploitability 0/25

Stealth signal 0/15

Affected reach 0/15

Confidence 10/10

Evidence quality 5/5

Human-validated context

Community notes

Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.

No validated notes yet.

The AI analysis stands alone for now. Submit a note if you can add evidence or important context.

What the developer wrote

What changed, and why it matters

Security signals we found

Evidence from the diff

Changed components

Why this scored 15/100

Community notes

Related watches

refactor(core): digest EIP-1559/EIP-7702 magic without RLP encoding

test(core): test RLP self-encoding bytestrings

chore(core): remove sha1 from regular FW