Merge pull request #172 from wordpress-mobile/feature/9705-update-username

[Update Username] Map new property in account settings GET and new method

Author: danielebogo

WordPressKit.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name          = "WordPressKit"
-  s.version       = "4.3.0-beta.3"
+  s.version       = "4.3.0-beta.4"
   s.summary       = "WordPressKit offers a clean and simple WordPress.com and WordPress.org API."
 
   s.description   = <<-DESC

WordPressKit/AccountSettings.swift

@@ -9,6 +9,7 @@ public struct AccountSettings {
 
     // MARK: - Account Settings
     public let username: String    // user_login
+    public let usernameCanBeChanged: Bool // user_login_can_be_changed
     public let email: String       // user_email
     public let emailPendingAddress: String? // new_user_email
     public let emailPendingChange: Bool // user_email_change_pending
@@ -22,6 +23,7 @@ public struct AccountSettings {
                 displayName: String,
                 aboutMe: String,
                 username: String,
+                usernameCanBeChanged: Bool,
                 email: String,
                 emailPendingAddress: String?,
                 emailPendingChange: Bool,
@@ -34,6 +36,7 @@ public struct AccountSettings {
         self.displayName = displayName
         self.aboutMe = aboutMe
         self.username = username
+        self.usernameCanBeChanged = usernameCanBeChanged
         self.email = email
         self.emailPendingAddress = emailPendingAddress
         self.emailPendingChange = emailPendingChange

WordPressKit/AccountSettingsRemote.swift

@@ -90,6 +90,29 @@ public class AccountSettingsRemote: ServiceRemoteWordPressComREST {
                                  })
     }
 
+    /// Validate the current user's username
+    ///
+    /// - Parameters:
+    ///   - username: The new username
+    ///   - success: block for success
+    ///   - failure: block for failure
+    public func validateUsername(to username: String, success: @escaping () -> Void, failure: @escaping (Error) -> Void) {
+        let endpoint = "me/username/validate/\(username)"
+        let path = self.path(forEndpoint: endpoint, withVersion: ._1_1)
+        
+        wordPressComRestApi.GET(path,
+                                parameters: nil,
+                                success: { responseObject, httpResponse in
+                                    // The success block needs to be changed if
+                                    // any allowed_actions is required
+                                    // by the changeUsername API
+                                    success()
+        },
+                                 failure: { error, httpResponse in
+                                    failure(error)
+        })
+    }
+
     public func suggestUsernames(base: String, finished: @escaping ([String]) -> Void) {
         let endpoint = "wpcom/v2/users/username/suggestions"
         let parameters = ["name": base]
@@ -131,6 +154,7 @@ public class AccountSettingsRemote: ServiceRemoteWordPressComREST {
             let displayName = response["display_name"] as? String,
             let aboutMe = response["description"] as? String,
             let username = response["user_login"] as? String,
+            let usernameCanBeChanged = response["user_login_can_be_changed"] as? Bool,
             let email = response["user_email"] as? String,
             let emailPendingAddress = response["new_user_email"] as? String?,
             let emailPendingChange = response["user_email_change_pending"] as? Bool,
@@ -149,6 +173,7 @@ public class AccountSettingsRemote: ServiceRemoteWordPressComREST {
                                displayName: displayName,
                                aboutMe: aboutMeText!,
                                username: username,
+                               usernameCanBeChanged: usernameCanBeChanged,
                                email: email,
                                emailPendingAddress: emailPendingAddress,
                                emailPendingChange: emailPendingChange,

WordPressKitTests/AccountSettingsRemoteTests.swift

@@ -62,6 +62,7 @@ class AccountSettingsRemoteTests: RemoteTestCase, RESTTestable {
         stubRemoteResponse(meSettingsEndpoint, filename: getAccountSettingsSuccessMockFilename, contentType: .ApplicationJSON)
         remote.getSettings(success: { settings in
             XCTAssertEqual(settings.username, self.username, "The usernames should be equal.")
+            XCTAssertEqual(settings.usernameCanBeChanged, false, "The usernames can't be changed and its value must be false")
             XCTAssertEqual(settings.email, self.email, "The email addresses should be equal.")
             XCTAssertEqual(settings.webAddress, self.userURL, "The web addresses should be equal.")
             XCTAssertEqual(settings.primarySiteID, self.siteID, "The primary site ID's should be equal.")