Fix remarks

WultraMobileTokenSDK/Operations/Service/WMTOperationsImpl.swift

@@ -131,7 +131,7 @@ class WMTOperationsImpl<T: WMTUserOperation>: WMTOperations, WMTService {
     private var isPollingPaused: Bool { return pollingTimer?.isValid == false }
     private let pollingLock = WMTLock()
     private var notificationObservers = [NSObjectProtocol]()
-    private let minimumTimePollingInterval = TimeInterval(5)
+    private let minimumTimePollingInterval = 5.0
 
     /// Operation register holds operations in order
     private lazy var operationsRegister = OperationsRegister { [weak self] ops, added, removed in
@@ -406,8 +406,6 @@ class WMTOperationsImpl<T: WMTUserOperation>: WMTOperations, WMTService {
             return
         }
 
-        let requestStartDate = Date()
-
         networking.post(data: .init(), signedWith: .possession(), to: WMTOperationEndpoints.List<T>.endpoint) { response, error in
 
             assert(Thread.isMainThread)

WultraMobileTokenSDK/Operations/WMTOperations.swift

@@ -127,7 +127,7 @@ public protocol WMTOperations: AnyObject {
     ///
     /// If operations are already polling this call is ignored and
     /// polling interval won't be changed.
-    /// - Parameter interval: Polling interval
+    /// - Parameter interval: Polling interval, minimum is 5s
     /// - Parameter delayStart: When true, polling starts after
     ///                         the first `interval` time passes
     func startPollingOperations(interval: TimeInterval, delayStart: Bool)
@@ -158,14 +158,14 @@ public extension WMTOperations {
 
     /// Starts the operations polling.
     ///
-    /// The default value for `interval` is 7 seconds
+    /// Deafula implementation of startPollingOperations
+    /// The `interval` is set to 7 seconds, with a minimum value of 5 seconds.
     ///
-    /// If operations are already polling this call is ignored and
-    /// polling interval won't be changed.
-    /// - Parameter delayStart: When true, polling starts after
-    ///                         the first `interval` time passes
-    func startPollingOperations(delayStart: Bool) {
-        return startPollingOperations(interval: TimeInterval(7), delayStart: delayStart)
+    /// - Parameters:
+    ///   - interval: Default is set to 7 seconds, with a minimum value of 5 seconds.
+    ///   - delayStart: Default is set to true. Polling starts after the first `interval` time passes
+    func startPollingOperations() {
+        return startPollingOperations(interval: 7, delayStart: true)
     }
 }
 

WultraMobileTokenSDKTests/IntegrationTests.swift

@@ -356,16 +356,16 @@ class IntegrationTests: XCTestCase {
         XCTAssertFalse(ops.isPollingOperations)
         let delegate = OpDelegate()
         delegate.loadingCountCallback = { count in
-            if count == 4 {
+            if count == 3 {
                 self.ops.stopPollingOperations()
                 exp.fulfill()
             }
         }
         ops.delegate = delegate
-        ops.startPollingOperations(interval: 1, delayStart: false)
+        ops.startPollingOperations()
         XCTAssertTrue(ops.isPollingOperations)
 
-        waitForExpectations(timeout: 20, handler: nil)
+        waitForExpectations(timeout: 30, handler: nil)
 
         XCTAssertFalse(ops.isPollingOperations)
     }

docs/Changelog.md

@@ -3,7 +3,7 @@
 ## 1.10.0
 
 - Removed `currentServerTime` property [(#148)](https://github.com/wultra/mtoken-sdk-android/pull/139)
-- Implemented default and minimum pollingInterval [(#151)](https://github.com/wultra/mtoken-sdk-ios/pull/151)
+- Added default and minimum pollingInterval [(#151)](https://github.com/wultra/mtoken-sdk-ios/pull/151)
 
 ## 1.9.0 (Jan 24, 2024)