test(core/ethereum): don't duplicate "skip_models" for EIP-7702 tests
What the developer wrote
Authored by Roman Zeyde
test(core/ethereum): don't duplicate "skip_models" for EIP-7702 tests
It will be supported on T2T1.
[no changelog]
What changed, and why it matters
This commit only changes a test data file. It removes duplicate 'skip_models' entries from Ethereum EIP-7702 test fixtures so that the T2T1 model is no longer excluded from these tests. There is no change to firmware code, no security fix, and no user-facing behavior change.
No security action needed. Treat as routine test maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies common/tests/fixtures/ethereum/sign_auth_eip7702.json, deleting eight ‘skip_models’: [‘t1’, ‘t2t1’] lines. The commit message states EIP-7702 will be supported on T2T1. This is purely a test configuration cleanup; it does not alter production code, cryptography, parsing, or device behavior.
Changed components
common/tests/fixtures/ethereum/sign_auth_eip7702.jsonInspect captured patch +0 / −8
diff --git a/common/tests/fixtures/ethereum/sign_auth_eip7702.json b/common/tests/fixtures/ethereum/sign_auth_eip7702.json
index 1b760786..7852dd63 100644
--- a/common/tests/fixtures/ethereum/sign_auth_eip7702.json
+++ b/common/tests/fixtures/ethereum/sign_auth_eip7702.json
@@ -6,7 +6,6 @@
"tests": [
{
"name": "ambire_nonce_1_chainid_0_all_networks",
- "skip_models": ["t1", "t2t1"],
"parameters": {
"path": "m/44'/60'/0'/0/0",
"delegate": "0x5A7fC11397E9A8AD41Bf10bF13f22b0A63F96f6D",
@@ -21,7 +20,6 @@
},
{
"name": "ambire_nonce_1_chainid_1_mainnet",
- "skip_models": ["t1", "t2t1"],
"parameters": {
"path": "m/44'/60'/0'/0/0",
"delegate": "0x5A7fC11397E9A8AD41Bf10bF13f22b0A63F96f6D",
@@ -36,7 +34,6 @@
},
{
"name": "ambire_nonce_3_chainid_8453_base",
- "skip_models": ["t1", "t2t1"],
"parameters": {
"path": "m/44'/60'/0'/0/0",
"delegate": "0x5A7fC11397E9A8AD41Bf10bF13f22b0A63F96f6D",
@@ -51,7 +48,6 @@
},
{
"name": "ambire_nonce_2_chainid_42161_arbitrum",
- "skip_models": ["t1", "t2t1"],
"parameters": {
"path": "m/44'/60'/0'/0/0",
"delegate": "0x5A7fC11397E9A8AD41Bf10bF13f22b0A63F96f6D",
@@ -66,7 +62,6 @@
},
{
"name": "revoke_nonce_123456_chainid_0_all_networks",
- "skip_models": ["t1", "t2t1"],
"parameters": {
"path": "m/44'/60'/0'/0/0",
"delegate": "0x0000000000000000000000000000000000000000",
@@ -81,7 +76,6 @@
},
{
"name": "revoke_nonce_123456_chainid_1_mainnet",
- "skip_models": ["t1", "t2t1"],
"parameters": {
"path": "m/44'/60'/0'/0/0",
"delegate": "0x0000000000000000000000000000000000000000",
@@ -96,7 +90,6 @@
},
{
"name": "revoke_nonce_123456_chainid_8453_base",
- "skip_models": ["t1", "t2t1"],
"parameters": {
"path": "m/44'/60'/0'/0/0",
"delegate": "0x0000000000000000000000000000000000000000",
@@ -111,7 +104,6 @@
},
{
"name": "revoke_nonce_123456_chainid_42161_arbitrum",
- "skip_models": ["t1", "t2t1"],
"parameters": {
"path": "m/44'/60'/0'/0/0",
"delegate": "0x0000000000000000000000000000000000000000",
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.