Skip to content

Commit b09aa58

Browse files
committed
Add shipping lines to order creation
1 parent c5e15cc commit b09aa58

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed

Yosemite/Yosemite/Stores/OrderStore.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,7 @@ private extension OrderStore {
315315
/// Creates a manual order with the provided order details.
316316
///
317317
func createOrder(siteID: Int64, order: Order, onCompletion: @escaping (Result<Order, Error>) -> Void) {
318-
remote.createOrder(siteID: siteID, order: order, fields: [.status, .items, .billingAddress, .shippingAddress]) { [weak self] result in
318+
remote.createOrder(siteID: siteID, order: order, fields: [.status, .items, .billingAddress, .shippingAddress, .shippingLines]) { [weak self] result in
319319
switch result {
320320
case .success(let order):
321321
self?.upsertStoredOrdersInBackground(readOnlyOrders: [order], onCompletion: {

Yosemite/YosemiteTests/Stores/OrderStoreTests.swift

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -768,6 +768,28 @@ final class OrderStoreTests: XCTestCase {
768768
let receivedNote = try XCTUnwrap(request.parameters["customer_note"] as? String)
769769
assertEqual(receivedNote, note)
770770
}
771+
772+
func test_create_order_sends_expected_fields() throws {
773+
// Given
774+
let store = OrderStore(dispatcher: dispatcher, storageManager: storageManager, network: network)
775+
network.simulateResponse(requestUrlSuffix: "orders/963", filename: "order")
776+
777+
// When
778+
let action = OrderAction.createOrder(siteID: sampleSiteID, order: sampleOrder()) { _ in }
779+
store.onAction(action)
780+
781+
// Then
782+
let request = try XCTUnwrap(network.requestsForResponseData.last as? JetpackRequest)
783+
let receivedKeys = Array(request.parameters.keys).sorted()
784+
let expectedKeys = [
785+
"billing",
786+
"line_items",
787+
"shipping",
788+
"shipping_lines",
789+
"status"
790+
]
791+
assertEqual(expectedKeys, receivedKeys)
792+
}
771793
}
772794

773795

0 commit comments

Comments
 (0)