snow/wireguard-go
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

device: increase timeout in tests

Browse source 
When running many concurrent test processing using
https://godoc.org/golang.org/x/tools/cmd/stress
the processing sometimes cannot complete a ping in under 300ms.
Increase the timeout to 5s to reduce the rate of false positives.

Signed-off-by: Josh Bleecher Snyder <josh@tailscale.com>
This commit is contained in:
Josh Bleecher Snyder 2020-12-14 14:11:33 -08:00 committed by Jason A. Donenfeld
parent fc0aabbae9
commit db1edc7e91
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
device/device_test.go
View file

@ -112,7 +112,7 @@ func TestTwoDevicePing(t *testing.T) {
if !bytes.Equal(msg2to1, msgRecv) { if !bytes.Equal(msg2to1, msgRecv) {
t.Error("ping did not transit correctly") t.Error("ping did not transit correctly")
} }
case <-time.After(300 * time.Millisecond): case <-time.After(5 * time.Second):
t.Error("ping did not transit") t.Error("ping did not transit")
} }
}) })
@ -125,7 +125,7 @@ func TestTwoDevicePing(t *testing.T) {
if !bytes.Equal(msg1to2, msgRecv) { if !bytes.Equal(msg1to2, msgRecv) {
t.Error("return ping did not transit correctly") t.Error("return ping did not transit correctly")
} }
case <-time.After(300 * time.Millisecond): case <-time.After(5 * time.Second):
t.Error("return ping did not transit") t.Error("return ping did not transit")
} }
}) })