fix(tests): ignore spaces on paginated content
What the developer wrote
Authored by Ioan Bizău
fix(tests): ignore spaces on paginated content
[no changelog]
What changed, and why it matters
This is a tiny change to a single test file. It makes a test more tolerant of how spaces appear in on-screen text by ignoring spaces when comparing expected and actual text. There is no change to the actual Trezor firmware or wallet behavior, and no security issue is present.
No action needed. This is a benign test-only change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies tests/click_tests/test_tutorial_eckhart.py. The assertion that checks whether translated tutorial text appears on a paginated screen now strips spaces from both the expected string and the captured screen text before comparing. This is purely a test robustness fix for layout rendering differences and does not touch firmware code, cryptography, communication, or user assets.
Changed components
tests/click_tests/test_tutorial_eckhart.pyInspect captured patch +1 / −1
diff --git a/tests/click_tests/test_tutorial_eckhart.py b/tests/click_tests/test_tutorial_eckhart.py
index f8b9c6c68..f2710ddb5 100644
--- a/tests/click_tests/test_tutorial_eckhart.py
+++ b/tests/click_tests/test_tutorial_eckhart.py
@@ -51,7 +51,7 @@ def _assert_tropic_info(debug: "DebugLink"):
layout = debug.read_layout()
assert layout.title() == TR.tutorial__what_is_tropic
text = _paginated_text(debug)
- assert TR.tutorial__tropic_info in text
+ assert TR.tutorial__tropic_info.replace(" ", "") in text.replace(" ", "")
debug.click(debug.screen_buttons.menu())
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.