From 68f67c54b63b0f4ee962500c74511f527dff9b45 Mon Sep 17 00:00:00 2001 From: Kyle Carberry Date: Sun, 24 Apr 2022 23:06:18 -0500 Subject: [PATCH] fix: Add sync.Once to prevent double close in test (#1124) 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 bae6de4693..dc86bcfedb 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)