From ba373eed34972e2019f16e0ba56ea678376edd79 Mon Sep 17 00:00:00 2001 From: "Bryan C. Mills" Date: Thu, 17 Nov 2022 13:57:18 -0500 Subject: [PATCH] playground/socket: eliminate an arbitrary timeout in TestLimiter Fixes golang/go#55238. Change-Id: I17fe3865f7ec4d8889cdf71497b10a2ad0b41c11 Reviewed-on: https://go-review.googlesource.com/c/tools/+/451598 TryBot-Result: Gopher Robot Run-TryBot: Bryan Mills Auto-Submit: Bryan Mills gopls-CI: kokoro Reviewed-by: Robert Findley --- playground/socket/socket_test.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/playground/socket/socket_test.go b/playground/socket/socket_test.go index b866e37afd..d410afea87 100644 --- a/playground/socket/socket_test.go +++ b/playground/socket/socket_test.go @@ -69,9 +69,5 @@ func TestLimiter(t *testing.T) { if n != msgLimit+1 { t.Errorf("received %v messages, want %v", n, msgLimit+1) } - select { - case <-kr: - case <-time.After(100 * time.Millisecond): - t.Errorf("process wasn't killed after reaching limit") - } + <-kr }