spanner: Fix applying default_labels changes

Before this fix just changing default_labels on the provider for a
google_spanner_instance resulted in this error:

googleapi: Error 400: Invalid UpdateInstance request.
[...]
Must specify a non-empty field mask

Fixes hashicorp/terraform-provider-google#18445

Added one acceptance tests to surface this issue.

- TestAccSpannerInstance_basicUpdateWithProviderDefaultLabels

Author: rickard-von-essen

mmv1/templates/terraform/encoders/spanner_instance_update.go.tmpl

@@ -21,6 +21,9 @@ if d.HasChange("display_name") {
 if d.HasChange("labels") {
 	updateMask = append(updateMask, "labels")
 }
+if d.HasChange("terraform_labels") {
+	updateMask = append(updateMask, "labels")
+}
 if d.HasChange("processing_units") {
 	updateMask = append(updateMask, "processingUnits")
 }

mmv1/third_party/terraform/services/spanner/resource_spanner_instance_test.go

@@ -36,6 +36,43 @@ func TestAccSpannerInstance_basic(t *testing.T) {
 	})
 }
 
+func TestAccSpannerInstance_basicUpdateWithProviderDefaultLabels(t *testing.T) {
+	t.Parallel()
+
+	idName := fmt.Sprintf("tf-test-%s", acctest.RandString(t, 10))
+	acctest.VcrTest(t, resource.TestCase{
+		PreCheck:                 func() { acctest.AccTestPreCheck(t) },
+		ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories(t),
+		CheckDestroy:             testAccCheckSpannerInstanceDestroyProducer(t),
+		Steps: []resource.TestStep{
+			{
+				Config: testAccSpannerInstance_basicWithProviderLabel(idName, false),
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttrSet("google_spanner_instance.basic", "state"),
+				),
+			},
+			{
+				ResourceName:            "google_spanner_instance.basic",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"labels", "terraform_labels"},
+			},
+			{
+				Config: testAccSpannerInstance_basicWithProviderLabel(idName, true),
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttrSet("google_spanner_instance.basic", "state"),
+				),
+			},
+			{
+				ResourceName:            "google_spanner_instance.basic",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"labels", "terraform_labels"},
+			},
+		},
+	})
+}
+
 func TestAccSpannerInstance_noNodeCountSpecified(t *testing.T) {
 	t.Parallel()
 
@@ -400,6 +437,62 @@ resource "google_spanner_instance" "basic" {
 `, name, name)
 }
 
+func testAccSpannerInstance_basicWithProviderLabel(name string, addLabel bool) string {
+	extraLabel := ""
+	if addLabel {
+		extraLabel = "\"key2\" = \"value2\""
+	}
+	return fmt.Sprintf(`
+provider "google" {
+  alias          = "with-labels"
+  default_labels = {
+    %s
+  }
+}
+
+resource "google_spanner_instance" "basic" {
+  provider     = google.with-labels
+  config       = "regional-us-central1"
+  name         = "%s"
+  display_name = "%s"
+
+  processing_units = 100
+
+  labels = {
+    "key1" = "value1"
+  }
+}
+`, extraLabel, name, name)
+}
+
+func testAccSpannerInstance_basicWithNodes(name string, nodes int) string {
+	return fmt.Sprintf(`
+resource "google_spanner_instance" "basic" {
+  name         = "%s"
+  config       = "regional-us-central1"
+  display_name = "%s-dname"
+
+  num_nodes                    = %d
+  edition                      = "ENTERPRISE"
+  default_backup_schedule_type = "NONE"
+}
+`, name, name, nodes)
+}
+
+func testAccSpannerInstance_basicWithProcessingUnits(name string, processingUnits int) string {
+	return fmt.Sprintf(`
+resource "google_spanner_instance" "basic" {
+  name         = "%s"
+  config       = "regional-us-central1"
+  display_name = "%s-dname"
+
+  processing_units             = %d
+  edition                      = "ENTERPRISE"
+  default_backup_schedule_type = "NONE"
+}
+`, name, name, processingUnits)
+}
+
 func testAccSpannerInstance_noNodeCountSpecified(name string) string {
 	return fmt.Sprintf(`
 resource "google_spanner_instance" "basic" {