fix: add mutex operations

DanielleMaywood · DanielleMaywood · commit 3580069bbb6e · 2024-09-20T08:40:02.000Z
diff --git a/agent/agenttest/client.go b/agent/agenttest/client.go
@@ -184,7 +184,9 @@ func (*FakeAgentAPI) GetServiceBanner(context.Context, *agentproto.GetServiceBan
 }
 
 func (f *FakeAgentAPI) GetTimings() []*agentproto.Timing {
-	return f.timings
+	f.Lock()
+	defer f.Unlock()
+	return slices.Clone(f.timings)
 }
 
 func (f *FakeAgentAPI) SetAnnouncementBannersFunc(fn func() ([]codersdk.BannerConfig, error)) {

Original file line number	Diff line number	Diff line change
`@@ -184,7 +184,9 @@ func (FakeAgentAPI) GetServiceBanner(context.Context, agentproto.GetServiceBan`
`184`	`184`	`}`
`185`	`185`
`186`	`186`	`func (f FakeAgentAPI) GetTimings() []agentproto.Timing {`
`187`		`- return f.timings`
	`187`	`+ f.Lock()`
	`188`	`+ defer f.Unlock()`
	`189`	`+ return slices.Clone(f.timings)`
`188`	`190`	`}`
`189`	`191`
`190`	`192`	`func (f *FakeAgentAPI) SetAnnouncementBannersFunc(fn func() ([]codersdk.BannerConfig, error)) {`