Skip to content

Commit 43c44b0

Browse files
committed
test(build): update TestPipeReader call to new splitReader(ctx, reader) signature
The earlier commit on this branch added context.Context as the first argument of splitReader so the copy goroutine can unblock on ctx.Done(). TestPipeReader still passed only the reader, which tripped the lint/CI typecheck: pkg/backend/build/builder_test.go:287:24: not enough arguments in call to splitReader have (*strings.Reader) want (context.Context, io.Reader) Pass context.Background() so the test exercises the new signature. `context` is already imported by other tests in this file. Signed-off-by: SAY-5 <say.apm35@gmail.com>
1 parent e5bbf20 commit 43c44b0

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

pkg/backend/build/builder_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -284,7 +284,7 @@ func TestBuilderSuite(t *testing.T) {
284284

285285
func TestPipeReader(t *testing.T) {
286286
r := strings.NewReader("some io.Reader stream to be read\n")
287-
r1, r2 := splitReader(r)
287+
r1, r2 := splitReader(context.Background(), r)
288288
var wg sync.WaitGroup
289289
wg.Add(1)
290290
go func() {

0 commit comments

Comments
 (0)