chore(provisionerd): close completeChan exactly once (coder#16011)

johnstcn · web-flow · commit 82070944f1d1 · 2025-01-02T15:14:06.000Z
Fixes coder/internal#263
diff --git a/provisionerd/provisionerd_test.go b/provisionerd/provisionerd_test.go
@@ -70,8 +70,11 @@ func TestProvisionerd(t *testing.T) {
 			close(done)
 		})
 		completeChan := make(chan struct{})
+		var completed sync.Once
 		closer := createProvisionerd(t, func(ctx context.Context) (proto.DRPCProvisionerDaemonClient, error) {
-			defer close(completeChan)
+			completed.Do(func() {
+				defer close(completeChan)
+			})
 			return nil, xerrors.New("an error")
 		}, provisionerd.LocalProvisioners{})
 		require.Condition(t, closedWithin(completeChan, testutil.WaitShort))
