From cbc3cfe101e684629f304288223d1f9171863041 Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Wed, 11 Sep 2019 16:24:53 +0200 Subject: [PATCH] integration-cli: TestAttachMultipleAndRestart: don't call t.Fatal from a goroutine ``` integration-cli/docker_cli_attach_test.go:44:: SA2002: the goroutine calls T.Fatal, which must be called in the same goroutine as the test (staticcheck) ``` Signed-off-by: Sebastiaan van Stijn --- integration-cli/docker_cli_attach_test.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/integration-cli/docker_cli_attach_test.go b/integration-cli/docker_cli_attach_test.go index b1ae40a72a..a22ef105bd 100644 --- a/integration-cli/docker_cli_attach_test.go +++ b/integration-cli/docker_cli_attach_test.go @@ -51,24 +51,24 @@ func (s *DockerSuite) TestAttachMultipleAndRestart(c *testing.T) { out, err := cmd.StdoutPipe() if err != nil { - c.Fatal(err) + c.Error(err) } defer out.Close() if err := cmd.Start(); err != nil { - c.Fatal(err) + c.Error(err) } buf := make([]byte, 1024) if _, err := out.Read(buf); err != nil && err != io.EOF { - c.Fatal(err) + c.Error(err) } startGroup.Done() if !strings.Contains(string(buf), "hello") { - c.Fatalf("unexpected output %s expected hello\n", string(buf)) + c.Errorf("unexpected output %s expected hello\n", string(buf)) } }() }