Skip to content

Commit 8a53c7a

Browse files
committed
Address nit comments
1 parent 300114d commit 8a53c7a

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

crates/forge-runner/src/gas/report.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,12 @@ type ContractName = String;
1010
type Selector = String;
1111

1212
#[derive(Debug, Clone)]
13-
pub struct GasSingleTestInfo {
13+
pub struct SingleTestGasInfo {
1414
pub gas_used: GasVector,
1515
pub report_data: ReportData,
1616
}
1717

18-
#[derive(Debug, Clone)]
18+
#[derive(Debug, Clone, Default)]
1919
pub struct ReportData(BTreeMap<ContractName, ContractInfo>);
2020

2121
#[derive(Debug, Clone, Default)]
@@ -31,12 +31,12 @@ pub struct SelectorReportData {
3131
pub records: Vec<u64>,
3232
}
3333

34-
impl GasSingleTestInfo {
34+
impl SingleTestGasInfo {
3535
#[must_use]
3636
pub fn new(gas_used: GasVector) -> Self {
3737
Self {
3838
gas_used,
39-
report_data: ReportData(BTreeMap::new()),
39+
report_data: ReportData::default(),
4040
}
4141
}
4242

crates/forge-runner/src/test_case_summary.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ use crate::build_trace_data::build_profiler_call_trace;
33
use crate::debugging::{TraceArgs, build_debugging_trace};
44
use crate::expected_result::{ExpectedPanicValue, ExpectedTestResult};
55
use crate::gas::check_available_gas;
6-
use crate::gas::report::GasSingleTestInfo;
6+
use crate::gas::report::SingleTestGasInfo;
77
use crate::gas::stats::GasStats;
88
use crate::package_tests::with_config_resolved::TestCaseWithResolvedConfig;
99
use crate::running::{RunCompleted, RunStatus};
@@ -84,7 +84,7 @@ impl TestType for Fuzzing {
8484
#[derive(Debug, PartialEq, Clone)]
8585
pub struct Single;
8686
impl TestType for Single {
87-
type GasInfo = GasSingleTestInfo;
87+
type GasInfo = SingleTestGasInfo;
8888
type TestStatistics = ();
8989
type TraceData = VersionedProfilerCallTrace;
9090
}
@@ -303,13 +303,13 @@ impl TestCaseSummary<Single> {
303303
);
304304

305305
let gas_info = if gas_report_enabled {
306-
GasSingleTestInfo::new_with_report(
306+
SingleTestGasInfo::new_with_report(
307307
gas_used,
308308
&call_trace.borrow(),
309309
&ContractsDataStore::new(contracts_data, &fork_data),
310310
)
311311
} else {
312-
GasSingleTestInfo::new(gas_used)
312+
SingleTestGasInfo::new(gas_used)
313313
};
314314

315315
match status {

0 commit comments

Comments
 (0)