fix(core): skip bitcoin.signtx_decred test on BTC_ONLY
What the developer wrote
Authored by M1nd3r
fix(core): skip bitcoin.signtx_decred test on BTC_ONLY
[no changelog]
What changed, and why it matters
This commit only changes a test file so that a Decred signing test is skipped when the firmware is built in Bitcoin-only mode. It does not modify any production firmware code, wallet behavior, or cryptographic logic. There is no security issue here.
No security action needed. Treat as a routine test-maintenance commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adjusts the guard condition in core/tests/test_apps.bitcoin.signtx_decred.py. Previously the test ran only on the T2T1 model; now it also requires that the build is not BITCOIN_ONLY. This is a test-suite compatibility fix for builds that exclude altcoin code, not a product security patch.
Changed components
core/tests/test_apps.bitcoin.signtx_decred.pyInspect captured patch +4 / −1
diff --git a/core/tests/test_apps.bitcoin.signtx_decred.py b/core/tests/test_apps.bitcoin.signtx_decred.py
index 0e6129eb..fda79355 100644
--- a/core/tests/test_apps.bitcoin.signtx_decred.py
+++ b/core/tests/test_apps.bitcoin.signtx_decred.py
@@ -1,7 +1,10 @@
# flake8: noqa: F403,F405
from common import * # isort:skip
-if utils.INTERNAL_MODEL in ("T2T1",): # pylint: disable=internal-model-tuple-comparison
+if (
+ utils.INTERNAL_MODEL in ("T2T1",) # pylint: disable=internal-model-tuple-comparison
+ and not utils.BITCOIN_ONLY
+):
from trezor.crypto import bip39
from trezor.enums import AmountUnit, OutputScriptType
from trezor.enums.RequestType import TXFINISHED, TXINPUT, TXMETA, TXOUTPUT
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.