Merge pull request #177 from wordpress-mobile/release/4.3.1

Release 4.3.1

Author: etoledom

WordPressKit/EditorServiceRemote.swift

@@ -18,6 +18,35 @@ public class EditorServiceRemote: ServiceRemoteWordPressComREST {
         }
     }
 
+    public func postDesignateMobileEditorForAllSites(_ editor: EditorSettings.Mobile, setOnlyIfEmpty: Bool = true, success: @escaping ([Int: EditorSettings.Mobile]) -> Void, failure: @escaping (Error) -> Void) {
+        let endpoint = "me/gutenberg"
+        let path = self.path(forEndpoint: endpoint, withVersion: ._2_0)
+
+        let parameters = [
+            "platform": "mobile",
+            "editor": editor.rawValue,
+            "set_only_if_empty": setOnlyIfEmpty,
+        ] as [String: AnyObject]
+
+        wordPressComRestApi.POST(path, parameters: parameters, success: { (responseObject, httpResponse) in
+            guard let response = responseObject as? [String: String] else {
+                if let boolResponse = responseObject as? Bool, boolResponse == false {
+                    return failure(EditorSettings.Error.badRequest)
+                }
+                return failure(EditorSettings.Error.badResponse)
+            }
+
+            let mappedResponse = response.reduce(into: [Int: EditorSettings.Mobile](), { (result, response) in
+                if let id = Int(response.key), let editor = EditorSettings.Mobile(rawValue: response.value) {
+                    result[id] = editor
+                }
+            })
+            success(mappedResponse)
+        }) { (error, httpError) in
+            failure(error)
+        }
+    }
+
     public func getEditorSettings(_ siteID: Int, success: @escaping (EditorSettings) -> Void, failure: @escaping (Error) -> Void) {
         let endpoint = "sites/\(siteID)/gutenberg"
         let path = self.path(forEndpoint: endpoint, withVersion: ._2_0)

WordPressKit/EditorSettings.swift

@@ -6,6 +6,9 @@ private struct RemoteEditorSettings: Codable {
 public struct EditorSettings {
     public enum Error: Swift.Error {
         case decodingFailed
+        case unknownEditor(String)
+        case badRequest
+        case badResponse
     }
 
     /// Editor choosen by the user to be used on Mobile

WordPressKitTests/EditorServiceRemoteTests.swift

@@ -179,6 +179,62 @@ class EditorServiceRemoteTests: XCTestCase {
 
         wait(for: [expec], timeout: 0.1)
     }
+
+    func testPostDesignateGutenbergMobileEditorForAllSites() {
+        let expec = expectation(description: "success")
+        let editor = EditorSettings.Mobile.gutenberg
+
+        let response: [String: String] = [
+            "1": editor.rawValue,
+            "2": editor.rawValue,
+        ]
+
+        let expected: [Int: EditorSettings.Mobile] = [
+            1: editor,
+            2: editor
+        ]
+
+        editorServiceRemote.postDesignateMobileEditorForAllSites(editor, success: {
+            XCTAssertEqual($0, expected)
+            expec.fulfill()
+        }) { (error) in
+            XCTFail("This call should NOT error")
+            expec.fulfill()
+        }
+
+        mockRemoteApi.successBlockPassedIn?(response as AnyObject, HTTPURLResponse())
+        XCTAssertTrue(mockRemoteApi.postMethodCalled)
+
+        wait(for: [expec], timeout: 0.1)
+    }
+
+    func testPostDesignateAztecMobileEditorForAllSites() {
+        let expec = expectation(description: "success")
+        let editor = EditorSettings.Mobile.aztec
+
+        let response: [String: String] = [
+            "1": editor.rawValue,
+            "2": editor.rawValue,
+        ]
+
+        let expected: [Int: EditorSettings.Mobile] = [
+            1: editor,
+            2: editor
+        ]
+
+        editorServiceRemote.postDesignateMobileEditorForAllSites(editor, success: {
+            XCTAssertEqual($0, expected)
+            expec.fulfill()
+        }) { (error) in
+            XCTFail("This call should NOT error")
+            expec.fulfill()
+        }
+
+        mockRemoteApi.successBlockPassedIn?(response as AnyObject, HTTPURLResponse())
+        XCTAssertTrue(mockRemoteApi.postMethodCalled)
+
+        wait(for: [expec], timeout: 0.1)
+    }
 }
 
 extension EditorServiceRemoteTests {