From 4273c9f7e749dccc4458bdaf8185efb9e7649399 Mon Sep 17 00:00:00 2001 From: greged93 <82421016+greged93@users.noreply.github.com> Date: Thu, 23 May 2024 11:43:35 +0200 Subject: [PATCH] remove unused mutex (#608) --- src/bin/cairo-native-test.rs | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/bin/cairo-native-test.rs b/src/bin/cairo-native-test.rs index 973179182..d3473f19d 100644 --- a/src/bin/cairo-native-test.rs +++ b/src/bin/cairo-native-test.rs @@ -36,7 +36,6 @@ use starknet_types_core::felt::Felt; use std::{ iter::once, path::{Path, PathBuf}, - sync::Mutex, vec::IntoIter, }; use tracing_subscriber::{EnvFilter, FmtSubscriber}; @@ -428,12 +427,12 @@ fn run_tests( .unwrap(); println!("running {} tests", named_tests.len()); - let wrapped_summary = Mutex::new(Ok(TestsSummary { + let mut wrapped_summary = Ok(TestsSummary { passed: vec![], failed: vec![], ignored: vec![], failed_run_results: vec![], - })); + }); named_tests .into_iter() .map( @@ -493,14 +492,10 @@ fn run_tests( }, ) .for_each(|r| { - let mut wrapped_summary = wrapped_summary.lock().unwrap(); - if wrapped_summary.is_err() { - return; - } let (name, status) = match r { Ok((name, status)) => (name, status), Err(err) => { - *wrapped_summary = Err(err); + wrapped_summary = Err(err); return; } }; @@ -526,7 +521,7 @@ fn run_tests( } res_type.push(name); }); - wrapped_summary.into_inner().unwrap() + wrapped_summary } pub struct TestSyscallHandler;