Skip to content

Commit 52fa263

Browse files
committed
fix: fix broken tests for CardanoTransactionSetProof
1 parent 35117be commit 52fa263

File tree

2 files changed

+22
-10
lines changed

2 files changed

+22
-10
lines changed

mithril-common/src/messages/cardano_transactions_proof.rs

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -350,13 +350,14 @@ mod tests {
350350
transactions: &[CardanoTransaction],
351351
immutable_file_number: u64,
352352
) -> ProtocolMessage {
353-
let transactions_hashes = transactions
354-
.iter()
355-
.map(|t| t.transaction_hash.clone())
356-
.collect::<Vec<_>>();
357-
358-
let proof = MKProof::from_leaves(transactions).unwrap();
359-
let set_proof = CardanoTransactionsSetProof::new(transactions_hashes, proof);
353+
let set_proof = CardanoTransactionsSetProof::from_leaves(
354+
transactions
355+
.iter()
356+
.map(|t| (t.block_number, t.transaction_hash.clone()))
357+
.collect::<Vec<_>>()
358+
.as_slice(),
359+
)
360+
.unwrap();
360361

361362
let verified_transactions_fake = VerifiedCardanoTransactions {
362363
certificate_hash: "whatever".to_string(),
@@ -378,8 +379,14 @@ mod tests {
378379
let mut mock_transaction_store = MockTransactionStore::new();
379380
mock_transaction_store
380381
.expect_store_transactions()
381-
.returning(|_| Ok(()));
382-
let mock_transaction_retriever = MockTransactionRetriever::new();
382+
.returning(|_| Ok(()))
383+
.times(1);
384+
let transaction_retrieved = transactions.to_vec();
385+
let mut mock_transaction_retriever = MockTransactionRetriever::new();
386+
mock_transaction_retriever
387+
.expect_get_up_to()
388+
.return_once(|_| Ok(transaction_retrieved))
389+
.times(1);
383390

384391
let cardano_transaction_signable_builder = CardanoTransactionsSignableBuilder::new(
385392
Arc::new(DumbTransactionParser::new(transactions.to_vec())),

mithril-common/src/signable_builder/cardano_transactions.rs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,12 @@ mod tests {
299299
.expect_store_transactions()
300300
.returning(|_| Ok(()));
301301
let transaction_store = Arc::new(mock_transaction_store);
302-
let mock_transaction_retriever = MockTransactionRetriever::new();
302+
let transactions_retrieved = transactions.clone();
303+
let mut mock_transaction_retriever = MockTransactionRetriever::new();
304+
mock_transaction_retriever
305+
.expect_get_up_to()
306+
.return_once(|_| Ok(transactions_retrieved))
307+
.times(1);
303308
let transaction_retriever = Arc::new(mock_transaction_retriever);
304309
let cardano_transactions_signable_builder = CardanoTransactionsSignableBuilder::new(
305310
transaction_parser,

0 commit comments

Comments
 (0)