From bb60203f30d4bc622bdd8d8dbe1d1601f060579c Mon Sep 17 00:00:00 2001
From: neil-yechenwei <yechenwei2007@hotmail.com>
Date: Wed, 1 Mar 2023 16:13:52 +0800
Subject: [PATCH] azurerm_communication_service - add forceNew for
 data_location

---
 .../communication_service_resource.go         |  1 +
 .../communication_service_resource_test.go    | 85 +------------------
 .../r/communication_service.html.markdown     |  2 +-
 3 files changed, 6 insertions(+), 82 deletions(-)

diff --git a/internal/services/communication/communication_service_resource.go b/internal/services/communication/communication_service_resource.go
index 9184eba56870..37af2edea362 100644
--- a/internal/services/communication/communication_service_resource.go
+++ b/internal/services/communication/communication_service_resource.go
@@ -58,6 +58,7 @@ func resourceArmCommunicationService() *pluginsdk.Resource {
 				Type: pluginsdk.TypeString,
 				// TODO: should this become Required and remove the default in 4.0?
 				Optional: true,
+				ForceNew: true,
 				Default:  "United States",
 				ValidateFunc: validation.StringInSlice([]string{
 					"Africa",
diff --git a/internal/services/communication/communication_service_resource_test.go b/internal/services/communication/communication_service_resource_test.go
index c9969ab535a4..85ff7aba9956 100644
--- a/internal/services/communication/communication_service_resource_test.go
+++ b/internal/services/communication/communication_service_resource_test.go
@@ -78,84 +78,7 @@ func TestAccCommunicationService_update(t *testing.T) {
 		},
 		data.ImportStep(),
 		{
-			Config: r.update(data, "Australia"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "Africa"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "Brazil"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "Canada"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "France"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "Germany"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "India"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "Japan"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "Korea"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "Norway"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "Switzerland"),
-			Check: acceptance.ComposeTestCheckFunc(
-				check.That(data.ResourceName).ExistsInAzure(r),
-			),
-		},
-		data.ImportStep(),
-		{
-			Config: r.update(data, "UAE"),
+			Config: r.update(data),
 			Check: acceptance.ComposeTestCheckFunc(
 				check.That(data.ResourceName).ExistsInAzure(r),
 			),
@@ -223,20 +146,20 @@ resource "azurerm_communication_service" "test" {
 `, r.template(data), data.RandomInteger)
 }
 
-func (r CommunicationServiceResource) update(data acceptance.TestData, dataLocation string) string {
+func (r CommunicationServiceResource) update(data acceptance.TestData) string {
 	return fmt.Sprintf(`
 %s
 
 resource "azurerm_communication_service" "test" {
   name                = "acctest-CommunicationService-%d"
   resource_group_name = azurerm_resource_group.test.name
-  data_location       = "%s"
+  data_location       = "United States"
 
   tags = {
     env = "Test2"
   }
 }
-`, r.template(data), data.RandomInteger, dataLocation)
+`, r.template(data), data.RandomInteger)
 }
 
 func (r CommunicationServiceResource) template(data acceptance.TestData) string {
diff --git a/website/docs/r/communication_service.html.markdown b/website/docs/r/communication_service.html.markdown
index 489ea5b05f7d..9d5aae8f44a8 100644
--- a/website/docs/r/communication_service.html.markdown
+++ b/website/docs/r/communication_service.html.markdown
@@ -35,7 +35,7 @@ The following arguments are supported:
 
 ---
 
-* `data_location` - (Optional) The location where the Communication service stores its data at rest. Possible values are `Africa`, `Asia Pacific`, `Australia`, `Brazil`, `Canada`, `Europe`, `France`, `Germany`, `India`, `Japan`, `Korea`, `Norway`, `Switzerland`, `UAE`, `UK` and `United States`. Defaults to `United States`.
+* `data_location` - (Optional) The location where the Communication service stores its data at rest. Possible values are `Africa`, `Asia Pacific`, `Australia`, `Brazil`, `Canada`, `Europe`, `France`, `Germany`, `India`, `Japan`, `Korea`, `Norway`, `Switzerland`, `UAE`, `UK` and `United States`. Defaults to `United States`. Changing this forces a new Communication Service to be created.
 
 * `tags` - (Optional) A mapping of tags which should be assigned to the Communication Service.