Rename PointOfSaleOrders to PointOfSaleOrderList

Author: staskus

Modules/Sources/Yosemite/PointOfSale/Orders/POSOrder.swift renamed to Modules/Sources/Yosemite/PointOfSale/OrderList/POSOrder.swift

@@ -0,0 +1,18 @@
+import Foundation
+import struct NetworkingCore.PagedItems
+
+public protocol PointOfSaleOrderListFetchStrategy {
+    func fetchOrders(pageNumber: Int) async throws -> PagedItems<POSOrder>
+}
+
+struct PointOfSaleDefaultOrderListFetchStrategy: PointOfSaleOrderListFetchStrategy {
+    private let orderListService: PointOfSaleOrderListServiceProtocol
+
+    init(orderListService: PointOfSaleOrderListServiceProtocol) {
+        self.orderListService = orderListService
+    }
+
+    func fetchOrders(pageNumber: Int) async throws -> PagedItems<POSOrder> {
+        try await orderListService.providePointOfSaleOrders(pageNumber: pageNumber)
+    }
+}

Modules/Sources/Yosemite/PointOfSale/Orders/POSOrderItem.swift renamed to Modules/Sources/Yosemite/PointOfSale/OrderList/POSOrderItem.swift

@@ -0,0 +1,24 @@
+import Foundation
+import class Networking.AlamofireNetwork
+import class Networking.OrdersRemote
+
+public protocol PointOfSaleOrderListFetchStrategyFactoryProtocol {
+    func defaultStrategy() -> PointOfSaleOrderListFetchStrategy
+}
+
+public final class PointOfSaleOrderListFetchStrategyFactory: PointOfSaleOrderListFetchStrategyFactoryProtocol {
+    private let siteID: Int64
+    private let ordersRemote: OrdersRemote
+
+    public init(siteID: Int64,
+                credentials: Credentials?) {
+        self.siteID = siteID
+        let network = AlamofireNetwork(credentials: credentials)
+        self.ordersRemote = OrdersRemote(network: network)
+    }
+
+    public func defaultStrategy() -> PointOfSaleOrderListFetchStrategy {
+        PointOfSaleDefaultOrderListFetchStrategy(orderListService: PointOfSaleOrderListService(siteID: siteID,
+                                                                                               ordersRemote: ordersRemote))
+    }
+}

Modules/Sources/Yosemite/PointOfSale/Orders/POSOrderRefund.swift renamed to Modules/Sources/Yosemite/PointOfSale/OrderList/POSOrderRefund.swift

@@ -4,7 +4,7 @@ import struct NetworkingCore.PagedItems
 import struct NetworkingCore.Order
 import protocol NetworkingCore.POSOrdersRemoteProtocol
 
-public final class PointOfSaleOrderService: PointOfSaleOrderServiceProtocol {
+public final class PointOfSaleOrderListService: PointOfSaleOrderListServiceProtocol {
     private let ordersRemote: POSOrdersRemoteProtocol
     private let siteID: Int64
 
@@ -32,9 +32,9 @@ public final class PointOfSaleOrderService: PointOfSaleOrderServiceProtocol {
                         hasMorePages: pagedOrders.hasMorePages,
                         totalItems: pagedOrders.totalItems)
         } catch AFError.explicitlyCancelled {
-            throw PointOfSaleOrderServiceError.requestCancelled
+            throw PointOfSaleOrderListServiceError.requestCancelled
         } catch {
-            throw PointOfSaleOrderServiceError.requestFailed
+            throw PointOfSaleOrderListServiceError.requestFailed
         }
     }
 }

Modules/Sources/Yosemite/PointOfSale/OrderList/PointOfSaleOrderListFetchStrategy.swift

@@ -1,11 +1,11 @@
 import Foundation
 import struct NetworkingCore.PagedItems
 
-public enum PointOfSaleOrderServiceError: Error, Equatable {
+public enum PointOfSaleOrderListServiceError: Error, Equatable {
     case requestFailed
     case requestCancelled
 }
 
-public protocol PointOfSaleOrderServiceProtocol {
+public protocol PointOfSaleOrderListServiceProtocol {
     func providePointOfSaleOrders(pageNumber: Int) async throws -> PagedItems<POSOrder>
 }

Modules/Sources/Yosemite/PointOfSale/OrderList/PointOfSaleOrderListFetchStrategyFactory.swift

@@ -6,13 +6,13 @@ import enum NetworkingCore.OrderStatusEnum
 
 final class PointOfSaleOrderServiceTests: XCTestCase {
     private let siteID: Int64 = 13092
-    private var orderProvider: PointOfSaleOrderServiceProtocol!
+    private var orderProvider: PointOfSaleOrderListServiceProtocol!
     private var mockOrdersRemote: MockPOSOrdersRemote!
 
     override func setUp() {
         super.setUp()
         mockOrdersRemote = MockPOSOrdersRemote()
-        orderProvider = PointOfSaleOrderService(siteID: siteID, ordersRemote: mockOrdersRemote)
+        orderProvider = PointOfSaleOrderListService(siteID: siteID, ordersRemote: mockOrdersRemote)
     }
 
     override func tearDown() {
@@ -22,14 +22,14 @@ final class PointOfSaleOrderServiceTests: XCTestCase {
     }
 
     func test_PointOfSaleOrderServiceProtocol_when_fails_request_with_requestFailed_then_throws_error() async throws {
-        let expectedError = PointOfSaleOrderServiceError.requestFailed
+        let expectedError = PointOfSaleOrderListServiceError.requestFailed
         mockOrdersRemote.mockPagedOrdersResult = .failure(expectedError)
 
         do {
             _ = try await orderProvider.providePointOfSaleOrders(pageNumber: 1)
             XCTFail("Expected an error, but got success.")
         } catch {
-            XCTAssertEqual(error as? PointOfSaleOrderServiceError, expectedError)
+            XCTAssertEqual(error as? PointOfSaleOrderListServiceError, expectedError)
         }
     }
 
@@ -106,7 +106,7 @@ final class PointOfSaleOrderServiceTests: XCTestCase {
         do {
             _ = try await orderProvider.providePointOfSaleOrders(pageNumber: 1)
             XCTFail("Expected error to be thrown")
-        } catch PointOfSaleOrderServiceError.requestFailed {
+        } catch PointOfSaleOrderListServiceError.requestFailed {
             // Expected
         } catch {
             XCTFail("Unexpected error occurred: \(error)")