update func description

pkg/kubelet/kuberuntime/helpers.go

@@ -139,8 +139,10 @@ func (m *kubeGenericRuntimeManager) getImageUser(image string) (*int64, string,
 	return new(int64), "", nil
 }
 
-// isInitContainerFailed returns true if container has exited and exitcode is not zero
+// isInitContainerFailed returns true under the following conditions:
-// or is in unknown state.
+// 1. container has exited and exitcode is not zero.
+// 2. container in unknown state.
+// 3. container occurs OOMKilled.
 func isInitContainerFailed(status *kubecontainer.Status) bool {
 	// When oomkilled occurs, init container should be considered as a failure.
 	if status.Reason == "OOMKilled" {

pkg/kubelet/kuberuntime/helpers_test.go

@@ -43,6 +43,7 @@ func TestIsInitContainerFailed(t *testing.T) {
 	tests := []struct {
 		status      *kubecontainer.Status
 		isFailed    bool
+		description string
 	}{
 		{
 			status: &kubecontainer.Status{
@@ -50,18 +51,22 @@ func TestIsInitContainerFailed(t *testing.T) {
 				ExitCode: 1,
 			},
 			isFailed:    true,
+			description: "Init container which state is exited and exitcode is non-zero shoud return true",
 		},
 		{
 			status: &kubecontainer.Status{
 				State: kubecontainer.ContainerStateUnknown,
 			},
 			isFailed:    true,
+			description: "Init container which state is unknown should return true",
 		},
 		{
 			status: &kubecontainer.Status{
 				Reason:   "OOMKilled",
+				ExitCode: 0,
 			},
 			isFailed:    true,
+			description: "Init container which state is exited and exitcode is zero shoud return true",
 		},
 		{
 			status: &kubecontainer.Status{
@@ -69,23 +74,26 @@ func TestIsInitContainerFailed(t *testing.T) {
 				ExitCode: 0,
 			},
 			isFailed:    false,
+			description: "Init container which state is exited and exitcode is zero shoud return false",
 		},
 		{
 			status: &kubecontainer.Status{
 				State: kubecontainer.ContainerStateRunning,
 			},
 			isFailed:    false,
+			description: "Init container which state is running should return false",
 		},
 		{
 			status: &kubecontainer.Status{
 				State: kubecontainer.ContainerStateCreated,
 			},
 			isFailed:    false,
+			description: "Init container which state is created should return false",
 		},
 	}
 	for i, test := range tests {
 		isFailed := isInitContainerFailed(test.status)
-		assert.Equal(t, test.isFailed, isFailed, "TestCase[%d]", i)
+		assert.Equal(t, test.isFailed, isFailed, "TestCase[%d]: %s", i, test.description)
 	}
 }