Fix document additions parameters

Author: bidoubiwa

Sources/MeiliSearch/Client.swift

@@ -156,7 +156,7 @@ public struct MeiliSearch {
     UID: String,
     documents: [T],
     encoder: JSONEncoder? = nil,
-    primaryKey: String?,
+    primaryKey: String? = nil,
     _ completion: @escaping (Result<Update, Swift.Error>) -> Void) where T: Encodable {
     self.documents.add(
       UID,
@@ -184,7 +184,7 @@ public struct MeiliSearch {
   public func addDocuments(
     UID: String,
     documents: Data,
-    primaryKey: String?,
+    primaryKey: String? = nil,
     _ completion: @escaping (Result<Update, Swift.Error>) -> Void) {
     self.documents.add(
       UID,
@@ -212,7 +212,7 @@ public struct MeiliSearch {
   public func updateDocuments(
     UID: String,
     documents: Data,
-    primaryKey: String?,
+    primaryKey: String? = nil,
     _ completion: @escaping (Result<Update, Swift.Error>) -> Void) {
     self.documents.update(
       UID,

Sources/MeiliSearch/Documents.swift

@@ -70,7 +70,7 @@ struct Documents {
   func add(
     _ UID: String,
     _ document: Data,
-    _ primaryKey: String?,
+    _ primaryKey: String? = nil,
     _ completion: @escaping (Result<Update, Swift.Error>) -> Void) {
 
     var query: String = "/indexes/\(UID)/documents"
@@ -93,7 +93,7 @@ struct Documents {
     _ UID: String,
     _ documents: [T],
     _ encoder: JSONEncoder? = nil,
-    _ primaryKey: String?,
+    _ primaryKey: String? =  nil,
     _ completion: @escaping (Result<Update, Swift.Error>) -> Void) where T: Encodable {
 
     var query: String = "/indexes/\(UID)/documents"
@@ -127,7 +127,7 @@ struct Documents {
   func update(
     _ UID: String,
     _ document: Data,
-    _ primaryKey: String?,
+    _ primaryKey: String? = nil,
     _ completion: @escaping (Result<Update, Swift.Error>) -> Void) {
 
     var query: String = "/indexes/\(UID)/documents"

Sources/MeiliSearch/Error.swift

@@ -89,7 +89,10 @@ public extension MeiliSearch {
       case .invalidJSON:
         return "Invalid json"
       case .invalidURL(let url):
-        return "Invalid URL: \(url)"
+        if let strUrl: String = url {
+          return "Invalid URL: \(strUrl)"
+        }
+        return "Invalid URL"
       case .meiliSearchCommunicationError(let message, let url):
         return "meiliSearchCommunicationError \(message) \(url) "
 

Tests/MeiliSearchUnitTests/DocumentsTests.swift

@@ -61,8 +61,8 @@ class DocumentsTests: XCTestCase {
 
     self.client.addDocuments(
       UID: uid,
-      documents: [movie],
-      primaryKey: "") { result in
+      documents: [movie]
+    ) { result in
 
       switch result {
       case .success(let update):
@@ -71,7 +71,6 @@ class DocumentsTests: XCTestCase {
       case .failure:
         XCTFail("Failed to add or replace Movies document")
       }
-
     }
 
     self.wait(for: [expectation], timeout: 1.0)
@@ -255,7 +254,6 @@ class DocumentsTests: XCTestCase {
     // Start the test with the mocked server
 
     let uid: String = "Movies"
-    let limit: Int = 10
 
     let expectation = XCTestExpectation(description: "Get Movies documents")