Fix updateSubscriptions signature and data types

Author: roninopf

Tests/swift-sdk-swift-tests/IterableAPITests.swift

@@ -616,9 +616,11 @@ class IterableAPITests: XCTestCase {
 
     func testUpdateSubscriptions() {
         let expectation1 = expectation(description: "update subscriptions")
-        let emailListIds = ["[email protected]"]
-        let unsubscriptedChannelIds = ["channedl1", "channel2"]
-        let unsubscribedMessageTypeIds = ["messageType1" ,"messageType2"]
+        let campaignId = NSNumber(value: 23)
+        let templateId = NSNumber(value: 10)
+        let emailListIds = [NSNumber(value: 382)]
+        let unsubscriptedChannelIds = [NSNumber(value: 7845), NSNumber(value: 1048)]
+        let unsubscribedMessageTypeIds = [NSNumber(value: 5671), NSNumber(value: 9087)]
 
         let networkSession = MockNetworkSession(statusCode: 200)
         networkSession.callback = {(_,_,_) in
@@ -629,6 +631,8 @@ class IterableAPITests: XCTestCase {
                                queryParams: [])
 
             let body = networkSession.getRequestBody() as! [String : Any]
+            TestUtils.validateMatch(keyPath: KeyPath(AnyHashable.ITBL_KEY_CAMPAIGN_ID), value: campaignId, inDictionary: body)
+            TestUtils.validateMatch(keyPath: KeyPath(AnyHashable.ITBL_KEY_TEMPLATE_ID), value: templateId, inDictionary: body)
             TestUtils.validateMatch(keyPath: KeyPath(AnyHashable.ITBL_KEY_EMAIL_LIST_IDS), value: emailListIds, inDictionary: body)
             TestUtils.validateMatch(keyPath: KeyPath(AnyHashable.ITBL_KEY_UNSUB_CHANNEL), value: unsubscriptedChannelIds, inDictionary: body)
             TestUtils.validateMatch(keyPath: KeyPath(AnyHashable.ITBL_KEY_UNSUB_MESSAGE), value: unsubscribedMessageTypeIds, inDictionary: body)
@@ -637,7 +641,7 @@ class IterableAPITests: XCTestCase {
         let config = IterableConfig()
         TestUtils.getTestUserDefaults().set("[email protected]", forKey: .ITBL_USER_DEFAULTS_EMAIL_KEY)
         IterableAPI.initializeForTesting(apiKey: IterableAPITests.apiKey, config:config, networkSession: networkSession)
-        IterableAPI.updateSubscriptions(emailListIds, unsubscribedChannelIds: unsubscriptedChannelIds, unsubscribedMessageTypeIds: unsubscribedMessageTypeIds)
+        IterableAPI.updateSubscriptions(campaignId: campaignId, templateId: templateId, emailListIds: emailListIds, unsubscribedChannelIds: unsubscriptedChannelIds, unsubscribedMessageTypeIds: unsubscribedMessageTypeIds, subscribedMessageTypeIds: nil)
         wait(for: [expectation1], timeout: testExpectationTimeout)
     }
 

swift-sdk/ITBConsts.swift

@@ -60,6 +60,7 @@ public extension AnyHashable {
     static let ITBL_KEY_TOTAL = "total"
     static let ITBL_KEY_UNSUB_CHANNEL = "unsubscribedChannelIds"
     static let ITBL_KEY_UNSUB_MESSAGE = "unsubscribedMessageTypeIds"
+    static let ITBL_KEY_SUB_MESSAGE = "subscribedMessageTypeIds"
     static let ITBL_KEY_USER = "user"
     static let ITBL_KEY_USER_ID = "userId"
     static let ITBL_KEY_ACTION_IDENTIFIER = "actionIdentifier"

swift-sdk/Internal/IterableAPIInternal.swift

@@ -383,20 +383,34 @@ final class IterableAPIInternal : NSObject, PushTrackerProtocol {
         }
     }
 
-    func updateSubscriptions(_ emailListIds: [String]?, unsubscribedChannelIds: [String]?, unsubscribedMessageTypeIds: [String]?) {
+    func updateSubscriptions(campaignId: NSNumber?, templateId: NSNumber?, emailListIds: [NSNumber]?, unsubscribedChannelIds: [NSNumber]?, unsubscribedMessageTypeIds: [NSNumber]?, subscribedMessageTypeIds: [NSNumber]?) {
         var dictionary = [AnyHashable : Any]()
         addEmailOrUserId(args: &dictionary)
 
+        if let campaignId = campaignId?.intValue {
+            dictionary[.ITBL_KEY_CAMPAIGN_ID] = campaignId
+        }
+        
+        if let templateId = templateId?.intValue {
+            dictionary[.ITBL_KEY_TEMPLATE_ID] = templateId
+        }
+        
         if let emailListIds = emailListIds {
             dictionary[.ITBL_KEY_EMAIL_LIST_IDS] = emailListIds
         }
+        
         if let unsubscribedChannelIds = unsubscribedChannelIds {
             dictionary[.ITBL_KEY_UNSUB_CHANNEL] = unsubscribedChannelIds
         }
+        
         if let unsubscribedMessageTypeIds = unsubscribedMessageTypeIds {
             dictionary[.ITBL_KEY_UNSUB_MESSAGE] = unsubscribedMessageTypeIds
         }
 
+        if let subscribedMessageTypeIds = subscribedMessageTypeIds {
+            dictionary[.ITBL_KEY_SUB_MESSAGE] = subscribedMessageTypeIds
+        }
+        
         if let request = createPostRequest(forPath: .ITBL_PATH_UPDATE_SUBSCRIPTIONS, withBody: dictionary) {
             sendRequest(request, onSuccess: IterableAPIInternal.defaultOnSucess(identifier: "updateSubscriptions"), onFailure: IterableAPIInternal.defaultOnFailure(identifier: "updateSubscriptions"))
         }

swift-sdk/IterableAPI.swift

@@ -353,8 +353,8 @@ public final class IterableAPI : NSObject {
 
      - remark: passing in an empty array will clear subscription list, passing in nil will not modify the list
      */
-    @objc(updateSubscriptions:unsubscribedChannelIds:unsubscribedMessageTypeIds:) public static func updateSubscriptions(_ emailListIds: [String]?, unsubscribedChannelIds: [String]?, unsubscribedMessageTypeIds: [String]?) {
-        internalImplementation?.updateSubscriptions(emailListIds, unsubscribedChannelIds: unsubscribedChannelIds, unsubscribedMessageTypeIds: unsubscribedMessageTypeIds)
+    @objc(updateSubscriptions:templateId:emailListIds:unsubscribedChannelIds:unsubscribedMessageTypeIds:subscribedMessageTypeIds:) public static func updateSubscriptions(campaignId: NSNumber?, templateId: NSNumber?, emailListIds: [NSNumber]?, unsubscribedChannelIds: [NSNumber]?, unsubscribedMessageTypeIds: [NSNumber]?, subscribedMessageTypeIds: [NSNumber]?) {
+        internalImplementation?.updateSubscriptions(campaignId: campaignId, templateId: templateId, emailListIds: emailListIds, unsubscribedChannelIds: unsubscribedChannelIds, unsubscribedMessageTypeIds: unsubscribedMessageTypeIds, subscribedMessageTypeIds: subscribedMessageTypeIds)
     }
 
     //MARK: In-App Notifications