Fix lint errors

segevfiner · Feb 1, 2022 · 75f8b6c · 75f8b6c
1 parent 809037c
commit 75f8b6c
Showing 1 changed file with 12 additions and 7 deletions.
diff --git a/dockerexec_test.go b/dockerexec_test.go
@@ -39,7 +39,10 @@ func TestMain(m *testing.M) {
 			}
 			defer pullOutput.Close()
 
-			jsonmessage.DisplayJSONMessagesStream(pullOutput, os.Stderr, 0, false, nil)
+			err = jsonmessage.DisplayJSONMessagesStream(pullOutput, os.Stderr, 0, false, nil)
+			if err != nil {
+				panic(err)
+			}
 		} else {
 			panic(err)
 		}
@@ -125,15 +128,15 @@ func TestExitStatus(t *testing.T) {
 func TestExitCode(t *testing.T) {
 	// Test that exit code are returned correctly
 	cmd := dockerexec.Command(dockerClient, testImage, "sh", "-c", "exit 42")
-	cmd.Run()
+	_ = cmd.Run()
 	assert.Equal(t, int64(42), cmd.StatusCode)
 
 	cmd = dockerexec.Command(dockerClient, testImage, "sh", "-c", "exit 255")
-	cmd.Run()
+	_ = cmd.Run()
 	assert.Equal(t, int64(255), cmd.StatusCode)
 
 	cmd = dockerexec.Command(dockerClient, testImage, "cat")
-	cmd.Run()
+	_ = cmd.Run()
 	assert.Equal(t, int64(0), cmd.StatusCode)
 
 	// Test when command does not call Run().
@@ -185,17 +188,19 @@ func TestContext(t *testing.T) {
 	defer cancel()
 
 	cmd := dockerexec.CommandContext(ctx, dockerClient, testImage, "sleep", "120")
-	cmd.Start()
+	err := cmd.Start()
+	require.NoError(t, err)
+
 	cancel()
-	err := cmd.Wait()
+	err = cmd.Wait()
 	assert.Error(t, err)
 	assert.IsType(t, context.Canceled, err)
 }
 
 func TestNilContext(t *testing.T) {
 	assert.Panics(t, func() {
 		//lint:ignore SA1012 Test for panic
-		dockerexec.CommandContext(nil, dockerClient, testImage, "cat")
+		dockerexec.CommandContext(nil, dockerClient, testImage, "cat") //nolint:staticcheck
 	})
 }