test(stellar): remove signer using StellarSetOptionsOp
What the developer wrote
Authored by Jakub Janků
test(stellar): remove signer using StellarSetOptionsOp
[no changelog]
(cherry picked from commit fd5e83f9c32e3ec3e41341e1091a7fbedb2cc767)
What changed, and why it matters
This commit only adds a new test case for the Stellar cryptocurrency feature. It records an expected transaction signature for a 'remove signer' operation so the device can be tested against a known-good value. There is no code change, no bug fix, and no security-relevant behavior change.
No action needed. This is a benign test-data addition.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds one JSON fixture in common/tests/fixtures/stellar/sign_tx.json. The fixture exercises StellarSetOptionsOp with signer_weight: 0, which is the standard way to remove a signer on Stellar. It includes the XDR blob, derivation path, network passphrase, parsed transaction fields, operation details, and the expected public key/signature. No firmware or test logic is modified.
Changed components
common/tests/fixtures/stellar/sign_tx.jsonInspect captured patch +28 / −0
### common/tests/fixtures/stellar/sign_tx.json
@@ -803,6 +803,34 @@
"signature": "HgJas9f4I/o1gneU6+t7fhfdFMttSO57a8P/1wgMQ87ySt3WW/RQvFqqTX+evXmLFb/XZ5A9K3tcw1WLBhSdBQ=="
}
},
+ {
+ "name": "StellarSetOptionsOp-remove-signer",
+ "parameters": {
+ "xdr": "AAAAAgAAAAAvIrnGLwi3dPPr5t1ufbk8PsLL3gJ5Vho9nFIluMMikgAAAGQAAAAAAAAD6AAAAAEAAAAAG4J3zQAAAABd5CqEAAAAAAAAAAEAAAAAAAAABQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAQAAAAJyGHrbh5xBQ0bXfHGvjM57bqpXtSjpmf2R/q5rZBhijgAAAAAAAAAAAAAAAA==",
+ "address_n": "m/44'/148'/0'",
+ "network_passphrase": "Test SDF Network ; September 2015",
+ "tx": {
+ "source_account": "GAXSFOOGF4ELO5HT5PTN23T5XE6D5QWL3YBHSVQ2HWOFEJNYYMRJENBV",
+ "fee": 100,
+ "sequence_number": 1000,
+ "timebounds_start": 461535181,
+ "timebounds_end": 1575234180,
+ "memo_type": "NONE"
+ },
+ "operations": [
+ {
+ "_message_type": "StellarSetOptionsOp",
+ "signer_type": "HASH",
+ "signer_key": "72187adb879c414346d77c71af8cce7b6eaa57b528e999fd91feae6b6418628e",
+ "signer_weight": 0
+ }
+ ]
+ },
+ "result": {
+ "public_key": "2f22b9c62f08b774f3ebe6dd6e7db93c3ec2cbde0279561a3d9c5225b8c32292",
+ "signature": "G5x2PBOw79NkpkWyS5DMfpS7TKcXzPhzjccMJgKmTG12MXWdU66XELqxPVXuoiKIo27H1Zqc8XIi8VpdRkpOBQ=="
+ }
+ },
{
"name": "timebounds-0-0",
"parameters": {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.