From 78ac4df22a0abbd2fd74adbcd4122a5701e42603 Mon Sep 17 00:00:00 2001 From: Kyle Carberry Date: Mon, 25 Apr 2022 03:58:59 +0000 Subject: [PATCH] fix: Add sync.Once to prevent double close in test https://github.com/coder/coder/runs/6151451291?check_suite_focus=true --- provisionerd/provisionerd_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/provisionerd/provisionerd_test.go b/provisionerd/provisionerd_test.go index bae6de46933af..dc86bcfedb4e0 100644 --- a/provisionerd/provisionerd_test.go +++ b/provisionerd/provisionerd_test.go @@ -310,12 +310,15 @@ func TestProvisionerd(t *testing.T) { didLog atomic.Bool didAcquireJob atomic.Bool ) + var complete sync.Once completeChan := make(chan struct{}) closer := createProvisionerd(t, func(ctx context.Context) (proto.DRPCProvisionerDaemonClient, error) { return createProvisionerDaemonClient(t, provisionerDaemonTestServer{ acquireJob: func(ctx context.Context, _ *proto.Empty) (*proto.AcquiredJob, error) { if didAcquireJob.Load() { - close(completeChan) + complete.Do(func() { + close(completeChan) + }) return &proto.AcquiredJob{}, nil } didAcquireJob.Store(true)