]> Cypherpunks repositories - gostls13.git/commit
os: eliminate arbitrary sleep in Kill tests
authorBryan C. Mills <bcmills@google.com>
Wed, 16 Feb 2022 15:24:42 +0000 (10:24 -0500)
committerBryan Mills <bcmills@google.com>
Wed, 16 Feb 2022 21:34:51 +0000 (21:34 +0000)
commiteaf040502b763a6f00dced35e4173c2ce90eb52f
treeeb71692c0596910e59c2b4260ede9754d8bef0dc
parentc016133c50512e9a83e7442bd7ac614fe7ca62de
os: eliminate arbitrary sleep in Kill tests

The test spawned a subprocess that arbitrarily slept for one second.
However, on some platforms, longer than one second may elapse between
starting the subprocess and sending the termination signal.

Instead, the subprocess now closes stdout and reads stdin until EOF,
eliminating the need for an arbitrary duration. (If the parent test
times out, the stdin pipe will break, so the subprocess still won't
leak forever.)

This also makes the test much faster in the typical case: since it
uses synchronization instead of sleeping, it can run as quickly as the
host OS can start and kill the process.

Fixes #44131

Change-Id: I9753571438380dc14fc3531efdaea84578a47fae
Reviewed-on: https://go-review.googlesource.com/c/go/+/386174
Trust: Bryan Mills <bcmills@google.com>
Run-TryBot: Bryan Mills <bcmills@google.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
src/os/os_test.go