Skip to content

Commit 113bc5f

Browse files
committed
Apply code review suggestions
1 parent c691cb1 commit 113bc5f

File tree

3 files changed

+24
-54
lines changed

3 files changed

+24
-54
lines changed

crates/cheatnet/src/runtime_extensions/forge_runtime_extension/cheatcodes/mod.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ pub mod replace_bytecode;
2020
pub mod spy_events;
2121
pub mod spy_messages_to_l1;
2222
pub mod storage;
23-
pub mod testing;
2423

2524
/// A structure used for returning cheatcode errors in tests
2625
#[derive(Debug)]

crates/cheatnet/src/runtime_extensions/forge_runtime_extension/cheatcodes/testing.rs

Lines changed: 0 additions & 52 deletions
This file was deleted.

crates/cheatnet/src/runtime_extensions/forge_runtime_extension/mod.rs

Lines changed: 24 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ use self::contracts_data::ContractsData;
22
use crate::runtime_extensions::call_to_blockifier_runtime_extension::rpc::UsedResources;
33
use crate::runtime_extensions::common::sum_syscall_usage;
44
use crate::runtime_extensions::forge_runtime_extension::cheatcodes::replace_bytecode::ReplaceBytecodeError;
5-
use crate::runtime_extensions::forge_runtime_extension::cheatcodes::testing::calculate_steps_from_calls;
65
use crate::runtime_extensions::{
76
call_to_blockifier_runtime_extension::{
87
CallToBlockifierRuntime,
@@ -20,6 +19,7 @@ use crate::runtime_extensions::{
2019
};
2120
use crate::trace_data::{CallTrace, CallTraceNode, GasReportData};
2221
use anyhow::{Context, Result, anyhow};
22+
use blockifier::blockifier_versioned_constants::VersionedConstants;
2323
use blockifier::bouncer::vm_resources_to_sierra_gas;
2424
use blockifier::context::TransactionContext;
2525
use blockifier::execution::call_info::{
@@ -881,3 +881,26 @@ pub fn get_all_used_resources(
881881
l1_handler_payload_lengths,
882882
}
883883
}
884+
885+
fn calculate_steps_from_calls(
886+
top_call: &Rc<RefCell<CallTrace>>,
887+
top_call_syscalls: &SyscallUsageMap,
888+
) -> usize {
889+
// Resources from inner calls already include syscall resources used in them
890+
let used_resources =
891+
&top_call
892+
.borrow()
893+
.nested_calls
894+
.iter()
895+
.fold(ExecutionResources::default(), |acc, node| match node {
896+
CallTraceNode::EntryPointCall(call_trace) => {
897+
&acc + &call_trace.borrow().used_execution_resources
898+
}
899+
CallTraceNode::DeployWithoutConstructor => acc,
900+
});
901+
let total_syscalls_exeucution_resources = &VersionedConstants::latest_constants()
902+
.get_additional_os_syscall_resources(&top_call_syscalls);
903+
let resources_from_calls = used_resources + total_syscalls_exeucution_resources;
904+
905+
resources_from_calls.n_steps
906+
}

0 commit comments

Comments
 (0)