refactor(starknet_committer_and_os_cli): remove the python flow test (#6241)

Author: nimrod-starkware

crates/starknet_committer_and_os_cli/src/os_cli/tests/python_tests.rs

@@ -1,6 +1,4 @@
 use std::collections::{HashMap, HashSet};
-use std::fs;
-use std::path::Path;
 
 // TODO(Amos): When available in the VM crate, use an existing set, instead of using each hint
 //   const explicitly.
@@ -199,13 +197,10 @@ use cairo_vm::hint_processor::builtin_hint_processor::hint_code::{
 };
 use starknet_os::hints::enum_definition::{AggregatorHint, HintExtension, OsHint};
 use starknet_os::hints::types::HintEnum;
-use starknet_os::io::os_output::StarknetOsRunnerOutput;
-use starknet_os::runner::run_os_stateless;
 use starknet_os::test_utils::cairo_runner::EntryPointRunnerConfig;
 use strum::IntoEnumIterator;
 
 use crate::os_cli::commands::{validate_input, Input};
-use crate::os_cli::run_os_cli::OsCliOutput;
 use crate::os_cli::tests::aliases::aliases_test;
 use crate::os_cli::tests::bls_field::test_bls_field;
 use crate::os_cli::tests::types::{OsPythonTestError, OsPythonTestResult, OsSpecificTestError};
@@ -219,7 +214,6 @@ pub enum OsPythonTestRunner {
     CompareOsHints,
     InputDeserialization,
     RunDummyFunction,
-    RunOsFlowTest,
 }
 
 // Implements conversion from a string to the test runner.
@@ -233,7 +227,6 @@ impl TryFrom<String> for OsPythonTestRunner {
             "compare_os_hints" => Ok(Self::CompareOsHints),
             "input_deserialization" => Ok(Self::InputDeserialization),
             "run_dummy_function" => Ok(Self::RunDummyFunction),
-            "run_os_flow_test" => Ok(Self::RunOsFlowTest),
             _ => Err(PythonTestError::UnknownTestName(value)),
         }
     }
@@ -248,7 +241,6 @@ impl PythonTestRunner for OsPythonTestRunner {
             Self::CompareOsHints => compare_os_hints(Self::non_optional_input(input)?),
             Self::InputDeserialization => input_deserialization(Self::non_optional_input(input)?),
             Self::RunDummyFunction => run_dummy_cairo_function(Self::non_optional_input(input)?),
-            Self::RunOsFlowTest => run_os_flow_test(Self::non_optional_input(input)?),
         }
     }
 }
@@ -611,20 +603,3 @@ else:
     value = (-y) % SECP256R1.prime"#,
     ])
 }
-// TODO(Nimrod): Remove this python test and run the os directly with `parse_and_run_os`.
-/// Runs the OS with the given input and returns the deserialized output.
-fn run_os_flow_test(input: &str) -> OsPythonTestResult {
-    let Input { layout, compiled_os_path, os_hints, cairo_pie_zip_path } =
-        serde_json::from_str(input)?;
-    // Load the compiled_os from the compiled_os_path.
-    let compiled_os =
-        fs::read(Path::new(&compiled_os_path)).expect("Failed to read compiled_os file");
-    let StarknetOsRunnerOutput { os_output, cairo_pie, unused_hints } =
-        run_os_stateless(&compiled_os, layout, os_hints)?;
-    let merge_extra_segments = false;
-    cairo_pie
-        .write_zip_file(Path::new(&cairo_pie_zip_path), merge_extra_segments)
-        .expect("Failed to write cairo pie.");
-    let output = OsCliOutput { os_output, unused_hints };
-    Ok(serde_json::to_string(&output)?)
-}