style: prefer formatting prints to mock output instead of writing string (#250)

zimeg · web-flow · commit b861e2d5d042 · 2025-10-31T13:56:20.000-07:00
diff --git a/internal/config/experiments_test.go b/internal/config/experiments_test.go
@@ -300,7 +300,7 @@ func Test_Config_GetExperiments(t *testing.T) {
 func setupMockPrintDebug() (*bytes.Buffer, func(context.Context, string, ...interface{})) {
 	mockOutput := &bytes.Buffer{}
 	mockPrintDebug := func(ctx context.Context, format string, a ...interface{}) {
-		mockOutput.WriteString(fmt.Sprintf(format, a))
+		fmt.Fprintf(mockOutput, format, a...)
 	}
 	return mockOutput, mockPrintDebug
 }

Original file line number	Diff line number	Diff line change
`@@ -300,7 +300,7 @@ func Test_Config_GetExperiments(t *testing.T) {`
`300`	`300`	`func setupMockPrintDebug() (*bytes.Buffer, func(context.Context, string, ...interface{})) {`
`301`	`301`	`mockOutput := &bytes.Buffer{}`
`302`	`302`	`mockPrintDebug := func(ctx context.Context, format string, a ...interface{}) {`
`303`		`- mockOutput.WriteString(fmt.Sprintf(format, a))`
	`303`	`+ fmt.Fprintf(mockOutput, format, a...)`
`304`	`304`	`}`
`305`	`305`	`return mockOutput, mockPrintDebug`
`306`	`306`	`}`