Move arg validation to a separate func

lambdaclass · Jan 18, 2024 · 9b0a7ae · 9b0a7ae
1 parent cd3dbaf
commit 9b0a7ae
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/cairo-vm-cli/src/main.rs b/cairo-vm-cli/src/main.rs
@@ -112,9 +112,7 @@ impl FileWriter {
     }
 }
 
-fn run(args: impl Iterator<Item = String>) -> Result<(), Error> {
-    let args = Args::try_parse_from(args)?;
-
+fn validate_args(args: &Args) -> Result<(), Error> {
     if args.air_public_input.is_some() && !args.proof_mode {
         let error = Args::command().error(
             clap::error::ErrorKind::ArgumentConflict,
@@ -155,6 +153,14 @@ fn run(args: impl Iterator<Item = String>) -> Result<(), Error> {
         return Err(Error::Cli(error));
     }
 
+    Ok(())
+}
+
+fn run(args: impl Iterator<Item = String>) -> Result<(), Error> {
+    let args = Args::try_parse_from(args)?;
+
+    validate_args(&args)?;
+
     let trace_enabled = args.trace_file.is_some() || args.air_public_input.is_some();
     let mut hint_executor = BuiltinHintProcessor::new_empty();
     let cairo_run_config = cairo_run::CairoRunConfig {