Fix the tests for datasource

GoogleCloudPlatform · Sep 15, 2023 · 65dc3ab · 65dc3ab
1 parent 64ad963
commit 65dc3ab
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 4 deletions.
diff --git a/mmv1/products/cloudidentity/Group.yaml b/mmv1/products/cloudidentity/Group.yaml
@@ -138,7 +138,7 @@ properties:
     output: true
     description: |
       The time when the Group was last updated.
-  - !ruby/object:Api::Type::KeyValueLabels
+  - !ruby/object:Api::Type::KeyValuePairs
     name: 'labels'
     required: true
     description: |

diff --git a/mmv1/third_party/terraform/acctest/test_utils.go.erb b/mmv1/third_party/terraform/acctest/test_utils.go.erb
@@ -50,6 +50,12 @@ func CheckDataSourceStateMatchesResourceStateWithIgnores(dataSourceName, resourc
 			if _, ok := ignoreFields[k]; ok {
 				continue
 			}
+			if _, ok := ignoreFields["labels.%"]; ok && strings.HasPrefix(k, "labels.") {
+				continue
+			}
+			if _, ok := ignoreFields["terraform_labels.%"]; ok && strings.HasPrefix(k, "terraform_labels.") {
+				continue
+			}
 			if k == "%" {
 				continue
 			}

diff --git a/...arty/terraform/services/artifactregistry/data_source_artifact_registry_repository_test.go b/...arty/terraform/services/artifactregistry/data_source_artifact_registry_repository_test.go
@@ -23,8 +23,8 @@ func TestAccDataSourceGoogleArtifactRegistryRepositoryConfig(t *testing.T) {
 			{
 				Config: testAccDataSourceGoogleArtifactRegistryRepositoryConfig(context),
 				Check: resource.ComposeTestCheckFunc(
-					acctest.CheckDataSourceStateMatchesResourceState(funcDataName,
-						"google_artifact_registry_repository.my-repo"),
+					acctest.CheckDataSourceStateMatchesResourceStateWithIgnores(funcDataName,
+						"google_artifact_registry_repository.my-repo", map[string]struct{}{"labels.%": {}, "terraform_labels.%": {}}),
 				),
 			},
 		},

diff --git a/...ty/terraform/services/cloudfunctions2/data_source_google_cloudfunctions2_function_test.go b/...ty/terraform/services/cloudfunctions2/data_source_google_cloudfunctions2_function_test.go
@@ -24,9 +24,11 @@ func TestAccDataSourceGoogleCloudFunctions2Function_basic(t *testing.T) {
 			{
 				Config: testAccDataSourceGoogleCloudFunctions2FunctionConfig(functionName,
 					bucketName, zipFilePath),
+				// As the value of "labels" and "terraform_lables" in the state is dpendent on the configuration,
+				// and these fields are not set in the configuration of the data source, so these fields are empty in the state of the data source.
 				Check: resource.ComposeTestCheckFunc(
 					acctest.CheckDataSourceStateMatchesResourceStateWithIgnores(funcDataNameHttp,
-						"google_cloudfunctions2_function.function_http_v2", map[string]struct{}{"build_config.0.source.0.storage_source.0.bucket": {}, "build_config.0.source.0.storage_source.0.object": {}}),
+						"google_cloudfunctions2_function.function_http_v2", map[string]struct{}{"build_config.0.source.0.storage_source.0.bucket": {}, "build_config.0.source.0.storage_source.0.object": {}, "labels.%": {}, "terraform_labels.%": {}}),
 				),
 			},
 		},