What the developer wrote
Authored by obrusvit
test: add emulator marker
- adds pytest.mark.emulator for tests which are only executable on the
emulator
[no changelog]
What changed, and why it matters
This commit only adds a new test marker called 'emulator' so that certain automated tests can be skipped when running on real Trezor hardware instead of the software emulator. It does not change any wallet, firmware, or security code that end users rely on.
No security action needed; this is a benign test-framework change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change registers a new pytest marker ‘emulator’ and updates conftest.py to skip tests carrying that marker unless the test context is running against the emulator. It is purely a test-infrastructure change with no functional or security impact on the firmware.
Changed components
tests/REGISTERED_MARKERStests/conftest.pyInspect captured patch +5 / −1
diff --git a/tests/REGISTERED_MARKERS b/tests/REGISTERED_MARKERS
index 24d610cb..841a6d03 100644
--- a/tests/REGISTERED_MARKERS
+++ b/tests/REGISTERED_MARKERS
@@ -1,7 +1,7 @@
altcoin
-slow
cardano
decred
+emulator
eos
ethereum
komodo
@@ -13,6 +13,7 @@ peercoin
protocol
ripple
sd_card
+slow
solana
stellar
tezos
diff --git a/tests/conftest.py b/tests/conftest.py
index 5c5122c9..a15202c2 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -335,6 +335,9 @@ def _prepared_test_ctx(
if _raw_test_ctx.model not in models_filter:
pytest.skip(f"Skipping test for model {_raw_test_ctx.model.internal_name}")
+ if request.node.get_closest_marker("emulator") and not _raw_test_ctx.is_emulator:
+ pytest.skip("Skipping emulator-only test")
+
is_btc_only = messages.Capability.Bitcoin_like not in _raw_test_ctx.capabilities
if request.node.get_closest_marker("altcoin") and is_btc_only:
pytest.skip("Skipping altcoin test")
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.