Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions .changeset/cart-purchase-order-number.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
---
"@labdigital/commercetools-mock": minor
---

Model `purchaseOrderNumber` on carts: store it from `CartDraft` and support the
`setPurchaseOrderNumber` update action.

B2B checkout captures the buyer's purchase order number on the cart before the
order exists, but the mock dropped the draft field and rejected the action, so
that write path had no coverage available. The cart's `purchaseOrderNumber` now
also carries through to an order created from it, the way it does on
commercetools.
9 changes: 9 additions & 0 deletions src/repositories/cart/actions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ import type {
CartSetLineItemTaxAmountAction,
CartSetLineItemTaxRateAction,
CartSetLocaleAction,
CartSetPurchaseOrderNumberAction,
CartSetShippingAddressAction,
CartSetShippingAddressCustomFieldAction,
CartSetShippingAddressCustomTypeAction,
Expand Down Expand Up @@ -1120,6 +1121,14 @@ export class CartUpdateHandler
} as ItemShippingDetails;
}

setPurchaseOrderNumber(
_context: RepositoryContext,
resource: Writable<Cart>,
{ purchaseOrderNumber }: CartSetPurchaseOrderNumberAction,
) {
resource.purchaseOrderNumber = purchaseOrderNumber;
}

setLocale(
context: RepositoryContext,
resource: Writable<Cart>,
Expand Down
1 change: 1 addition & 0 deletions src/repositories/cart/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,7 @@ export class CartRepository extends AbstractResourceRepository<"cart"> {
shipping: [],
shippingInfo: undefined,
origin: draft.origin ?? "Customer",
purchaseOrderNumber: draft.purchaseOrderNumber,
refusedGifts: [],
custom: await createCustomFields(
draft.custom,
Expand Down
1 change: 1 addition & 0 deletions src/repositories/order/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,7 @@ export class OrderRepository extends AbstractResourceRepository<"order"> {
orderState: "Open",
origin: "Customer",
paymentInfo: cart.paymentInfo,
purchaseOrderNumber: cart.purchaseOrderNumber,
refusedGifts: [],
shipping: cart.shipping,
shippingAddress: cart.shippingAddress,
Expand Down
50 changes: 50 additions & 0 deletions src/services/cart.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -536,6 +536,56 @@ describe("Cart Update Actions", () => {
expect(response.json().lineItems).toHaveLength(1);
});

test("setPurchaseOrderNumber", async () => {
assert(cart, "cart not created");

const response = await ctMock.app.inject({
method: "POST",
url: `/dummy/carts/${cart.id}`,
payload: {
version: 1,
actions: [
{ action: "setPurchaseOrderNumber", purchaseOrderNumber: "PO-4711" },
],
},
});
expect(response.statusCode).toBe(200);
expect(response.json().version).toBe(2);
expect(response.json().purchaseOrderNumber).toBe("PO-4711");

const unset = await ctMock.app.inject({
method: "POST",
url: `/dummy/carts/${cart.id}`,
payload: {
version: 2,
actions: [{ action: "setPurchaseOrderNumber" }],
},
});
expect(unset.statusCode).toBe(200);
expect(unset.json().purchaseOrderNumber).toBeUndefined();
});

test("purchaseOrderNumber from the draft carries through to the order", async () => {
const created = await ctMock.app.inject({
method: "POST",
url: "/dummy/carts",
payload: { currency: "EUR", purchaseOrderNumber: "PO-4711" },
});
expect(created.statusCode).toBe(201);
expect(created.json().purchaseOrderNumber).toBe("PO-4711");

const order = await ctMock.app.inject({
method: "POST",
url: "/dummy/orders",
payload: {
cart: { typeId: "cart", id: created.json().id },
version: created.json().version,
},
});
expect(order.statusCode).toBe(201);
expect(order.json().purchaseOrderNumber).toBe("PO-4711");
});

test.each([
["EUR", 29800],
["GBP", 37800],
Expand Down
Loading