Remove most instances of VCR skips referencing hashicorp/terraform-pr…

…ovider-google#14158 - Use of older TPG versions through ExternalProviders breaks VCR, so that is also removed from TestAccDataSourceGoogleFirebaseAppleAppConfig
SarahFrench · Oct 3, 2024 · f64f805 · f64f805
1 parent 1b0712f
commit f64f805
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 23 deletions.
diff --git a/mmv1/third_party/terraform/fwprovider/framework_provider_test.go.tmpl b/mmv1/third_party/terraform/fwprovider/framework_provider_test.go.tmpl
@@ -16,8 +16,6 @@ import (
 )
 
 func TestAccFrameworkProviderMeta_setModuleName(t *testing.T) {
-	// TODO: https://github.com/hashicorp/terraform-provider-google/issues/14158
-	acctest.SkipIfVcr(t)
 	t.Parallel()
 
 	moduleName := "my-module"
@@ -62,8 +60,6 @@ func TestAccFrameworkProviderBasePath_setInvalidBasePath(t *testing.T) {
 }
 
 func TestAccFrameworkProviderBasePath_setBasePath(t *testing.T) {
-	// TODO: https://github.com/hashicorp/terraform-provider-google/issues/14158
-	acctest.SkipIfVcr(t)
 	t.Parallel()
 
 	acctest.VcrTest(t, resource.TestCase{

diff --git a/...y/terraform/services/firebase/data_source_google_firebase_android_app_config_test.go.tmpl b/...y/terraform/services/firebase/data_source_google_firebase_android_app_config_test.go.tmpl
@@ -11,8 +11,6 @@ import (
 
 func TestAccDataSourceGoogleFirebaseAndroidAppConfig(t *testing.T) {
 	t.Parallel()
-	// Framework-based resources and datasources don't work with VCR yet
-	acctest.SkipIfVcr(t)
 
 	context := map[string]interface{}{
 		"project_id":   envvar.GetTestProjectFromEnv(),

diff --git a/...rty/terraform/services/firebase/data_source_google_firebase_apple_app_config_test.go.tmpl b/...rty/terraform/services/firebase/data_source_google_firebase_apple_app_config_test.go.tmpl
@@ -10,8 +10,6 @@ import (
 )
 
 func TestAccDataSourceGoogleFirebaseAppleAppConfig(t *testing.T) {
-	// TODO: https://github.com/hashicorp/terraform-provider-google/issues/14158
-	acctest.SkipIfVcr(t)
 	t.Parallel()
 
 	context := map[string]interface{}{
@@ -23,20 +21,9 @@ func TestAccDataSourceGoogleFirebaseAppleAppConfig(t *testing.T) {
 	}
 
 	acctest.VcrTest(t, resource.TestCase{
-		PreCheck: func() { acctest.AccTestPreCheck(t) },
+		PreCheck:     func() { acctest.AccTestPreCheck(t) },
+		CheckDestroy: testAccCheckFirebaseAppleAppDestroyProducer(t),
 		Steps: []resource.TestStep{
-			{
-				ExternalProviders: map[string]resource.ExternalProvider{
-					"google": {
-						VersionConstraint: "4.58.0",
-						Source:            "hashicorp/google{{- if ne $.TargetVersionName "ga" -}}-{{$.TargetVersionName}}{{- end }}",
-					},
-				},
-				Config: testAccDataSourceGoogleFirebaseAppleAppConfig(context),
-				Check: resource.ComposeTestCheckFunc(
-					testAccDataSourceFirebaseAppleAppConfigCheck("data.google_firebase_apple_app_config.my_app_config"),
-				),
-			},
 			{
 				ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories(t),
 				Config:                   testAccDataSourceGoogleFirebaseAppleAppConfig(context),

diff --git a/mmv1/third_party/terraform/services/firebase/resource_firebase_web_app_test.go.tmpl b/mmv1/third_party/terraform/services/firebase/resource_firebase_web_app_test.go.tmpl
@@ -16,8 +16,6 @@ import (
 )
 
 func TestAccFirebaseWebApp_firebaseWebAppFull(t *testing.T) {
-	// TODO: https://github.com/hashicorp/terraform-provider-google/issues/14158
-	acctest.SkipIfVcr(t)
 	t.Parallel()
 
 	context := map[string]interface{}{