Merge pull request #800 from Iterable/bug-fix/MOB-7855-universal-link-behavior

Bug fix/mob 7855 universal link behavior

Author: Ayyanchira

swift-sdk/Internal/DeepLinkManager.swift

@@ -43,17 +43,18 @@ class DeepLinkManager: NSObject {
             // Always return true for deep link
             return (true, pending)
         } else {
+            var result: Bool = false
             if let action = IterableAction.actionOpenUrl(fromUrlString: url.absoluteString) {
                 let context = IterableActionContext(action: action, source: .universalLink)
 
-                ActionRunner.execute(action: action,
+                result = ActionRunner.execute(action: action,
                                              context: context,
                                              urlHandler: IterableUtil.urlHandler(fromUrlDelegate: urlDelegate,
                                                                                  inContext: context),
                                              urlOpener: urlOpener,
                                              allowedProtocols: allowedProtocols)
             }
-            return (false, Fulfill<IterableAttributionInfo?, Error>(value: nil))
+            return (result, Fulfill<IterableAttributionInfo?, Error>(value: nil))
         }
     }
 

tests/unit-tests/DeepLinkTests.swift

@@ -70,11 +70,10 @@ class DeepLinkTests: XCTestCase {
 
         let deepLinkManager = DeepLinkManager(redirectNetworkSessionProvider: createNoRedirectNetworkSessionProvider())
 
-        let (isIterableLink, _) = deepLinkManager.handleUniversalLink(URL(string: iterableNoRewriteURL)!,
+        let (_, _) = deepLinkManager.handleUniversalLink(URL(string: iterableNoRewriteURL)!,
                                                                       urlDelegate: mockUrlDelegate,
                                                                       urlOpener: MockUrlOpener())
-        
-        XCTAssertFalse(isIterableLink)
+
         wait(for: [expectation1], timeout: testExpectationTimeout)
     }