tgc-revival: unskip tests (#15914)

Author: zli82016

mmv1/products/compute/RegionHealthCheck.yaml

@@ -58,9 +58,9 @@ tgc_tests:
   - name: 'TestAccComputeRegionHealthCheck_typeTransition'
     skip: 'Test data has mismatched steps'
   - name: 'TestAccComputeRegionHealthCheck_grpcWithTls_create'
-    skip: 'Test data has mismatched steps'
+    skip: 'grpcTlsHealthCheck is not in CAI asset, but is required in this test.'
   - name: 'TestAccComputeRegionHealthCheck_grpcWithTls_update'
-    skip: 'Test data has mismatched steps'
+    skip: 'grpcTlsHealthCheck is not in CAI asset, but is required in this test.'
 sweeper:
   url_substitutions:
     - region: "us-central1"

mmv1/products/compute/Subnetwork.yaml

@@ -47,9 +47,6 @@ base_url: 'projects/{{project}}/regions/{{region}}/subnetworks'
 has_self_link: true
 include_in_tgc_next: true
 tgc_include_handwritten_tests: true
-tgc_tests:
-  - name: 'TestAccComputeSubnetwork_secondaryIpRanges'
-    skip: 'Test data has mismatched steps'
 immutable: true
 timeouts:
   insert_minutes: 20

mmv1/products/compute/UrlMap.yaml

@@ -38,10 +38,6 @@ async:
 collection_url_key: 'items'
 include_in_tgc_next: true
 tgc_include_handwritten_tests: true
-tgc_tests:
-  - name: 'TestAccComputeUrlMap_trafficDirectorRemoveRouteRule'
-    skip: 'Test data has mismatched steps'
-custom_code:
 examples:
   - name: 'url_map_bucket_and_service'
     primary_resource_id: 'urlmap'

mmv1/products/kms/CryptoKey.yaml

@@ -40,9 +40,6 @@ import_format:
   - '{{key_ring}}/cryptoKeys/{{name}}'
 include_in_tgc_next: true
 tgc_include_handwritten_tests: true
-tgc_tests:
-  - name: 'TestAccKmsCryptoKey_rotation'
-    skip: 'Test data has mismatched steps'
 timeouts:
   insert_minutes: 20
   update_minutes: 20

mmv1/third_party/terraform/services/compute/resource_compute_region_health_check_test.go

@@ -186,21 +186,51 @@ func TestAccComputeRegionHealthCheck_typeTransition(t *testing.T) {
 			{
 				Config: testAccComputeRegionHealthCheck_https(hckName),
 			},
+			{
+				ResourceName:      "google_compute_region_health_check.foobar",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
 			{
 				Config: testAccComputeRegionHealthCheck_http(hckName),
 			},
+			{
+				ResourceName:      "google_compute_region_health_check.foobar",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
 			{
 				Config: testAccComputeRegionHealthCheck_ssl(hckName),
 			},
+			{
+				ResourceName:      "google_compute_region_health_check.foobar",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
 			{
 				Config: testAccComputeRegionHealthCheck_tcp(hckName),
 			},
+			{
+				ResourceName:      "google_compute_region_health_check.foobar",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
 			{
 				Config: testAccComputeRegionHealthCheck_http2(hckName),
 			},
+			{
+				ResourceName:      "google_compute_region_health_check.foobar",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
 			{
 				Config: testAccComputeRegionHealthCheck_https(hckName),
 			},
+			{
+				ResourceName:      "google_compute_region_health_check.foobar",
+				ImportState:       true,
+				ImportStateVerify: true,
+			},
 		},
 	})
 }

mmv1/third_party/tgc_next/test/setup.go

@@ -39,6 +39,7 @@ type TgcMetadataPayload struct {
 	RawConfig        string                       `json:"raw_config"`
 	ResourceMetadata map[string]*ResourceMetadata `json:"resource_metadata"`
 	PrimaryResource  string                       `json:"primary_resource"`
+	CaiReadTime      time.Time                    `json:"cai_read_time"`
 }
 
 type ResourceTestData struct {