made some more tests

Author: dak1n1

kubernetes/provider_test.go

@@ -494,6 +494,40 @@ func TestAccKubernetesProvider_config_path(t *testing.T) {
 				PlanOnly:           true,
 				ExpectNonEmptyPlan: true,
 			},
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_config_path("./missing/file"),
+				),
+				ExpectError:        regexp.MustCompile("could not open kubeconfig"),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_config_path("./testdata/kubeconfig") +
+						providerConfig_token("test-token"),
+				),
+				ExpectError:        regexp.MustCompile(`"config_path": conflicts with token`),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_config_path("./testdata/kubeconfig") +
+						providerConfig_host("test-host"),
+				),
+				ExpectError:        regexp.MustCompile(`"config_path": conflicts with host`),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_config_path("./testdata/kubeconfig") +
+						providerConfig_config_context("test-context"),
+				),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
 		},
 	})
 }
@@ -503,6 +537,21 @@ func TestAccKubernetesProvider_config_paths(t *testing.T) {
 		PreCheck:          func() { testAccPreCheckInternal(t) },
 		ProviderFactories: testAccProviderFactoriesInternal,
 		Steps: []resource.TestStep{
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_config_paths(`["./testdata/kubeconfig"]`),
+				),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_config_paths(`["./testdata/kubeconfig"]`) +
+						providerConfig_config_context("test-context"),
+				),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
 			{
 				Config: testAccKubernetesProviderConfig(
 					providerConfig_config_paths(`["./missing/file", "./testdata/kubeconfig"]`),
@@ -583,6 +632,31 @@ func TestAccKubernetesProvider_config_paths_env_wantError(t *testing.T) {
 	})
 }
 
+func TestAccKubernetesProvider_config_paths_host_env_wantError(t *testing.T) {
+	wantError := `"config_paths": conflicts with host`
+
+	resource.Test(t, resource.TestCase{
+		PreCheck: func() {
+			testAccPreCheckInternal_setEnv(t, map[string]string{
+				"KUBE_CONFIG_PATHS": strings.Join([]string{
+					"./testdata/kubeconfig",
+					"./testdata/kubeconfig",
+				}, string(os.PathListSeparator)),
+				"KUBE_HOST": "test-host",
+			})
+		},
+		ProviderFactories: testAccProviderFactoriesInternal,
+		Steps: []resource.TestStep{
+			{
+				Config:             testAccKubernetesProviderConfig("# empty"),
+				ExpectError:        regexp.MustCompile(wantError),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
+		},
+	})
+}
+
 // testAccKubernetesProviderConfig is used together with the providerConfig_* functions
 // to assemble a Kubernetes provider configuration with interchangeable options.
 func testAccKubernetesProviderConfig(providerConfig string) string {
@@ -605,43 +679,49 @@ func providerConfig_config_path(path string) string {
 `, path)
 }
 
+func providerConfig_config_context(context string) string {
+	return fmt.Sprintf(`
+  config_context = "%s"
+`, context)
+}
+
 func providerConfig_config_paths(paths string) string {
 	return fmt.Sprintf(`
   config_paths = %s
 `, paths)
 }
 
-func testAccKubernetesProviderConfig_token(token string) string {
+func providerConfig_token(token string) string {
 	return fmt.Sprintf(`
-  token = %s
+  token = "%s"
 `, token)
 }
 
-func testAccKubernetesProviderConfig_ca_cert(ca_cert string) string {
+func providerConfig_ca_cert(ca_cert string) string {
 	return fmt.Sprintf(`
-  cluster_ca_certificate = %s
+  cluster_ca_certificate = "%s"
 `, ca_cert)
 }
 
-func testAccKubernetesProviderConfig_client_cert(client_cert string) string {
+func providerConfig_client_cert(client_cert string) string {
 	return fmt.Sprintf(`
-  client_certificate = %s
+  client_certificate = "%s"
 `, client_cert)
 }
 
-func testAccKubernetesProviderConfig_client_key(client_key string) string {
+func providerConfig_client_key(client_key string) string {
 	return fmt.Sprintf(`
-  client_key = %s
+  client_key = "%s"
 `, client_key)
 }
 
-func testAccKubernetesProviderConfig_host(host string) string {
+func providerConfig_host(host string) string {
 	return fmt.Sprintf(`
-  host = %s
+  host = "%s"
 `, host)
 }
 
-func testAccKubernetesProvider_exec(clusterName string) string {
+func provider_exec(clusterName string) string {
 	return fmt.Sprintf(`
   exec {
     api_version = "client.authentication.k8s.io/v1alpha1"