From 3030608de00046c4fdb9fcc490438541368eaa18 Mon Sep 17 00:00:00 2001 From: Douglas <32344964+NotTheDr01ds@users.noreply.github.com> Date: Thu, 27 Mar 2025 14:47:43 -0400 Subject: [PATCH] Ignore problematic overlapping tests for SHLVL (#15430) The `$env.SHLVL` tests, while improved, still cause CI (usually local) an irritating percentage of the time. Until we can come with a better way of testing, we're going to ignore them. --- tests/shell/environment/env.rs | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/shell/environment/env.rs b/tests/shell/environment/env.rs index 6513db1fbc..265eed5dd0 100644 --- a/tests/shell/environment/env.rs +++ b/tests/shell/environment/env.rs @@ -246,6 +246,11 @@ fn env_shlvl_commandstring_does_not_increment() { // test run will likely hang, at least for some users. // Instead, use -e / --execute with an `exit` to test REPL // functionality as demonstrated below. +// +// We've also learned that `-e 'exit'` is not enough to +// prevent failures entirely. For now we're going to ignore +// these tests until we can find a better solution. +#[ignore = "Causing hangs when both tests overlap"] #[test] fn env_shlvl_in_repl() { let actual = nu!(" @@ -256,6 +261,7 @@ fn env_shlvl_in_repl() { assert_eq!(actual.out, "6"); } +#[ignore = "Causing hangs when both tests overlap"] #[test] fn env_shlvl_in_exec_repl() { let actual = nu!(r#"