Merge pull request #249 from Iterable/MOB-1113-update-subscriptions

[MOB-1113] 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 = ["user1@example.com"]
-        let unsubscriptedChannelIds = ["channedl1", "channel2"]
-        let unsubscribedMessageTypeIds = ["messageType1" ,"messageType2"]
+        let emailListIds = [NSNumber(value: 382)]
+        let unsubscriptedChannelIds = [NSNumber(value: 7845), NSNumber(value: 1048)]
+        let unsubscribedMessageTypeIds = [NSNumber(value: 5671), NSNumber(value: 9087)]
+        let campaignId = NSNumber(value: 23)
+        let templateId = NSNumber(value: 10)
 
         let networkSession = MockNetworkSession(statusCode: 200)
         networkSession.callback = {(_,_,_) in
@@ -632,12 +634,19 @@ class IterableAPITests: XCTestCase {
             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)
+            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)
             expectation1.fulfill()
         }
         let config = IterableConfig()
         TestUtils.getTestUserDefaults().set("user1@example.com", forKey: .ITBL_USER_DEFAULTS_EMAIL_KEY)
         IterableAPI.initializeForTesting(apiKey: IterableAPITests.apiKey, config:config, networkSession: networkSession)
-        IterableAPI.updateSubscriptions(emailListIds, unsubscribedChannelIds: unsubscriptedChannelIds, unsubscribedMessageTypeIds: unsubscribedMessageTypeIds)
+        IterableAPI.updateSubscriptions(emailListIds,
+                                        unsubscribedChannelIds: unsubscriptedChannelIds,
+                                        unsubscribedMessageTypeIds: unsubscribedMessageTypeIds,
+                                        subscribedMessageTypeIds: nil,
+                                        campaignId: campaignId,
+                                        templateId: templateId)
         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,39 @@ final class IterableAPIInternal : NSObject, PushTrackerProtocol {
         }
     }
 
-    func updateSubscriptions(_ emailListIds: [String]?, unsubscribedChannelIds: [String]?, unsubscribedMessageTypeIds: [String]?) {
+    func updateSubscriptions(_ emailListIds: [NSNumber]? = nil,
+                             unsubscribedChannelIds: [NSNumber]? = nil,
+                             unsubscribedMessageTypeIds: [NSNumber]? = nil,
+                             subscribedMessageTypeIds: [NSNumber]? = nil,
+                             campaignId: NSNumber? = nil,
+                             templateId: NSNumber? = nil) {
         var dictionary = [AnyHashable : Any]()
         addEmailOrUserId(args: &dictionary)
 
         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 campaignId = campaignId?.intValue {
+            dictionary[.ITBL_KEY_CAMPAIGN_ID] = campaignId
+        }
+        
+        if let templateId = templateId?.intValue {
+            dictionary[.ITBL_KEY_TEMPLATE_ID] = templateId
+        }
+        
         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,19 @@ 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:unsubscribedChannelIds:unsubscribedMessageTypeIds:subscribedMessageTypeIds:campaignIds:templateId:)
+    public static func updateSubscriptions(_ emailListIds: [NSNumber]?,
+                                           unsubscribedChannelIds: [NSNumber]?,
+                                           unsubscribedMessageTypeIds: [NSNumber]?,
+                                           subscribedMessageTypeIds: [NSNumber]?,
+                                           campaignId: NSNumber?,
+                                           templateId: NSNumber?) {
+        internalImplementation?.updateSubscriptions(emailListIds,
+                                                    unsubscribedChannelIds: unsubscribedChannelIds,
+                                                    unsubscribedMessageTypeIds: unsubscribedMessageTypeIds,
+                                                    subscribedMessageTypeIds: subscribedMessageTypeIds,
+                                                    campaignId: campaignId,
+                                                    templateId: templateId)
     }
 
     //MARK: In-App Notifications