Fix forces replacement on empty array (#15618)

* Fix forces replacement on empty array * adding tests on dns_config for PR:#15618 * update tests on dns_config for PR:#15618 * update tests on dns_config for PR:#15618 Co-authored-by: Sébastien Caloone <sebastien.caloone@fr.clara.net>
hashicorp · Mar 3, 2022 · 55ceaa0 · 55ceaa0
1 parent 1cdd94b
commit 55ceaa0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/internal/services/containers/container_group_resource.go b/internal/services/containers/container_group_resource.go
@@ -1660,14 +1660,14 @@ func flattenContainerGroupDnsConfig(input *containerinstance.DNSConfiguration) [
 	// We're converting to TypeSet here from an API response that looks like "a b c" (assumes space delimited)
 	var searchDomains []string
 	if input.SearchDomains != nil {
-		searchDomains = strings.Split(*input.SearchDomains, " ")
+		searchDomains = strings.Fields(*input.SearchDomains)
 	}
 	output["search_domains"] = searchDomains
 
 	// We're converting to TypeSet here from an API response that looks like "a b c" (assumes space delimited)
 	var options []string
 	if input.Options != nil {
-		options = strings.Split(*input.Options, " ")
+		options = strings.Fields(*input.Options)
 	}
 	output["options"] = options
 

diff --git a/internal/services/containers/container_group_resource_test.go b/internal/services/containers/container_group_resource_test.go
@@ -1370,9 +1370,7 @@ resource "azurerm_container_group" "test" {
   }
 
   dns_config {
-    nameservers    = ["reddog.microsoft.com", "somecompany.somedomain"]
-    options        = ["one:option", "two:option", "red:option", "blue:option"]
-    search_domains = ["default.svc.cluster.local."]
+    nameservers = ["reddog.microsoft.com", "somecompany.somedomain"]
   }
 
   identity {