wip

Author: dak1n1

kubernetes/provider_test.go

@@ -206,6 +206,22 @@ func testAccPreCheck(t *testing.T) {
 	return
 }
 
+// testAccPreCheckInternal configures the provider for internal tests.
+// This is the equivalent of running `terraform init`, but with a bare
+// minimum configuration, to create a fully separate environment where
+// all configuration options (including environment variables) can be
+// tested separately from the user's environment. It is used exclusively
+// in functions labelled testAccKubernetesProvider_*.
+func testAccPreCheckInternal(t *testing.T) {
+	ctx := context.TODO()
+	unsetEnv(t)
+	diags := testAccProvider.Configure(ctx, terraform.NewResourceConfigRaw(nil))
+	if diags.HasError() {
+		t.Fatal(diags[0].Summary)
+	}
+	return
+}
+
 func getClusterVersion() (*gversion.Version, error) {
 	meta := testAccProvider.Meta()
 
@@ -453,12 +469,12 @@ var testAccProviderFactoriesInternal = map[string]func() (*schema.Provider, erro
 func TestAccKubernetesProvider_config_path(t *testing.T) {
 
 	resource.Test(t, resource.TestCase{
-		PreCheck:          func() { testAccPreCheck(t); skipIfNotRunningInMinikube(t); unsetEnv(t) },
+		PreCheck:          func() { testAccPreCheckInternal(t) },
 		ProviderFactories: testAccProviderFactoriesInternal,
 		Steps: []resource.TestStep{
 			{
 				Config: testAccKubernetesProviderConfig(
-					providerConfig_config_path("~/.kube/config"),
+					providerConfig_config_path("./testdata/kubeconfig"),
 				),
 				Check: resource.TestCheckResourceAttr("data.kubernetes_namespace.test", "metadata.0.name", "default"),
 			},
@@ -470,26 +486,67 @@ func TestAccKubernetesProvider_config_paths(t *testing.T) {
 	wantError := `"config_path": conflicts with config_paths`
 
 	resource.Test(t, resource.TestCase{
-		PreCheck:          func() { skipIfNotRunningInMinikube(t); unsetEnv(t) },
+		// Unset env vars to prevent them from being used in the test.
+		PreCheck:          func() { testAccPreCheckInternal(t) },
 		ProviderFactories: testAccProviderFactoriesInternal,
 		Steps: []resource.TestStep{
 			{
 				Config: testAccKubernetesProviderConfig(
-					providerConfig_config_paths([]string{"~/.kube/config", "~/.kube/config2"}),
+					providerConfig_config_paths(`["./testdata/kubeconfig", "./testdata/kubeconfig"]`),
 				),
 				Check: resource.TestCheckResourceAttr("data.kubernetes_namespace.test", "metadata.0.name", "default"),
 			},
 			{
 				Config: testAccKubernetesProviderConfig(
-					providerConfig_config_path("~/.kube/config") +
-						providerConfig_config_paths([]string{"~/.kube/config", "~/.kube/config2"}),
+					providerConfig_config_path("./internal/testdata/kubeconfig") +
+						providerConfig_config_paths(`["./testdata/kubeconfig", "./testdata/kubeconfig"]`),
 				),
 				ExpectError: regexp.MustCompile(wantError),
 			},
 		},
 	})
 }
 
+func TestAccKubernetesProvider_config_paths_env(t *testing.T) {
+	os.Setenv("KUBE_CONFIG_PATHS", strings.Join([]string{
+		"./testdata/kubeconfig",
+		"./testdata/kubeconfig",
+	}, string(os.PathListSeparator)))
+
+	resource.Test(t, resource.TestCase{
+		PreCheck:          func() { testAccPreCheckInternal(t) },
+		ProviderFactories: testAccProviderFactoriesInternal,
+		Steps: []resource.TestStep{
+			{
+				Config: testAccKubernetesProviderConfig("# empty"),
+				Check:  resource.TestCheckResourceAttr("data.kubernetes_namespace.test", "metadata.0.name", "default"),
+			},
+		},
+	})
+}
+
+func TestAccKubernetesProvider_config_paths_env_wantError(t *testing.T) {
+	wantError := `"config_path": conflicts with config_paths`
+
+	unsetEnv(t)
+	os.Setenv("KUBE_CONFIG_PATHS", strings.Join([]string{
+		"testdata/kubeconfig",
+		"testdata/kubeconfig",
+	}, string(os.PathListSeparator)))
+
+	os.Setenv("KUBE_CONFIG_PATH", "testdata/kubeconfig")
+	resource.Test(t, resource.TestCase{
+		PreCheck:          func() { testAccPreCheckInternal(t) },
+		ProviderFactories: testAccProviderFactoriesInternal,
+		Steps: []resource.TestStep{
+			{
+				Config:      testAccKubernetesProviderConfig("# empty"),
+				ExpectError: regexp.MustCompile(wantError),
+			},
+		},
+	})
+}
+
 // testAccKubernetesProviderConfig is used together with the providerConfig_* functions
 // to assemble a Kubernetes provider configuration with interchangeable options.
 func testAccKubernetesProviderConfig(providerConfig string) string {
@@ -511,10 +568,10 @@ func providerConfig_config_path(path string) string {
 `, path)
 }
 
-func providerConfig_config_paths(paths []string) string {
+func providerConfig_config_paths(paths string) string {
 	return fmt.Sprintf(`
-  config_paths = [%q]
-`, strings.Join(paths, ", "))
+  config_paths = %s
+`, paths)
 }
 
 func testAccKubernetesProviderConfig_token(token string) string {