fix: prevent race in mockLogger

dannykopping · dannykopping · commit 48db30f8066d · 2025-06-13T10:42:38.000+02:00
Signed-off-by: Danny Kopping &lt;dannykopping@gmail.com&gt;
diff --git a/provisioner/terraform/executor_internal_test.go b/provisioner/terraform/executor_internal_test.go
@@ -19,12 +19,17 @@ import (
 )
 
 type mockLogger struct {
+	mu sync.Mutex
+
 	logs []*proto.Log
 }
 
 var _ logSink = &mockLogger{}
 
 func (m *mockLogger) ProvisionLog(l proto.LogLevel, o string) {
+	m.mu.Lock()
+	defer m.mu.Unlock()
+
 	m.logs = append(m.logs, &proto.Log{Level: l, Output: o})
 }
 

Original file line number	Diff line number	Diff line change
`@@ -19,12 +19,17 @@ import (`
`19`	`19`	`)`
`20`	`20`
`21`	`21`	`type mockLogger struct {`
	`22`	`+ mu sync.Mutex`
	`23`	`+`
`22`	`24`	`logs []*proto.Log`
`23`	`25`	`}`
`24`	`26`
`25`	`27`	`var _ logSink = &mockLogger{}`
`26`	`28`
`27`	`29`	`func (m *mockLogger) ProvisionLog(l proto.LogLevel, o string) {`
	`30`	`+ m.mu.Lock()`
	`31`	`+ defer m.mu.Unlock()`
	`32`	`+`
`28`	`33`	`m.logs = append(m.logs, &proto.Log{Level: l, Output: o})`
`29`	`34`	`}`
`30`	`35`