From cd8b29b0fe8aaf25ae32f926d940956cfea48ae5 Mon Sep 17 00:00:00 2001 From: Jack Christensen Date: Sat, 24 Sep 2022 12:54:59 -0500 Subject: [PATCH] Fix flickering on TestConnectTimeoutStuckOnTLSHandshake Ensure that even if the outer function finishes the goroutine can still send an error. --- pgconn/pgconn_test.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pgconn/pgconn_test.go b/pgconn/pgconn_test.go index 5b6ca284..63a3d19b 100644 --- a/pgconn/pgconn_test.go +++ b/pgconn/pgconn_test.go @@ -235,8 +235,7 @@ func TestConnectTimeoutStuckOnTLSHandshake(t *testing.T) { require.NoError(t, err) defer ln.Close() - serverErrChan := make(chan error) - defer close(serverErrChan) + serverErrChan := make(chan error, 1) go func() { conn, err := ln.Accept() if err != nil {