Skip to content

Commit

Permalink
Run forge tests with cairo steps explicitly set in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
THenry14 committed Mar 6, 2025
1 parent 56e4694 commit 393a3c5
Show file tree
Hide file tree
Showing 36 changed files with 213 additions and 175 deletions.
7 changes: 5 additions & 2 deletions crates/forge/test_utils/src/running_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,10 @@ use tempfile::tempdir;
use tokio::runtime::Runtime;

#[must_use]
pub fn run_test_case(test: &TestCase) -> Vec<TestTargetSummary> {
pub fn run_test_case(
test: &TestCase,
tracked_resource: ForgeTrackedResource,
) -> Vec<TestTargetSummary> {
ScarbCommand::new_with_stdio()
.current_dir(test.path().unwrap())
.arg("build")
Expand Down Expand Up @@ -66,7 +69,7 @@ pub fn run_test_case(test: &TestCase) -> Vec<TestTargetSummary> {
.unwrap()
.join(CACHE_DIR),
contracts_data: ContractsData::try_from(test.contracts().unwrap()).unwrap(),
tracked_resource: ForgeTrackedResource::CairoSteps,
tracked_resource,
environment_variables: test.env().clone(),
}),
output_config: Arc::new(OutputConfig {
Expand Down
7 changes: 4 additions & 3 deletions crates/forge/tests/integration/available_gas.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use test_utils::runner::{assert_case_output_contains, assert_failed, assert_passed};
use test_utils::running_tests::run_test_case;
Expand All @@ -14,7 +15,7 @@ fn correct_available_gas() {
"
));

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand All @@ -31,7 +32,7 @@ fn available_gas_exceeded() {
"
));

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_failed(&result);
assert_case_output_contains(
Expand All @@ -54,7 +55,7 @@ fn available_gas_fuzzing() {
"
));

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
3 changes: 2 additions & 1 deletion crates/forge/tests/integration/builtins.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use test_utils::runner::assert_passed;
use test_utils::running_tests::run_test_case;
Expand Down Expand Up @@ -29,7 +30,7 @@ fn builtin_test() {
"
));

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
7 changes: 4 additions & 3 deletions crates/forge/tests/integration/cheat_block_number.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use std::path::Path;
use test_utils::runner::{Contract, assert_passed};
Expand Down Expand Up @@ -130,7 +131,7 @@ fn cheat_block_number_basic() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -213,7 +214,7 @@ fn cheat_block_number_complex() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -310,7 +311,7 @@ fn cheat_block_number_with_span() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
7 changes: 4 additions & 3 deletions crates/forge/tests/integration/cheat_block_timestamp.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use std::path::Path;
use test_utils::runner::{Contract, assert_passed};
Expand Down Expand Up @@ -135,7 +136,7 @@ fn cheat_block_timestamp_basic() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -221,7 +222,7 @@ fn cheat_block_timestamp_complex() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -318,7 +319,7 @@ fn cheat_block_timestamp_with_span() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
5 changes: 3 additions & 2 deletions crates/forge/tests/integration/cheat_caller_address.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use std::path::Path;
use test_utils::runner::{Contract, assert_passed};
Expand Down Expand Up @@ -131,7 +132,7 @@ fn cheat_caller_address() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -220,7 +221,7 @@ fn cheat_caller_address_with_span() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
15 changes: 8 additions & 7 deletions crates/forge/tests/integration/cheat_execution_info.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use std::path::Path;
use test_utils::runner::{Contract, assert_passed};
Expand Down Expand Up @@ -95,7 +96,7 @@ fn start_and_stop_cheat_transaction_hash_single_attribute() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -268,7 +269,7 @@ fn start_cheat_execution_info_all_attributes_mocked() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -344,7 +345,7 @@ fn start_cheat_transaction_hash_cancel_mock_by_setting_attribute_to_none() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -397,7 +398,7 @@ fn start_cheat_transaction_hash_multiple() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -531,7 +532,7 @@ fn start_cheat_execution_info_all() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -596,7 +597,7 @@ fn start_cheat_transaction_hash_complex() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -706,7 +707,7 @@ fn cheat_transaction_hash_with_span() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
11 changes: 6 additions & 5 deletions crates/forge/tests/integration/cheat_fork.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::formatdoc;
use shared::test_utils::node_url::node_rpc_url;
use test_utils::runner::assert_passed;
Expand Down Expand Up @@ -47,7 +48,7 @@ fn cheat_caller_address_cairo0_contract() {
)
.as_str());

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -95,7 +96,7 @@ fn cheat_block_number_cairo0_contract() {
)
.as_str());

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -145,7 +146,7 @@ fn cheat_block_timestamp_cairo0_contract() {
)
.as_str());

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -188,7 +189,7 @@ fn mock_call_cairo0_contract() {
.as_str()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -233,7 +234,7 @@ fn store_load_cairo0_contract() {
)
.as_str());

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
7 changes: 4 additions & 3 deletions crates/forge/tests/integration/cheat_sequencer_address.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use std::path::Path;
use test_utils::runner::{Contract, assert_passed};
Expand Down Expand Up @@ -132,7 +133,7 @@ fn cheat_sequencer_address_basic() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -209,7 +210,7 @@ fn cheat_sequencer_address_complex() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -306,7 +307,7 @@ fn cheat_sequencer_address_with_span() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
7 changes: 4 additions & 3 deletions crates/forge/tests/integration/declare.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use std::path::Path;
use test_utils::runner::{Contract, assert_passed};
Expand Down Expand Up @@ -59,7 +60,7 @@ fn simple_declare() {
)
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -93,7 +94,7 @@ fn declare_simple() {
contract
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -134,7 +135,7 @@ fn redeclare() {
contract
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
11 changes: 6 additions & 5 deletions crates/forge/tests/integration/deploy.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
use forge_runner::forge_config::ForgeTrackedResource;
use indoc::indoc;
use std::path::Path;
use test_utils::runner::{Contract, assert_passed};
Expand Down Expand Up @@ -34,7 +35,7 @@ fn error_handling() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -79,7 +80,7 @@ fn deploy_syscall_check() {
.unwrap()
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -123,7 +124,7 @@ fn constructor_retdata_span() {
)
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -167,7 +168,7 @@ fn constructor_retdata_felt() {
)
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Expand Down Expand Up @@ -224,7 +225,7 @@ fn constructor_retdata_struct() {
)
);

let result = run_test_case(&test);
let result = run_test_case(&test, ForgeTrackedResource::CairoSteps);

assert_passed(&result);
}
Loading

0 comments on commit 393a3c5

Please sign in to comment.