Skip to content

Commit 99ddc33

Browse files
Fix several VCR tests, skip some others (#6449) (#4625)
* Fix several VCR tests, skip some others * Readd test Signed-off-by: Modular Magician <[email protected]> Signed-off-by: Modular Magician <[email protected]>
1 parent 5c4e818 commit 99ddc33

File tree

6 files changed

+20
-0
lines changed

6 files changed

+20
-0
lines changed

.changelog/6449.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
```release-note:none
2+
3+
```

google-beta/resource_cloudfunction2_function_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -259,13 +259,15 @@ resource "google_project_iam_member" "event-receiving" {
259259
project = "%{project}"
260260
role = "roles/eventarc.eventReceiver"
261261
member = "serviceAccount:${google_service_account.account.email}"
262+
depends_on = [google_project_iam_member.invoking]
262263
}
263264
264265
resource "google_project_iam_member" "artifactregistry-reader" {
265266
provider = google-beta
266267
project = "%{project}"
267268
role = "roles/artifactregistry.reader"
268269
member = "serviceAccount:${google_service_account.account.email}"
270+
depends_on = [google_project_iam_member.event-receiving]
269271
}
270272
271273
resource "google_cloudfunctions2_function" "function" {

google-beta/resource_cloudfunctions2_function_generated_test.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -271,18 +271,21 @@ resource "google_project_iam_member" "invoking" {
271271
project = "%{project}"
272272
role = "roles/run.invoker"
273273
member = "serviceAccount:${google_service_account.account.email}"
274+
depends_on = [google_project_iam_member.gcs-pubsub-publishing]
274275
}
275276
276277
resource "google_project_iam_member" "event-receiving" {
277278
project = "%{project}"
278279
role = "roles/eventarc.eventReceiver"
279280
member = "serviceAccount:${google_service_account.account.email}"
281+
depends_on = [google_project_iam_member.invoking]
280282
}
281283
282284
resource "google_project_iam_member" "artifactregistry-reader" {
283285
project = "%{project}"
284286
role = "roles/artifactregistry.reader"
285287
member = "serviceAccount:${google_service_account.account.email}"
288+
depends_on = [google_project_iam_member.event-receiving]
286289
}
287290
288291
resource "google_cloudfunctions2_function" "function" {
@@ -409,12 +412,14 @@ resource "google_project_iam_member" "event-receiving" {
409412
project = "%{project}"
410413
role = "roles/eventarc.eventReceiver"
411414
member = "serviceAccount:${google_service_account.account.email}"
415+
depends_on = [google_project_iam_member.invoking]
412416
}
413417
414418
resource "google_project_iam_member" "artifactregistry-reader" {
415419
project = "%{project}"
416420
role = "roles/artifactregistry.reader"
417421
member = "serviceAccount:${google_service_account.account.email}"
422+
depends_on = [google_project_iam_member.event-receiving]
418423
}
419424
420425
resource "google_cloudfunctions2_function" "function" {

google-beta/resource_sql_database_instance_test.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -761,6 +761,8 @@ func TestAccSqlDatabaseInstance_withPrivateNetwork_withAllocatedIpRange(t *testi
761761
}
762762

763763
func TestAccSqlDatabaseInstance_withPrivateNetwork_withAllocatedIpRangeReplica(t *testing.T) {
764+
// Service Networking
765+
skipIfVcr(t)
764766
t.Parallel()
765767

766768
databaseName := "tf-test-" + randString(t, 10)
@@ -792,6 +794,8 @@ func TestAccSqlDatabaseInstance_withPrivateNetwork_withAllocatedIpRangeReplica(t
792794
}
793795

794796
func TestAccSqlDatabaseInstance_withPrivateNetwork_withAllocatedIpRangeClone(t *testing.T) {
797+
// Service Networking
798+
skipIfVcr(t)
795799
t.Parallel()
796800

797801
databaseName := "tf-test-" + randString(t, 10)

google-beta/resource_sql_user_test.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -307,6 +307,7 @@ resource "google_sql_user" "user2" {
307307
instance = google_sql_database_instance.instance.name
308308
host = "gmail.com"
309309
password = "hunter2"
310+
depends_on = [google_sql_user.user1]
310311
}
311312
`, instance, password, disabled)
312313
}

website/docs/r/cloudfunctions2_function.html.markdown

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -192,18 +192,21 @@ resource "google_project_iam_member" "invoking" {
192192
project = "my-project-name"
193193
role = "roles/run.invoker"
194194
member = "serviceAccount:${google_service_account.account.email}"
195+
depends_on = [google_project_iam_member.gcs-pubsub-publishing]
195196
}
196197
197198
resource "google_project_iam_member" "event-receiving" {
198199
project = "my-project-name"
199200
role = "roles/eventarc.eventReceiver"
200201
member = "serviceAccount:${google_service_account.account.email}"
202+
depends_on = [google_project_iam_member.invoking]
201203
}
202204
203205
resource "google_project_iam_member" "artifactregistry-reader" {
204206
project = "my-project-name"
205207
role = "roles/artifactregistry.reader"
206208
member = "serviceAccount:${google_service_account.account.email}"
209+
depends_on = [google_project_iam_member.event-receiving]
207210
}
208211
209212
resource "google_cloudfunctions2_function" "function" {
@@ -302,12 +305,14 @@ resource "google_project_iam_member" "event-receiving" {
302305
project = "my-project-name"
303306
role = "roles/eventarc.eventReceiver"
304307
member = "serviceAccount:${google_service_account.account.email}"
308+
depends_on = [google_project_iam_member.invoking]
305309
}
306310
307311
resource "google_project_iam_member" "artifactregistry-reader" {
308312
project = "my-project-name"
309313
role = "roles/artifactregistry.reader"
310314
member = "serviceAccount:${google_service_account.account.email}"
315+
depends_on = [google_project_iam_member.event-receiving]
311316
}
312317
313318
resource "google_cloudfunctions2_function" "function" {

0 commit comments

Comments
 (0)