Merge pull request #67347 from juanvallejo/jvallejo/use-temp-file-config-tests

Automatic merge from submit-queue (batch tested with PRs 67347, 67307, 67358, 67364, 67385). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. use temp file for kubeconfig in test Followup to https://github.com/kubernetes/kubernetes/pull/67093 Updates client_config_test to use a temporary file for kubeconfig. **Release note**: ```release-note NONE ``` cc @deads2k @soltysh

Merge pull request #67347 from juanvallejo/jvallejo/use-temp-file-config-tests
46bfa1ab · Kubernetes Submit Queue · GitHub · c65f65cf · 03af5089 · 46bfa1ab
Unverified Commit 46bfa1ab authored Aug 15, 2018 by Kubernetes Submit Queue Committed by GitHub Aug 15, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 0 deletions

client_config_test.go ...rc/k8s.io/client-go/tools/clientcmd/client_config_test.go +8 -0

No files found.
--- a/staging/src/k8s.io/client-go/tools/clientcmd/client_config_test.go
+++ b/staging/src/k8s.io/client-go/tools/clientcmd/client_config_test.go
@@ -132,9 +132,17 @@ func TestModifyContext(t *testing.T) {
 		"clean":   true,
 	}

+	tempPath, err := ioutil.TempFile("", "testclientcmd-")
+	if err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+	defer os.Remove(tempPath.Name())
+
 	pathOptions := NewDefaultPathOptions()
 	config := createValidTestConfig()

+	pathOptions.GlobalFile = tempPath.Name()
+
 	// define new context and assign it - our path options config
 	config.Contexts["updated"] = &clientcmdapi.Context{
 		Cluster:  "updated",