Reset KUBERNETES env variables to ensure there's no Kube connection f…

…or unit test (#379)
knative-extensions · Oct 23, 2024 · 72533af · 72533af
1 parent a6872a8
commit 72533af
Showing 1 changed file with 15 additions and 1 deletion.
diff --git a/pkg/k8s/kubeclient_test.go b/pkg/k8s/kubeclient_test.go
@@ -27,7 +27,7 @@ import (
 func TestNewClients(t *testing.T) {
 	basic, err := clientcmd.NewClientConfigFromBytes([]byte(basicKubeconfig))
 	assert.NilError(t, err)
-	t.Setenv("KUBECONFIG", "/var/blackhole/not-existing.yaml")
+	emptyKubeconfigEnv(t)
 	tcs := []newClientsTestCase{{
 		name:    "nil",
 		wantErr: k8s.ErrNoKubernetesConnection,
@@ -46,6 +46,20 @@ func TestNewClients(t *testing.T) {
 	}
 }
 
+func emptyKubeconfigEnv(t *testing.T) {
+	t.Helper()
+	t.Setenv("KUBECONFIG", "/var/blackhole/not-existing.yaml")
+	t.Setenv("KUBERNETES_MASTER", "")
+	t.Setenv("KUBERNETES_SERVICE_PORT_HTTPS", "")
+	t.Setenv("KUBERNETES_SERVICE_PORT", "")
+	t.Setenv("KUBERNETES_PORT_443_TCP", "")
+	t.Setenv("KUBERNETES_PORT_443_TCP_PROTO", "")
+	t.Setenv("KUBERNETES_PORT_443_TCP_ADDR", "")
+	t.Setenv("KUBERNETES_SERVICE_HOST", "")
+	t.Setenv("KUBERNETES_PORT", "")
+	t.Setenv("KUBERNETES_PORT_443_TCP_PORT", "")
+}
+
 func just(cc clientcmd.ClientConfig) func() (clientcmd.ClientConfig, error) {
 	return func() (clientcmd.ClientConfig, error) {
 		return cc, nil