From a3ac3badb8f4fd1c90f479da46353c6568a06e9b Mon Sep 17 00:00:00 2001 From: Sylvestre Ledru Date: Wed, 15 Apr 2026 10:51:32 +0200 Subject: [PATCH] tests(env): increase signal delivery delay on macOS to fix flaky CI The test_env_arg_ignore_signal_valid_signals test was failing on macOS CI because 100ms was not always enough for the process to terminate after receiving a fatal signal. Increase the delay to 500ms on macOS only, keeping 100ms on other platforms. Fixes flaky failure seen in: https://github.com/uutils/coreutils/actions/runs/24442074142/job/71409227191 --- tests/by-util/test_env.rs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/by-util/test_env.rs b/tests/by-util/test_env.rs index 4a989bd05..1a1d504a1 100644 --- a/tests/by-util/test_env.rs +++ b/tests/by-util/test_env.rs @@ -52,6 +52,10 @@ impl Target { .spawn() .expect("failed to send signal") .wait(); + // macOS CI needs a longer delay for process teardown after signal delivery + #[cfg(target_os = "macos")] + self.child.delay(500); + #[cfg(not(target_os = "macos"))] self.child.delay(100); } fn is_alive(&mut self) -> bool {