wip

Author: dak1n1

kubernetes/provider.go

@@ -160,6 +160,7 @@ func Provider() *schema.Provider {
 				},
 				Description:   "Configuration block to use an exec-based credential plugin, e.g. call an external command to receive user credentials.",
 				ConflictsWith: []string{"config_path", "config_paths", "token", "client_certificate", "client_key", "username", "password"},
+				RequiredWith:  []string{"host"},
 			},
 		},
 

kubernetes/provider_test.go

@@ -673,39 +673,62 @@ func TestAccKubernetesProvider_config_paths_host_env_wantError(t *testing.T) {
 	})
 }
 
-func TestAccKubernetesProvider_token(t *testing.T) {
-
+func TestAccKubernetesProvider_host(t *testing.T) {
 	resource.Test(t, resource.TestCase{
 		PreCheck:          func() { testAccPreCheckInternal(t) },
 		ProviderFactories: testAccProviderFactoriesInternal,
 		Steps: []resource.TestStep{
 			{
 				Config: testAccKubernetesProviderConfig(
-					providerConfig_token("test-token") +
-						providerConfig_host("https://test-host"),
+					providerConfig_host("https://test-host") +
+						providerConfig_token("test-token"),
 				),
 				PlanOnly:           true,
 				ExpectNonEmptyPlan: true,
 			},
 			{
 				Config: testAccKubernetesProviderConfig(
-					providerConfig_token("test-token") +
-						providerConfig_host("test-host"),
+					providerConfig_host("http://test-host") +
+						providerConfig_token("test-token"),
+				),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_host("https://127.0.0.1") +
+						providerConfig_token("test-token"),
+				),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+			},
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_host("test-host") +
+						providerConfig_token("test-token"),
 				),
 				PlanOnly:           true,
 				ExpectNonEmptyPlan: true,
-				// "token": all of `host,token` must be specified
-				ExpectError: regexp.MustCompile(`Error: expected "host" to have a host, got test-host`),
+				ExpectError:        regexp.MustCompile(`Error: expected "host" to have a host, got test-host`),
 			},
 			{
 				Config: testAccKubernetesProviderConfig(
 					providerConfig_token("test-token"),
 				),
 				PlanOnly:           true,
 				ExpectNonEmptyPlan: true,
-				// "token": all of `host,token` must be specified
+				// "host": all of `host,token` must be specified
 				ExpectError: regexp.MustCompile("host,token"),
 			},
+			{
+				Config: testAccKubernetesProviderConfig(
+					providerConfig_exec("test-exec"),
+				),
+				PlanOnly:           true,
+				ExpectNonEmptyPlan: true,
+				// "exec": all of `host,exec` must be specified
+				ExpectError: regexp.MustCompile("exec,host"),
+			},
 		},
 	})
 }
@@ -774,7 +797,7 @@ func providerConfig_host(host string) string {
 `, host)
 }
 
-func provider_exec(clusterName string) string {
+func providerConfig_exec(clusterName string) string {
 	return fmt.Sprintf(`
   exec {
     api_version = "client.authentication.k8s.io/v1alpha1"