Use params over queryParams for backwards compatibility's sake

Sources/Networking/Async Api/Get.swift

@@ -34,7 +34,7 @@ public extension NetworkingClient {
     }
 
     func get(_ route: String, params: Params? = nil) async throws -> Data {
-        try await request(.get, route, queryParams: params).execute()
+        try await request(.get, route, params: params).execute()
     }
 }
 

Sources/Networking/Combine Api/Get+Combine.swift

@@ -41,7 +41,7 @@ public extension NetworkingClient {
     }
 
     func get(_ route: String, params: Params? = nil) -> AnyPublisher<Data, Error> {
-        request(.get, route, queryParams: params).publisher()
+        request(.get, route, params: params).publisher()
     }
 }
 

Sources/Networking/NetworkingClient+Requests.swift

@@ -10,8 +10,8 @@ import Combine
 
 public extension NetworkingClient {
 
-    func getRequest(_ route: String, queryParams: Params? = nil) -> NetworkingRequest {
-        request(.get, route, queryParams: queryParams)
+    func getRequest(_ route: String, params: Params? = nil) -> NetworkingRequest {
+        request(.get, route, params: params)
     }
 
     func postRequest(_ route: String, body: HTTPBody? = nil) -> NetworkingRequest {
@@ -32,13 +32,13 @@ public extension NetworkingClient {
 
     internal func request(_ httpMethod: HTTPMethod,
                           _ route: String,
-                          queryParams: Params? = nil,
+                          params: Params? = nil,
                           body: HTTPBody? = nil
     ) -> NetworkingRequest {
         let req = NetworkingRequest()
         req.httpMethod             = httpMethod
         req.route                = route
-        req.queryParams = queryParams
+        req.params = params ?? Params()
         req.httpBody = body
 
         let updateRequest = { [weak req, weak self] in

Sources/Networking/NetworkingRequest.swift

@@ -17,7 +17,7 @@ public class NetworkingRequest: NSObject, URLSessionTaskDelegate {
     var route = ""
     var httpMethod = HTTPMethod.get
     var httpBody: HTTPBody? = nil
-    public var queryParams: Params? = nil
+    public var params: Params = Params()
     var headers = [String: String]()
     var logLevel: NetworkingLogLevel {
         get { return logger.logLevel }
@@ -134,14 +134,13 @@ public class NetworkingRequest: NSObject, URLSessionTaskDelegate {
 
     private func getURLWithParams() -> String {
         let urlString = baseURL + route
-        guard let queryParams else { return urlString }
-        if queryParams.isEmpty { return urlString }
+        if params.isEmpty { return urlString }
         guard let url = URL(string: urlString) else {
             return urlString
         }
         if var urlComponents = URLComponents(url: url ,resolvingAgainstBaseURL: false) {
             var queryItems = urlComponents.queryItems ?? [URLQueryItem]()
-            queryParams.forEach { param in
+            params.forEach { param in
                 // arrayParam[] syntax
                 if let array = param.value as? [CustomStringConvertible] {
                     array.forEach {

Tests/NetworkingTests/GetRequestTests.swift

@@ -63,7 +63,7 @@ final class GetRequestTests: XCTestCase {
         { "response": "OK" }
         """
 
-        let _:Void = try await network.get("/users", urlParams: ["search" : "lion"])
+        let _:Void = try await network.get("/users", params: ["search" : "lion"])
         XCTAssertEqual(MockingURLProtocol.currentRequest?.httpMethod, "GET")
         XCTAssertEqual(MockingURLProtocol.currentRequest?.url?.absoluteString, "https://mocked.com/users?search=lion")
     }