From a88fc9cf636bffb7640cc43e1dd1706677d3c720 Mon Sep 17 00:00:00 2001 From: Amaan Qureshi Date: Sun, 19 May 2024 19:25:00 -0400 Subject: [PATCH] fix: always reset to the first language when iterating over language attributes --- cli/src/test.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/cli/src/test.rs b/cli/src/test.rs index f9a95ca7..87ecae52 100644 --- a/cli/src/test.rs +++ b/cli/src/test.rs @@ -313,6 +313,7 @@ pub fn opt_color(use_color: bool, color: ansi_term::Colour, text: &str) -> Strin } } +/// This will return false if we want to "fail fast". It will bail and not parse any more tests. #[allow(clippy::too_many_arguments)] fn run_tests( parser: &mut Parser, @@ -450,16 +451,15 @@ fn run_tests( failures.push((name.clone(), actual, output.clone())); if attributes.fail_fast { - // return value of false means to fail fast return Ok(false); } - - if i == attributes.languages.len() - 1 { - // reset back to first language - parser.set_language(opts.languages.values().next().unwrap())?; - } } } + + if i == attributes.languages.len() - 1 { + // reset to the first language + parser.set_language(opts.languages.values().next().unwrap())?; + } } opts.test_num += 1; }