fix defer in for loop, maybe resource leak

staging/src/k8s.io/kubectl/pkg/cmd/clusterinfo/clusterinfo_dump_test.go

@@ -27,16 +27,31 @@ import (
 )
 
 func TestSetupOutputWriterNoOp(t *testing.T) {
-	tests := []string{"", "-"}
+	tests := []struct {
-	for _, test := range tests {
+		name         string
+		outputWriter string
+	}{
+		{
+			name:         "empty",
+			outputWriter: "",
+		},
+		{
+			name:         "stdout",
+			outputWriter: "-",
+		},
+	}
+	for _, tt := range tests {
+		tt := tt
+		t.Run(tt.name, func(t *testing.T) {
 			_, _, buf, _ := genericclioptions.NewTestIOStreams()
 			f := cmdtesting.NewTestFactory()
 			defer f.Cleanup()
 
-		writer := setupOutputWriter(test, buf, "/some/file/that/should/be/ignored", "")
+			writer := setupOutputWriter(tt.outputWriter, buf, "/some/file/that/should/be/ignored", "")
 			if writer != buf {
 				t.Errorf("expected: %v, saw: %v", buf, writer)
 			}
+		})
 	}
 }