Skip to content

Commit 87e1f5a

Browse files
committed
Unskip test_conference_recordings and adjust polling logic for recording verification
1 parent 360b6ab commit 87e1f5a

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

test/smoke/test_conferences_api.py

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,6 @@ def test_conference_and_members(self):
248248
BW_ACCOUNT_ID, call_id, update_call
249249
)
250250

251-
# @unittest.skip("PV Issues")
252251
def test_conference_recordings(self) -> None:
253252
"""
254253
Tests a successful flow of creating a call with a recording.
@@ -261,8 +260,6 @@ def test_conference_recordings(self) -> None:
261260
answer_url = MANTECA_BASE_URL + "bxml/joinConferencePause"
262261
(test_id, call_id, conference_id) = self.create_conference(answer_url)
263262

264-
print(test_id, call_id, conference_id)
265-
266263
list_conferences_response = self.conference_api_instance.list_conferences(
267264
BW_ACCOUNT_ID)
268265

@@ -273,16 +270,8 @@ def test_conference_recordings(self) -> None:
273270
BW_ACCOUNT_ID, conference_id, updateBxmlBody)
274271
assert_that(update_conference_bxml_response.status_code, 204)
275272

276-
# Poll Manteca to ensure our conference is recorded
277-
call_status = self.get_test_status(test_id)
278-
retries = 0
279-
while call_status['callRecorded'] == False and retries < self.MAX_RETRIES:
280-
time.sleep(self.TEST_SLEEP)
281-
call_status = self.get_test_status(test_id)
282-
retries += 1
283-
284-
# If we failed to get a recorded conference, fail due to polling timeout
285-
assert call_status['callRecorded'] == True
273+
# Sleep 30 seconds to ensure recording exists
274+
time.sleep(30)
286275

287276
list_conference_recordings_response: ApiResponse = self.conference_api_instance.list_conference_recordings_with_http_info(
288277
BW_ACCOUNT_ID, conference_id)

0 commit comments

Comments
 (0)