diff --git a/.changelog/3554.txt b/.changelog/3554.txt new file mode 100644 index 0000000000..8ec013c069 --- /dev/null +++ b/.changelog/3554.txt @@ -0,0 +1,3 @@ +```release-note:none + +``` diff --git a/google-beta/resource_bigtable_app_profile_test.go b/google-beta/resource_bigtable_app_profile_test.go index 665095e5de..96a2b758ac 100644 --- a/google-beta/resource_bigtable_app_profile_test.go +++ b/google-beta/resource_bigtable_app_profile_test.go @@ -8,6 +8,8 @@ import ( ) func TestAccBigtableAppProfile_update(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) diff --git a/google-beta/resource_bigtable_gc_policy_test.go b/google-beta/resource_bigtable_gc_policy_test.go index 367d2e94e1..bcb18dfe68 100644 --- a/google-beta/resource_bigtable_gc_policy_test.go +++ b/google-beta/resource_bigtable_gc_policy_test.go @@ -10,6 +10,8 @@ import ( ) func TestAccBigtableGCPolicy_basic(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) @@ -33,6 +35,8 @@ func TestAccBigtableGCPolicy_basic(t *testing.T) { } func TestAccBigtableGCPolicy_union(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) diff --git a/google-beta/resource_bigtable_instance_iam_test.go b/google-beta/resource_bigtable_instance_iam_test.go index b5ae8846f1..20659c62d0 100644 --- a/google-beta/resource_bigtable_instance_iam_test.go +++ b/google-beta/resource_bigtable_instance_iam_test.go @@ -8,6 +8,8 @@ import ( ) func TestAccBigtableInstanceIamBinding(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instance := "tf-bigtable-iam-" + randString(t, 10) @@ -51,6 +53,8 @@ func TestAccBigtableInstanceIamBinding(t *testing.T) { } func TestAccBigtableInstanceIamMember(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instance := "tf-bigtable-iam-" + randString(t, 10) @@ -89,6 +93,8 @@ func TestAccBigtableInstanceIamMember(t *testing.T) { } func TestAccBigtableInstanceIamPolicy(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instance := "tf-bigtable-iam-" + randString(t, 10) diff --git a/google-beta/resource_bigtable_instance_test.go b/google-beta/resource_bigtable_instance_test.go index 377fcbb3c9..a847b80161 100644 --- a/google-beta/resource_bigtable_instance_test.go +++ b/google-beta/resource_bigtable_instance_test.go @@ -11,6 +11,8 @@ import ( ) func TestAccBigtableInstance_basic(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) @@ -47,6 +49,8 @@ func TestAccBigtableInstance_basic(t *testing.T) { } func TestAccBigtableInstance_cluster(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) @@ -101,6 +105,8 @@ func TestAccBigtableInstance_cluster(t *testing.T) { } func TestAccBigtableInstance_development(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) @@ -124,6 +130,8 @@ func TestAccBigtableInstance_development(t *testing.T) { } func TestAccBigtableInstance_allowDestroy(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) diff --git a/google-beta/resource_bigtable_table_test.go b/google-beta/resource_bigtable_table_test.go index 0fbf709f85..490dd81d42 100644 --- a/google-beta/resource_bigtable_table_test.go +++ b/google-beta/resource_bigtable_table_test.go @@ -10,6 +10,8 @@ import ( ) func TestAccBigtableTable_basic(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) @@ -33,6 +35,8 @@ func TestAccBigtableTable_basic(t *testing.T) { } func TestAccBigtableTable_splitKeys(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) @@ -57,6 +61,8 @@ func TestAccBigtableTable_splitKeys(t *testing.T) { } func TestAccBigtableTable_family(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) @@ -81,6 +87,8 @@ func TestAccBigtableTable_family(t *testing.T) { } func TestAccBigtableTable_familyMany(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) @@ -105,6 +113,8 @@ func TestAccBigtableTable_familyMany(t *testing.T) { } func TestAccBigtableTable_familyUpdate(t *testing.T) { + // bigtable instance does not use the shared HTTP client, this test creates an instance + skipIfVcr(t) t.Parallel() instanceName := fmt.Sprintf("tf-test-%s", randString(t, 10)) diff --git a/google-beta/resource_compute_instance_migrate_test.go b/google-beta/resource_compute_instance_migrate_test.go index 231a6dca53..3790cc9e8b 100644 --- a/google-beta/resource_compute_instance_migrate_test.go +++ b/google-beta/resource_compute_instance_migrate_test.go @@ -930,6 +930,8 @@ func cleanUpDisk(config *Config, diskName, zone string) { } func getInitializedConfig(t *testing.T) *Config { + // Migrate tests are non standard and handle the config directly + skipIfVcr(t) // Check that all required environment variables are set testAccPreCheck(t)