Skip to content

Commit 164bbfe

Browse files
committed
CI: Set process I/O streams before starting copying.
1 parent bc032f1 commit 164bbfe

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

internal/integrationtest/arduino-cli.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -389,22 +389,22 @@ func (cli *ArduinoCLI) run(ctx context.Context, stdoutBuff, stderrBuff io.Writer
389389

390390
cli.t.NoError(cliProc.Start())
391391

392+
if stdoutBuff == nil {
393+
stdoutBuff = io.Discard
394+
}
395+
if stderrBuff == nil {
396+
stderrBuff = io.Discard
397+
}
392398
var wg sync.WaitGroup
393399
wg.Add(2)
394400
go func() {
395401
defer wg.Done()
396-
if stdoutBuff == nil {
397-
stdoutBuff = io.Discard
398-
}
399402
if _, err := io.Copy(stdoutBuff, io.TeeReader(stdout, terminalOut)); err != nil {
400403
fmt.Fprintln(terminalOut, color.HiBlackString("<<< stdout copy error:"), err)
401404
}
402405
}()
403406
go func() {
404407
defer wg.Done()
405-
if stderrBuff == nil {
406-
stderrBuff = io.Discard
407-
}
408408
if _, err := io.Copy(stderrBuff, io.TeeReader(stderr, terminalErr)); err != nil {
409409
fmt.Fprintln(terminalErr, color.HiBlackString("<<< stderr copy error:"), err)
410410
}

0 commit comments

Comments
 (0)