Skip to content

Commit

Permalink
chore: increase wait time to ensure TCP connections (#1293)
Browse files Browse the repository at this point in the history
Fixes #1291
  • Loading branch information
enocom committed Aug 29, 2022
1 parent b011ed5 commit 9308532
Showing 1 changed file with 2 additions and 17 deletions.
19 changes: 2 additions & 17 deletions internal/proxy/proxy_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -354,32 +354,17 @@ func TestClientCloseWaitsForActiveConnections(t *testing.T) {
Instances: []proxy.InstanceConnConfig{
{Name: "proj:region:pg"},
},
WaitOnClose: 5 * time.Second,
}

c, err := proxy.NewClient(context.Background(), &fakeDialer{}, logger, in)
if err != nil {
t.Fatalf("proxy.NewClient error: %v", err)
}
go c.Serve(context.Background(), func() {})

conn := tryTCPDial(t, "127.0.0.1:5000")
_ = conn.Close()

if err := c.Close(); err != nil {
t.Fatalf("c.Close error: %v", err)
}

in.WaitOnClose = time.Second
in.Port = 5001
c, err = proxy.NewClient(context.Background(), &fakeDialer{}, logger, in)
if err != nil {
t.Fatalf("proxy.NewClient error: %v", err)
}
go c.Serve(context.Background(), func() {})

var open []net.Conn
for i := 0; i < 5; i++ {
conn = tryTCPDial(t, "127.0.0.1:5001")
conn := tryTCPDial(t, "127.0.0.1:5000")
open = append(open, conn)
}
defer func() {
Expand Down

0 comments on commit 9308532

Please sign in to comment.