Skip to content

Commit 08e8e1c

Browse files
committed
Fix tests after introducing inner transactions
1 parent b55a276 commit 08e8e1c

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

tests/application_client/command_sender.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,14 +113,15 @@ def sign_data(
113113

114114
@contextmanager
115115
def sign_tx(
116-
self, account_number: int, network_id: bytes, transaction: Transaction
116+
self, account_number: int, inner_tx: bool, network_id: bytes, transaction: Transaction
117117
) -> Generator[None, None, None]:
118118
tx_rlp = rlp.encode(Transaction.serialize(transaction))
119119
with self.backend.exchange_async(
120120
cla=CLA,
121121
ins=InsType.SIGN_TX,
122122
data=account_number.to_bytes(4, "big")
123123
+ len(tx_rlp).to_bytes(4, "big")
124+
+ inner_tx.to_bytes(1, "big")
124125
+ len(network_id).to_bytes(1, "big")
125126
+ network_id
126127
+ tx_rlp,

tests/test_sign_tx.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,10 @@ def run_sign_tx(backend, scenario_navigator, tx, approve=True, inner_tx=False):
114114
data_to_sign = network_id + blake2b_256.digest()
115115

116116
with client.sign_tx(
117-
account_number=account_number, network_id=network_id, transaction=tx
117+
account_number=account_number,
118+
inner_tx=inner_tx,
119+
network_id=network_id,
120+
transaction=tx,
118121
):
119122
scenario_navigator.review_approve()
120123

@@ -126,7 +129,10 @@ def run_sign_tx(backend, scenario_navigator, tx, approve=True, inner_tx=False):
126129
else:
127130
with pytest.raises(ExceptionRAPDU) as e:
128131
with client.sign_tx(
129-
account_number=account_number, network_id=network_id, transaction=tx
132+
account_number=account_number,
133+
inner_tx=inner_tx,
134+
network_id=network_id,
135+
transaction=tx,
130136
):
131137
scenario_navigator.review_reject()
132138

0 commit comments

Comments
 (0)