Merge pull request #748 from Iterable/MOB-7857-Add-user-logged-in-check-for-jwt

[MOB-7857] - Add user logged in check

Author: Ayyanchira

swift-sdk/Internal/AuthManager.swift

@@ -142,7 +142,11 @@ class AuthManager: IterableAuthManagerProtocol {
         ITBInfo()
 
         expirationRefreshTimer = Timer.scheduledTimer(withTimeInterval: interval, repeats: false) { [weak self] _ in
-            self?.requestNewAuthToken(hasFailedPriorAuth: false)
+            if self?.localStorage.email != nil || self?.localStorage.userId != nil {
+                self?.requestNewAuthToken(hasFailedPriorAuth: false)
+            } else {
+                ITBDebug("Email or userId is not available. Skipping token refresh")
+            }
         }
     }
 

tests/unit-tests/AuthTests.swift

@@ -470,6 +470,35 @@ class AuthTests: XCTestCase {
         wait(for: [condition1], timeout: testExpectationTimeout)
     }
 
+    func testAuthTokenRefreshSkippedIfUserLoggedOutAfterReschedule() {
+        let callbackNotCalledExpectation = expectation(description: "\(#function) - Callback got called. Which it shouldn't when there is no userId or emailId in memory")
+        callbackNotCalledExpectation.isInverted = true
+        
+        let authDelegate = createAuthDelegate({
+            callbackNotCalledExpectation.fulfill()
+            return nil
+        })
+        
+        let expirationRefreshPeriod: TimeInterval = 0
+        let waitTime: TimeInterval = 1.0
+        let expirationTimeSinceEpoch = Date(timeIntervalSinceNow: expirationRefreshPeriod + waitTime).timeIntervalSince1970
+        let mockEncodedPayload = createMockEncodedPayload(exp: Int(expirationTimeSinceEpoch))
+        
+        let mockLocalStorage = MockLocalStorage()
+        mockLocalStorage.authToken = mockEncodedPayload
+        mockLocalStorage.email = nil
+        mockLocalStorage.userId = nil
+        
+        let authManager = AuthManager(delegate: authDelegate,
+                                      expirationRefreshPeriod: expirationRefreshPeriod,
+                                      localStorage: mockLocalStorage,
+                                      dateProvider: MockDateProvider())
+        
+        let _ = authManager
+        
+        wait(for: [callbackNotCalledExpectation], timeout: 2.0)
+    }
+    
     func testAuthTokenCallbackOnSetEmail() {
         let condition1 = expectation(description: "\(#function) - callback didn't get called after setEmail")