fix(openapi): remove payload from required properties in customer ord…

…er get response

fix(get.spec.ts): add null checks to order properties
fix(get.ts): change NotFoundError to return null instead of throwing an error

openapi/paths/customer/order/get/response.yaml

@@ -7,4 +7,3 @@ properties:
     $ref: get.yaml
 required:
   - success
-  - payload

src/functions/customer/services/order/get.spec.ts

@@ -24,8 +24,8 @@ describe("CustomerOrderServiceGet", () => {
       orderId,
     });
 
-    expect(order.line_items.length).toBe(2);
-    expect(order.fulfillments.length).toBe(2);
-    expect(order.refunds.length).toBe(1);
+    expect(order?.line_items.length).toBe(2);
+    expect(order?.fulfillments.length).toBe(2);
+    expect(order?.refunds.length).toBe(1);
   });
 });

src/functions/customer/services/order/get.ts

@@ -1,5 +1,4 @@
 import { OrderModel } from "~/functions/order/order.models";
-import { NotFoundError } from "~/library/handler";
 import {
   OrderAggregate,
   OrderLineItemsAggreate,
@@ -133,13 +132,7 @@ export const CustomerOrderServiceGet = async ({
   ]);
 
   if (orders.length === 0) {
-    throw new NotFoundError([
-      {
-        code: "custom",
-        message: "ORDER_NOT_FOUND",
-        path: ["lineItemId"],
-      },
-    ]);
+    return null;
   }
 
   return orders[0];