]> git.ipfire.org Git - thirdparty/git.git/commit - t/t5561-http-backend.sh
t5561: get rid of racy appending to logfile
authorStephan Beyer <s-beyer@gmx.net>
Thu, 24 Sep 2015 18:12:22 +0000 (20:12 +0200)
committerJunio C Hamano <gitster@pobox.com>
Fri, 25 Sep 2015 15:45:25 +0000 (08:45 -0700)
commit362d8b6e0d313e0e06553f5c390dc7f2172c0569
treed28a5b26ee2d886edbd2eb84182ffcacbf3065c3
parente156455ea49124c140a67623f22a393db62d5d98
t5561: get rid of racy appending to logfile

The definition of log_div() appended information to the web server's
logfile to make the test more readable. However, log_div() was called
right after a request is served (which is done by git-http-backend);
the web server waits for the git-http-backend process to exit before
it writes to the log file. When the duration between serving a request
and exiting was long, the log_div() output was written before the last
request's log, and the test failed. (This duration could become
especially long for PROFILE=GEN builds.)

To get rid of this behavior, we should not change the logfile at all.
This commit removes log_div() and its calls. The additional information
is kept in the test (for readability reasons) but filtered out before
comparing it to the actual logfile.

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>
Reviewed-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t5560-http-backend-noserver.sh
t/t5561-http-backend.sh
t/t556x_common