Adds OrderRemote tests for notes and soem missing tests

Author: bummytime

Networking/NetworkingTests/Remote/OrdersRemoteTests.swift

@@ -26,6 +26,8 @@ class OrdersRemoteTests: XCTestCase {
     }
 
 
+    // MARK: - Load All Orders Tests
+
     /// Verifies that loadAllOrders properly parses the `orders-load-all` sample response.
     ///
     func testLoadAllOrdersProperlyReturnsParsedOrders() {
@@ -44,6 +46,24 @@ class OrdersRemoteTests: XCTestCase {
         wait(for: [expectation], timeout: Constants.expectationTimeout)
     }
 
+    /// Verifies that loadAllOrders properly relays Networking Layer errors.
+    ///
+    func testLoadAllOrdersProperlyRelaysNetwokingErrors() {
+        let remote = OrdersRemote(network: network)
+        let expectation = self.expectation(description: "Load All Orders")
+
+        remote.loadAllOrders(for: sampleSiteID) { orders, error in
+            XCTAssertNil(orders)
+            XCTAssertNotNil(error)
+            expectation.fulfill()
+        }
+
+        wait(for: [expectation], timeout: Constants.expectationTimeout)
+    }
+
+
+    // MARK: - Load Order Tests
+
     /// Verifies that loadOrder properly parses the `order` sample response.
     ///
     func testLoadSingleOrderProperlyReturnsParsedOrder() {
@@ -61,21 +81,24 @@ class OrdersRemoteTests: XCTestCase {
         wait(for: [expectation], timeout: Constants.expectationTimeout)
     }
 
-    /// Verifies that loadAllOrders properly relays Networking Layer errors.
+    /// Verifies that loadOrder properly relays any Networking Layer errors.
     ///
-    func testLoadAllOrdersProperlyRelaysNetwokingErrors() {
+    func testLoadSingleOrderProperlyRelaysNetwokingErrors() {
         let remote = OrdersRemote(network: network)
-        let expectation = self.expectation(description: "Load All Orders")
+        let expectation = self.expectation(description: "Update Order")
 
-        remote.loadAllOrders(for: sampleSiteID) { orders, error in
-            XCTAssertNil(orders)
+        remote.loadOrder(for: sampleSiteID, orderID: sampleOrderID) { order, error in
+            XCTAssertNil(order)
             XCTAssertNotNil(error)
             expectation.fulfill()
         }
 
         wait(for: [expectation], timeout: Constants.expectationTimeout)
     }
 
+
+    // MARK: - Update Orders Tests
+
     /// Verifies that updateOrder properly parses the `order` sample response.
     ///
     func testUpdateOrderProperlyReturnsParsedOrder() {
@@ -107,4 +130,40 @@ class OrdersRemoteTests: XCTestCase {
 
         wait(for: [expectation], timeout: Constants.expectationTimeout)
     }
+
+
+    // MARK: - Load Order Notes Tests
+
+    /// Verifies that loadOrderNotes properly parses the `order-notes` sample response.
+    ///
+    func testLoadOrderNotesProperlyReturnsParsedNotes() {
+        let remote = OrdersRemote(network: network)
+        let expectation = self.expectation(description: "Load Order Notes")
+
+        network.simulateResponse(requestUrlSuffix: "orders/\(sampleOrderID)/notes/", filename: "order-notes")
+
+        remote.loadOrderNotes(for: sampleSiteID, orderID: sampleOrderID) { orderNotes, error in
+            XCTAssertNil(error)
+            XCTAssertNotNil(orderNotes)
+            XCTAssertEqual(orderNotes?.count, 18)
+            expectation.fulfill()
+        }
+
+        wait(for: [expectation], timeout: Constants.expectationTimeout)
+    }
+
+    /// Verifies that loadOrderNotes properly relays any Networking Layer errors.
+    ///
+    func testLoadOrderNotesProperlyRelaysNetwokingErrors() {
+        let remote = OrdersRemote(network: network)
+        let expectation = self.expectation(description: "Load Order Notes")
+
+        remote.loadOrderNotes(for: sampleSiteID, orderID: sampleOrderID) { orderNotes, error in
+            XCTAssertNil(orderNotes)
+            XCTAssertNotNil(error)
+            expectation.fulfill()
+        }
+
+        wait(for: [expectation], timeout: Constants.expectationTimeout)
+    }
 }