]> Cypherpunks repositories - gostls13.git/commit
cmd/go/internal/test: scale the WaitDelay for test subprocesses
authorBryan C. Mills <bcmills@google.com>
Wed, 1 Feb 2023 21:44:21 +0000 (16:44 -0500)
committerGopher Robot <gobot@golang.org>
Thu, 2 Feb 2023 20:23:27 +0000 (20:23 +0000)
commit88a36c9e9a511ec6ad218633bce1e82f25e54d35
treef04169d114d8d943ea595b0ae0d261845aa0e181
parentfc86770d87360ddf0346bf407a80935fe2dddec8
cmd/go/internal/test: scale the WaitDelay for test subprocesses

Prior to CL 456116 we had an arbitrary 5-second delay after a test
times out before we kill the test. In CL 456116, I reused that
arbitrary 5-second delay as the WaitDelay as well, but on slower
builders it does not seem to be generous enough.

Instead of hard-coding the delay, for tests with a finite timout we
now use a hard-coded fraction of the overall timeout. That will
probably give delays that are longer than strictly necessary for very
long timeouts, but if the user is willing to wait for a very long
timeout they can probably wait a little longer for I/O too.

Fixes #58230.
Updates #24050.

Change-Id: Ifbf3e576c034c721aa00cd17bf88563474b09955
Reviewed-on: https://go-review.googlesource.com/c/go/+/464555
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Michael Pratt <mpratt@google.com>
Run-TryBot: Bryan Mills <bcmills@google.com>
Auto-Submit: Bryan Mills <bcmills@google.com>
src/cmd/go/internal/test/test.go
src/cmd/go/testdata/script/test_timeout_stdin.txt