test(core): use capability name when skipping a test
What the developer wrote
Authored by Roman Zeyde
test(core): use capability name when skipping a test
Previously, it was logging capability index instead.
[no changelog]
What changed, and why it matters
This is a tiny test-only change that makes a skip message more readable. Instead of printing a numeric capability index, it now prints the capability's human-readable name. It does not touch any wallet firmware, cryptography, or device behavior, and has no security relevance.
No security action needed; this is a cosmetic test-logging improvement.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In tests/conftest.py, the pytest.skip() message for missing device capabilities is changed from f”Missing {capability}” (which stringifies an enum member to its numeric value/index) to f”Missing {capability.name}” (which prints the enum member’s name). This affects only test logging output.
Changed components
tests/conftest.pyInspect captured patch +1 / −1
diff --git a/tests/conftest.py b/tests/conftest.py
index 29190a4a..d39c4326 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -661,6 +661,6 @@ def backup_method(request, _raw_test_ctx: TrezorTestContext) -> messages.BackupM
method: messages.BackupMethod = request.param
if (capability := REQUIRED_CAPABILITY.get(method)) is not None:
if capability not in _raw_test_ctx.capabilities:
- pytest.skip(f"Missing {capability}")
+ pytest.skip(f"Missing {capability.name}")
return method
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.