What the developer wrote
Authored by obrusvit
test(core): fix failing click test
What changed, and why it matters
This is a one-line change to a test file. It loosens a test assertion so that the homescreen check passes when the model name appears anywhere on screen, rather than requiring the screen content to be exactly the model name. It does not change any firmware code or device behavior.
No security action needed. Treat as a normal test-maintenance commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies tests/click_tests/device_menu/test_label.py, changing an exact equality assertion to a substring check: assert debug.read_layout().screen_content() == "Trezor Safe 7" becomes assert "Trezor Safe 7" in debug.read_layout().screen_content(). This is purely a test-suite adjustment for a failing UI click test and has no effect on the Trezor firmware runtime or security logic.
Changed components
tests/click_tests/device_menu/test_label.pyInspect captured patch +1 / −1
diff --git a/tests/click_tests/device_menu/test_label.py b/tests/click_tests/device_menu/test_label.py
index 9e95b034..1a70d012 100644
--- a/tests/click_tests/device_menu/test_label.py
+++ b/tests/click_tests/device_menu/test_label.py
@@ -188,7 +188,7 @@ def test_label_empty(device_handler: "BackgroundDeviceHandler"):
assert features.label == ""
# When the label is empty, the homescreen shows the model name
- assert debug.read_layout().screen_content() == "Trezor Safe 7"
+ assert "Trezor Safe 7" in debug.read_layout().screen_content()
@pytest.mark.setup_client(pin=None)
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.