| Message ID | 20181209225628.22216-2-szeder.dev@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | test-lib: add the '--stress' option to help reproduce occasional failures in flaky tests | expand |
On Sun, Dec 09, 2018 at 11:56:22PM +0100, SZEDER Gábor wrote: > Right now if a test script receives SIGTERM or SIGHUP (e.g., because a > test was hanging and the user 'kill'-ed it or simply closed the > terminal window the test was running in), the shell exits immediately. > This can be annoying if the test script did any global setup, like > starting apache or git-daemon, as it will not have an opportunity to > clean up after itself. A subsequent run of the test won't be able to > start its own daemon, and will either fail or skip the tests. > > Instead, let's trap SIGTERM and SIGHUP as well to make sure we do a > clean shutdown, and just chain it to a normal exit (which will trigger > any cleanup). > > This patch follows suit of da706545f7 (t: translate SIGINT to an exit, > 2015-03-13), and even stole its commit message as well. No wonder it was so nicely explained. ;) I think this is quite a reasonable thing to do. Since we're trying to clean up, in theory we would like to hook any signal death, but these three are the common ones in practice. We handle QUIT and PIPE as well in our C code; the latter isn't an issue here. SIGQUIT is a possibility, I guess, but seems rather unlikely. -Peff
diff --git a/t/test-lib.sh b/t/test-lib.sh index 0f1faa24b2..9a3f7930a3 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -476,7 +476,7 @@ die () { GIT_EXIT_OK= trap 'die' EXIT -trap 'exit $?' INT +trap 'exit $?' INT TERM HUP # The user-facing functions are loaded from a separate file so that # test_perf subshells can have them too
Right now if a test script receives SIGTERM or SIGHUP (e.g., because a test was hanging and the user 'kill'-ed it or simply closed the terminal window the test was running in), the shell exits immediately. This can be annoying if the test script did any global setup, like starting apache or git-daemon, as it will not have an opportunity to clean up after itself. A subsequent run of the test won't be able to start its own daemon, and will either fail or skip the tests. Instead, let's trap SIGTERM and SIGHUP as well to make sure we do a clean shutdown, and just chain it to a normal exit (which will trigger any cleanup). This patch follows suit of da706545f7 (t: translate SIGINT to an exit, 2015-03-13), and even stole its commit message as well. Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com> --- t/test-lib.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)