diff options
| author | Bernd Edlinger <bernd.edlinger@hotmail.de> | 2025-01-09 15:14:15 +0100 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2025-01-13 20:19:10 +0100 |
| commit | 37a954e5c39d94c5aa60cb694071c3ee82610997 (patch) | |
| tree | 6dac32b36184a0da89e1618074d6cf165a42c2a7 | |
| parent | coveralls.yml: Avoid excluding subdirectories of already excluded dirs (diff) | |
| download | openssl-37a954e5c39d94c5aa60cb694071c3ee82610997.tar.xz openssl-37a954e5c39d94c5aa60cb694071c3ee82610997.zip | |
Fix intermittent test failure in 80-test_cmp_http.t
output to stderr is unbuffered bypassing the normal output, which does
not happen at line boundaries and is therefore confusing the TAP parser.
This is known to cause random test failures like this one:
80-test_cmp_http.t (Wstat: 0 Tests: 5 Failed: 0)
Parse errors: Tests out of sequence. Found (6) but expected (5)
Bad plan. You planned 6 tests but ran 5.
Fixes #23992
Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Saša Nedvědický <sashan@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/26363)
| -rw-r--r-- | test/recipes/80-test_cmp_http.t | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/recipes/80-test_cmp_http.t b/test/recipes/80-test_cmp_http.t index da4b4ffc25..7b8a5b237b 100644 --- a/test/recipes/80-test_cmp_http.t +++ b/test/recipes/80-test_cmp_http.t @@ -293,7 +293,7 @@ sub start_server { $args ? $args : ()]), display => 1); print "Current directory is ".getcwd()."\n"; print "Launching $server_name server: $cmd\n"; - my $pid = open($server_fh, "$cmd|"); + my $pid = open($server_fh, "$cmd 2>".result_dir()."/error.txt |"); unless ($pid) { print "Error launching $cmd, cannot obtain $server_name server PID"; return 0; |