[PATCH stalld 30/52] tests: Remove redundant sleeps after start_stalld

Wander Lairson Costa <[email protected]> Mon, 8 Jun 2026 15:31:40 -0300
Newsgroups org.kernel.vger.linux-rt-users
Message-ID <[email protected]>
Remove redundant sleep commands from test_foreground.sh and
test_logging_destinations.sh. The start_stalld function already polls
for readiness before returning, rendering these post-return delays
unnecessary.

Signed-off-by: Wander Lairson Costa <[email protected]>
---
 tests/functional/test_foreground.sh           | 3 ---
 tests/functional/test_logging_destinations.sh | 1 -
 2 files changed, 4 deletions(-)

diff --git a/tests/functional/test_foreground.sh b/tests/functional/test_foreground.sh
index 792aee0..7dab793 100755
--- a/tests/functional/test_foreground.sh
+++ b/tests/functional/test_foreground.sh
@@ -24,7 +24,6 @@ require_root
 # Test 1: Without -f flag, stalld should daemonize
 test_section "Test 1: stalld daemonizes by default"
 start_stalld -l -t 5
-sleep 2
 
 assert_process_running "${STALLD_PID}" "stalld should be running"
 
@@ -42,7 +41,6 @@ test_section "Test 2: stalld stays in foreground with -f"
 
 # Start stalld in foreground but in background job
 start_stalld -f -l -t 5
-sleep 2
 
 assert_process_running "${STALLD_PID}" "stalld should be running with -f"
 
@@ -55,7 +53,6 @@ stop_stalld
 test_section "Test 3: -v implies foreground mode"
 
 start_stalld -v -l -t 5
-sleep 2
 
 assert_process_running "${STALLD_PID}" "stalld should be running with -v"
 
diff --git a/tests/functional/test_logging_destinations.sh b/tests/functional/test_logging_destinations.sh
index a25e571..eb06171 100755
--- a/tests/functional/test_logging_destinations.sh
+++ b/tests/functional/test_logging_destinations.sh
@@ -49,7 +49,6 @@ if command -v dmesg >/dev/null 2>&1; then
 	DMESG_BEFORE=$(dmesg | wc -l)
 
 	start_stalld -f -k -l -t 5
-	sleep 2
 
 	if assert_process_running "${STALLD_PID}" "stalld with -k should be running"; then
 		sleep 1
-- 
2.54.0