Skip to content

Commit 9dd8034

Browse files
cherylsyclaude
andcommitted
fix: eliminate race in FEC stream tests by killing server before clearing logs
Same bug pattern as 2e09e4f (frame_type_bit tests): clear_log runs before killall, so the previous test's server can write [error] to slog between truncation and termination. Fix XOR, RSC, and PM stream test blocks by reordering to killall + sleep + clear_log. Co-Authored-By: Claude (claude-opus-4-6) <noreply@anthropic.com>
1 parent 99acdfb commit 9dd8034

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

scripts/case_test.sh

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4589,8 +4589,9 @@ else
45894589
fi
45904590

45914591

4592-
clear_log
45934592
killall test_server 2> /dev/null
4593+
sleep 1
4594+
clear_log
45944595
stdbuf -oL ${SERVER_BIN} -l d -e -f -x 1 -M > /dev/null &
45954596
sleep 1
45964597

@@ -4607,8 +4608,9 @@ else
46074608
case_print_result "fec_recovered_function_of_stream_xor" "fail"
46084609
fi
46094610

4610-
clear_log
46114611
killall test_server 2> /dev/null
4612+
sleep 1
4613+
clear_log
46124614
stdbuf -oL ${SERVER_BIN} -l d -e -f -x 1 -M > /dev/null &
46134615
sleep 1
46144616

@@ -4625,8 +4627,9 @@ else
46254627
case_print_result "fec_recovered_function_of_stream_rsc" "fail"
46264628
fi
46274629

4628-
clear_log
46294630
killall test_server 2> /dev/null
4631+
sleep 1
4632+
clear_log
46304633
stdbuf -oL ${SERVER_BIN} -l d -e -f -x 1 -M > /dev/null &
46314634
sleep 1
46324635

0 commit comments

Comments
 (0)