Merge pull request #36732 from oulinbao/refactor_describe_containers

Automatic merge from submit-queue Refactor describe.go with PrefixWriter **What this PR does / why we need it**: refactor describeContainers function and decrease gocyclo result **Special notes for your reviewer**: the new PR for 36032 https://github.com/kubernetes/kubernetes/pull/36032 **Release note**: ```release-note ```NONE

Merge pull request #36732 from oulinbao/refactor_describe_containers
91d8c411 · Kubernetes Submit Queue · GitHub · 238ffdd0 · 0b74fa12 · 91d8c411
Commit 91d8c411 authored Dec 03, 2016 by Kubernetes Submit Queue Committed by GitHub Dec 03, 2016
Expand all Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

describe.go pkg/kubectl/describe.go +0 -0

describe_test.go pkg/kubectl/describe_test.go +2 -1

No files found.
--- a/pkg/kubectl/describe.go
+++ b/pkg/kubectl/describe.go
--- a/pkg/kubectl/describe_test.go
+++ b/pkg/kubectl/describe_test.go
@@ -363,7 +363,8 @@ func TestDescribeContainers(t *testing.T) {
 				ContainerStatuses: []api.ContainerStatus{testCase.status},
 			},
 		}
-		describeContainers("Containers", pod.Spec.Containers, pod.Status.ContainerStatuses, EnvValueRetriever(&pod), out, "")
+		writer := &PrefixWriter{out}
+		describeContainers("Containers", pod.Spec.Containers, pod.Status.ContainerStatuses, EnvValueRetriever(&pod), writer, "")
 		output := out.String()
 		for _, expected := range testCase.expectedElements {
 			if !strings.Contains(output, expected) {