Move test utils to testing module (#1422)

* move testing utils to testing module

* make the module available only when testing or with testing feature

* export macros

* move tests to their correct module

* fmt

---------

Co-authored-by: gabrielbosio <[email protected]>

Author: FrancoGiachetta

Cargo.toml

@@ -115,12 +115,14 @@ with-mem-tracing = []
 with-libfunc-profiling = []
 with-segfault-catcher = []
 with-trace-dump = ["dep:sierra-emu"]
-testing = ["dep:cairo-lang-compiler"]
+testing = ["dep:cairo-lang-compiler", "dep:cairo-lang-starknet", "dep:cairo-lang-filesystem"]
 
 [dependencies]
 aquamarine.workspace = true
 bumpalo.workspace = true
 cairo-lang-compiler = { workspace = true, optional = true }
+cairo-lang-filesystem = { workspace = true, optional = true }
+cairo-lang-starknet = { workspace = true, optional = true }
 cairo-lang-runner.workspace = true
 cairo-lang-sierra.workspace = true
 cairo-lang-utils.workspace = true

src/cache/aot.rs

@@ -87,7 +87,7 @@ where
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::{utils::test::load_cairo, values::Value};
+    use crate::{load_cairo, values::Value};
     use starknet_types_core::felt::Felt;
 
     #[test]

src/cache/jit.rs

@@ -70,7 +70,7 @@ where
 #[cfg(test)]
 mod test {
     use super::*;
-    use crate::utils::test::load_cairo;
+    use crate::load_cairo;
     use std::time::Instant;
 
     #[test]

src/executor.rs

@@ -686,8 +686,8 @@ fn parse_result(
 mod tests {
     use super::*;
     use crate::{
-        context::NativeContext, starknet_stub::StubSyscallHandler, utils::test::load_cairo,
-        utils::test::load_starknet, OptLevel,
+        context::NativeContext, load_cairo, load_starknet, starknet_stub::StubSyscallHandler,
+        OptLevel,
     };
     use cairo_lang_sierra::program::Program;
     use rstest::*;

src/executor/aot.rs

@@ -205,7 +205,7 @@ mod tests {
     use crate::{
         context::NativeContext,
         starknet_stub::StubSyscallHandler,
-        utils::test::{load_cairo, load_starknet},
+        {load_cairo, load_starknet},
     };
     use cairo_lang_sierra::program::Program;
     use rstest::*;

src/executor/contract.rs

@@ -814,7 +814,7 @@ impl Drop for LockFile {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::{starknet_stub::StubSyscallHandler, utils::test::load_starknet_contract};
+    use crate::{load_starknet_contract, starknet_stub::StubSyscallHandler};
     use cairo_lang_starknet_classes::contract_class::{
         version_id_from_serialized_sierra_program, ContractClass,
     };

src/libfuncs/array.rs

@@ -1303,10 +1303,8 @@ fn is_shared<'ctx, 'this>(
 #[cfg(test)]
 mod test {
     use crate::{
-        utils::{
-            felt252_str,
-            test::{jit_enum, jit_panic, jit_struct, load_cairo, run_program},
-        },
+        jit_enum, jit_panic, jit_struct, load_cairo,
+        utils::{felt252_str, testing::run_program},
         values::Value,
     };
     use pretty_assertions_sorted::assert_eq;

src/libfuncs/bool.rs

@@ -207,10 +207,7 @@ pub fn build_bool_to_felt252<'ctx, 'this>(
 
 #[cfg(test)]
 mod test {
-    use crate::{
-        utils::test::{jit_enum, jit_struct, load_cairo, run_program},
-        values::Value,
-    };
+    use crate::{jit_enum, jit_struct, load_cairo, utils::testing::run_program, values::Value};
 
     #[test]
     fn run_not() {

src/libfuncs/bounded_int.rs

@@ -855,7 +855,7 @@ mod test {
 
     use crate::{
         context::NativeContext, execution_result::ExecutionResult, executor::JitNativeExecutor,
-        utils::test::load_cairo, OptLevel, Value,
+        load_cairo, OptLevel, Value,
     };
 
     #[test]

src/libfuncs/box.rs

@@ -151,10 +151,7 @@ pub fn build_unbox<'ctx, 'this>(
 
 #[cfg(test)]
 mod test {
-    use crate::{
-        utils::test::{load_cairo, run_program_assert_output},
-        values::Value,
-    };
+    use crate::{load_cairo, utils::testing::run_program_assert_output, values::Value};
 
     #[test]
     fn run_box_unbox() {