Fix issue where call was rejoined if user tried to open a deeplink of the currently active call

Author: Stream Bot

DemoApp/Sources/Components/Router.swift

@@ -77,6 +77,20 @@ final class Router: ObservableObject {
             return
         }
 
+        let deeplinkCallCid = callCid(
+            from: deeplinkInfo.callId,
+            callType: deeplinkInfo.callType
+        )
+
+        if
+            deeplinkInfo.baseURL == AppEnvironment.baseURL,
+            appState.activeCall?.cId == deeplinkCallCid {
+            log.debug(
+                "Request to handle deeplink \(url) ignored because the call is already active."
+            )
+            return
+        }
+
         if
             deeplinkInfo.baseURL != AppEnvironment.baseURL,
             let currentUser = appState.currentUser {

DemoApp/Sources/Extensions/DemoApp+Sentry.swift

@@ -6,7 +6,14 @@ import Foundation
 import Sentry
 import StreamVideo
 
+private let terminalLogsEnvironmentKey = "STREAM_VIDEO_TERMINAL_LOGS"
+
 func configureSentry() {
+    let terminalDestinationTypes: [LogDestination.Type] =
+        ProcessInfo.processInfo.environment[terminalLogsEnvironmentKey] == "1"
+            ? [ConsoleLogDestination.self]
+            : []
+
     if AppEnvironment.configuration.isRelease {
         // We're tracking Crash Reports / Issues from the Demo App to keep improving the SDK
         SentrySDK.start { options in
@@ -24,14 +31,14 @@ func configureSentry() {
             SentryLogDestination.self,
             MemoryLogDestination.self,
             OSLogDestination.self
-        ]
+        ] + terminalDestinationTypes
     } else {
         LogConfig.level = .debug
         LogConfig.webRTCLogsEnabled = true
         LogConfig.destinationTypes = [
             MemoryLogDestination.self,
             OSLogDestination.self
-        ]
+        ] + terminalDestinationTypes
     }
 }
 

SwiftUIDemoAppUITests/Tests/DeeplinkTests.swift

@@ -25,6 +25,10 @@ final class DeeplinkTests: StreamTestCase {
         static let deeplinkUrlWithCallIdInPath: URL = .init(string: "https://getstream.io/video/demos/join/test-call")!
         static let customScheme: URL = .init(string: "streamvideo://video/demos?id=test-call")!
         static let customSchemeWithCallIdInPath: URL = .init(string: "streamvideo://video/demos/join/test-call")!
+
+        static func customScheme(callId: String) -> URL {
+            .init(string: "streamvideo://video/demos?id=\(callId)")!
+        }
     }
 
     func test_associationFile_validationWasSuccessful() throws {
@@ -96,4 +100,38 @@ final class DeeplinkTests: StreamTestCase {
                 .assertParticipantsAreVisible(count: 1)
         }
     }
+
+    func test_customSchemeURL_forActiveCall_doesNotRejoinAfterLeaving() throws {
+        let activeCallId = "test-call"
+
+        GIVEN("user starts the call referenced by the deeplink") {
+            userRobot
+                .waitForAutoLogin()
+                .startCall(activeCallId)
+        }
+        WHEN("user opens a deeplink for the active call") {
+            openURL(MockDeeplink.customScheme(callId: activeCallId))
+        }
+        AND("user leaves the call") {
+            userRobot.endCall()
+        }
+        THEN("user stays out of the call") {
+            userRobot.assertThereAreNoCallControls()
+            XCTAssertTrue(
+                CallDetailsPage.callIdInputField.wait().exists,
+                "callIdInputField should appear"
+            )
+
+            let rejoinExpectation = XCTNSPredicateExpectation(
+                predicate: NSPredicate(format: "exists == true"),
+                object: CallPage.hangUpButton
+            )
+            rejoinExpectation.isInverted = true
+
+            XCTAssertEqual(
+                XCTWaiter.wait(for: [rejoinExpectation], timeout: 3),
+                .completed
+            )
+        }
+    }
 }