tags: fixed permadiff for parent argument on google_tags_location_tag_binding (#14220) (#23331)

[upstream:732c7e545d70a00dd8c0d29cbca207b50e6fc15a]

Signed-off-by: Modular Magician <[email protected]>

Author: modular-magician

.changelog/14220.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+tags: fixed perma-diff for `parent` field in `google_tags_location_tag_binding` resource
+```

google/services/tags/resource_tags_location_tag_binding.go

@@ -48,10 +48,11 @@ func ResourceTagsLocationTagBinding() *schema.Resource {
 
 		Schema: map[string]*schema.Schema{
 			"parent": {
-				Type:        schema.TypeString,
-				Required:    true,
-				ForceNew:    true,
-				Description: `The full resource name of the resource the TagValue is bound to. E.g. //cloudresourcemanager.googleapis.com/projects/123`,
+				Type:             schema.TypeString,
+				Required:         true,
+				ForceNew:         true,
+				DiffSuppressFunc: tpgresource.ProjectNumberDiffSuppress,
+				Description:      `The full resource name of the resource the TagValue is bound to. E.g. //cloudresourcemanager.googleapis.com/projects/123`,
 			},
 			"tag_value": {
 				Type:        schema.TypeString,

google/services/tags/resource_tags_test.go

@@ -889,6 +889,76 @@ resource "google_tags_location_tag_binding" "binding" {
 `, context)
 }
 
+func TestAccTagsLocationTagBinding_locationTagBindingBasicWithProjectId(t *testing.T) {
+	t.Parallel()
+
+	context := map[string]interface{}{
+		"random_suffix": acctest.RandString(t, 10),
+	}
+
+	acctest.VcrTest(t, resource.TestCase{
+		PreCheck:                 func() { acctest.AccTestPreCheck(t) },
+		ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories(t),
+		ExternalProviders: map[string]resource.ExternalProvider{
+			"random": {},
+		},
+		CheckDestroy: testAccCheckTagsLocationTagBindingDestroyProducer(t),
+		Steps: []resource.TestStep{
+			{
+				Config: testAccTagsLocationTagBinding_locationTagBindingBasicExampleWithProjectId(context),
+			},
+			{
+				ResourceName:      "google_tags_location_tag_binding.binding",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
+		},
+	})
+}
+
+func testAccTagsLocationTagBinding_locationTagBindingBasicExampleWithProjectId(context map[string]interface{}) string {
+	return acctest.Nprintf(`
+data "google_project" "project" {
+}
+
+resource "google_tags_tag_key" "key" {
+	parent = "organizations/${data.google_project.project.org_id}"
+	short_name = "keyname%{random_suffix}"
+	description = "For a certain set of resources."
+}
+
+resource "google_tags_tag_value" "value" {
+	parent      = google_tags_tag_key.key.id
+	short_name  = "foo%{random_suffix}"
+	description = "For foo%{random_suffix} resources."
+}
+
+resource "google_cloud_run_service" "default" {
+	name     = "tf-test-cloudrun-srv%{random_suffix}"
+	location = "us-central1"
+  
+	template {
+	  spec {
+		containers {
+		  image = "us-docker.pkg.dev/cloudrun/container/hello"
+		}
+	  }
+	}
+  
+	traffic {
+	  percent         = 100
+	  latest_revision = true
+	}
+}
+  
+resource "google_tags_location_tag_binding" "binding" {
+	parent    = "//run.googleapis.com/projects/${data.google_project.project.project_id}/locations/${google_cloud_run_service.default.location}/services/${google_cloud_run_service.default.name}"
+	tag_value = google_tags_tag_value.value.id
+	location  = "us-central1"
+}
+`, context)
+}
+
 func testAccCheckTagsLocationTagBindingDestroyProducer(t *testing.T) func(s *terraform.State) error {
 	return func(s *terraform.State) error {
 		for name, rs := range s.RootModule().Resources {