Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

Commit 39585de

Browse files
fix message fee estimation (#811)
Co-authored-by: mmsc2 <88055861+mmsc2@users.noreply.github.com>
1 parent 0a73bb2 commit 39585de

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/lib.rs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ use crate::{
1818
use cairo_vm::felt::Felt252;
1919
use definitions::block_context::BlockContext;
2020
use state::cached_state::CachedState;
21-
use transaction::L1Handler;
21+
use transaction::{fee::calculate_tx_fee, L1Handler};
2222
use utils::Address;
2323

2424
#[cfg(test)]
@@ -171,9 +171,13 @@ where
171171

172172
// execute the transaction with the fake state.
173173
let transaction_result = l1_handler.execute(&mut cached_state, block_context, 1_000_000)?;
174+
let tx_fee = calculate_tx_fee(
175+
&transaction_result.actual_resources,
176+
block_context.starknet_os_config.gas_price,
177+
block_context,
178+
)?;
174179
if let Some(gas_usage) = transaction_result.actual_resources.get("l1_gas_usage") {
175-
let actual_fee = transaction_result.actual_fee;
176-
Ok((actual_fee, *gas_usage))
180+
Ok((tx_fee, *gas_usage))
177181
} else {
178182
Err(TransactionError::ResourcesError)
179183
}
@@ -374,7 +378,7 @@ mod test {
374378
block_context.starknet_os_config.gas_price = 1;
375379

376380
let estimated_fee = estimate_message_fee(&l1_handler, state, &block_context).unwrap();
377-
assert_eq!(estimated_fee, (0, 18471));
381+
assert_eq!(estimated_fee, (20081, 18471));
378382
}
379383

380384
#[test]

0 commit comments

Comments
 (0)