Skip to content

Commit 76d204b

Browse files
committed
Refactor test_conference_recordings to simplify recording availability check by removing retry logic and adding a fixed sleep duration
1 parent 33189da commit 76d204b

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

test/smoke/test_conferences_api.py

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -270,16 +270,12 @@ def test_conference_recordings(self) -> None:
270270
BW_ACCOUNT_ID, conference_id, updateBxmlBody)
271271
assert_that(update_conference_bxml_response.status_code, 204)
272272

273-
max_attempts = 5
274-
list_conference_recordings_response: ApiResponse = None
275-
for _ in range(max_attempts):
276-
list_conference_recordings_response: ApiResponse = self.conference_api_instance.list_conference_recordings_with_http_info(
277-
BW_ACCOUNT_ID, conference_id)
278-
if list_conference_recordings_response.status_code == 200:
279-
break
280-
time.sleep(self.TEST_SLEEP_LONG)
281-
else:
282-
raise AssertionError(f"Conference recordings not available. Response:{list_conference_recordings_response.data}")
273+
# Sleep 15 seconds to ensure recording exists
274+
time.sleep(15)
275+
276+
list_conference_recordings_response: ApiResponse = self.conference_api_instance.list_conference_recordings_with_http_info(
277+
BW_ACCOUNT_ID, conference_id)
278+
assert_that(list_conference_recordings_response.status_code, 200)
283279

284280
conference_recordings = list_conference_recordings_response.data
285281
assert_that(len(conference_recordings), greater_than(0))

0 commit comments

Comments
 (0)