Skip to content

Commit 005c67c

Browse files
chore: bump wait times (#69)
1 parent 57b0e52 commit 005c67c

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

tests/integration/test_mysql_sync.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,8 @@ def setup_and_teardown():
7575
delete_database_user(sql_instance, mysql_username(test_user), credentials)
7676
# re-add member to IAM group
7777
add_iam_member(iam_groups[0], test_user, credentials)
78+
# wait 30 seconds, adding IAM member is slow
79+
time.sleep(30)
7880
except Exception:
7981
print("------------------------Cleanup Failed!------------------------")
8082

@@ -110,8 +112,8 @@ async def test_service_mysql(credentials):
110112
# make sure test_user is member of IAM group
111113
try:
112114
add_iam_member(iam_groups[0], test_user, credentials)
113-
# wait 5 seconds, adding IAM member is slow
114-
time.sleep(5)
115+
# wait 30 seconds, adding IAM member is slow
116+
time.sleep(30)
115117
except Exception:
116118
print("Member must already belong to IAM Group.")
117119

@@ -135,8 +137,8 @@ async def test_service_mysql(credentials):
135137
# remove test_user from IAM group
136138
delete_iam_member(iam_groups[0], test_user, credentials)
137139

138-
# wait 5 seconds, deleting IAM member is slow
139-
time.sleep(5)
140+
# wait 30 seconds, deleting IAM member is slow
141+
time.sleep(30)
140142

141143
# run groups sync
142144
await groups_sync(iam_groups, [sql_instance], credentials, False)

tests/integration/test_postgres_sync.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,8 @@ def setup_and_teardown():
7676
delete_database_user(sql_instance, test_user, credentials)
7777
# re-add member to IAM group
7878
add_iam_member(iam_groups[0], test_user, credentials)
79+
# wait 30 seconds, adding IAM member is slow
80+
time.sleep(30)
7981
except Exception:
8082
print("------------------------Cleanup Failed!------------------------")
8183

@@ -111,8 +113,8 @@ async def test_service_postgres(credentials):
111113
# make sure test_user is member of IAM group
112114
try:
113115
add_iam_member(iam_groups[0], test_user, credentials)
114-
# wait 5 seconds, adding IAM member is slow
115-
time.sleep(5)
116+
# wait 30 seconds, adding IAM member is slow
117+
time.sleep(30)
116118
except Exception:
117119
print("Member must already belong to IAM Group.")
118120

@@ -136,8 +138,8 @@ async def test_service_postgres(credentials):
136138
# remove test_user from IAM group
137139
delete_iam_member(iam_groups[0], test_user, credentials)
138140

139-
# wait 5 seconds, deleting IAM member is slow
140-
time.sleep(5)
141+
# wait 30 seconds, deleting IAM member is slow
142+
time.sleep(30)
141143

142144
# run groups sync
143145
await groups_sync(iam_groups, [sql_instance], credentials, False)
@@ -147,5 +149,5 @@ async def test_service_postgres(credentials):
147149
assert test_user not in users_with_role
148150

149151
# close aiohttp client session for graceful exit
150-
if client_session.closed:
152+
if not client_session.closed:
151153
await client_session.close()

0 commit comments

Comments
 (0)