[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