Skip to content

Commit c3d134a

Browse files
andremionrahul-lohra
authored andcommitted
Give the reconnection recording test a window that outlives the reconnect
testReconnectionDuringCallRecording kept exhausting all 3 attempts on the nightly with 'expected Recording but was Reconnecting..'. The recording is server side and survives the user's reconnect fine; it was the test racing its own budget. The buddy participant stops the recording 30 seconds after its start request, the composite recorder alone needs 20-30s to start, and the drop plus reconnect plus the polling asserts consumed the rest on slow CI emulators, so the assert ran after the recording legitimately ended. Raise the window to 90 seconds. The plain recording test already uses 60 without a reconnect in the middle.
1 parent 7f6a006 commit c3d134a

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

demo-app/src/androidTestE2etestingDebug/kotlin/io/getstream/video/android/tests/ReconnectionTests.kt

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,9 +113,14 @@ class ReconnectionTests : StreamTestCase() {
113113
userRobot.joinCall()
114114
}
115115
step("AND participant joins the call") {
116+
// The recording window counts from the participant's start request, and the
117+
// composite recorder alone can take 20-30s to start. The window has to outlive
118+
// the drop, the reconnect and the final polling assert on a slow CI emulator,
119+
// otherwise the participant stops the recording on schedule before the assert
120+
// and the test fails on a recording that legitimately ended.
116121
participantRobot
117122
.setUserCount(participants)
118-
.setCallRecordingDuration(30)
123+
.setCallRecordingDuration(90)
119124
.joinCall(callId, actions = arrayOf(Actions.RECORD_CALL))
120125
}
121126
step("AND participant starts recording a call") {

0 commit comments

Comments
 (0)