feat(request-builder): add orderNumber support on order service

Author: jherey

packages/api-request-builder/src/create-service.js

@@ -35,6 +35,7 @@ const requiredDefinitionProps = ['type', 'endpoint', 'features']
 
 function getIdOrKey(params: Object): string {
   if (params.id) return `/${params.id}`
+  if (params.orderNumber) return `/order-number=${params.orderNumber}`
   if (params.key && !params.container) return `/key=${params.key}`
   if (params.key && params.container)
     return `/${params.container}/${params.key}`

packages/api-request-builder/src/default-params.js

@@ -133,6 +133,7 @@ export function setParams(params: ServiceBuilderParams) {
     'withTotal',
     'applyOrderEditTo',
     'container',
+    'orderNumber',
   ]
   Object.keys(params).forEach((key: string) => {
     if (!knownKeys.includes(key)) throw new Error(`Unknown key "${key}"`)
@@ -173,6 +174,7 @@ export function setParams(params: ServiceBuilderParams) {
   if (hasKey(params, 'priceCustomerGroup'))
     this.priceCustomerGroup(params.priceCustomerGroup)
   if (hasKey(params, 'priceChannel')) this.priceChannel(params.priceChannel)
+  if (hasKey(params, 'orderNumber')) this.byOrderNumber(params.orderNumber)
 
   // query-search
   if (params.text) this.text(params.text.value, params.text.language)

packages/api-request-builder/src/query-id.js

@@ -69,6 +69,24 @@ export function byCustomerId(custId: string): Object {
   return this
 }
 
+/**
+ * Set the given `orderNumber` to the `orderNumber` internal state of the service instance.
+ * For querying orders
+ *
+ * @param  {number} orderNumber - An order number
+ * @throws If `orderNumber` is missing or invalid
+ * @return {Object} The instance of the service, can be chained.
+ */
+export function byOrderNumber(orderNumber: number): Object {
+  if (typeof orderNumber !== 'number')
+    throw new Error(
+      'Required argument for `byOrderNumber` is missing or invalid'
+    )
+
+  this.params.orderNumber = orderNumber
+  return this
+}
+
 /**
  * Set the given `id` to the `cartId` internal state of the service instance.
  * For querying shipping methods by cart id

packages/api-request-builder/test/create-service.spec.js

@@ -426,7 +426,7 @@ describe('createService', () => {
 
       test('should mix orderNumber and queryParams', () => {
         expect(service.byOrderNumber(123).expand('baz').build()).toBe(
-          '/my-project1/test?orderNumber=123&expand=baz'
+          '/my-project1/test/order-number=123?expand=baz'
         )
       })
 

packages/api-request-builder/test/query-id.spec.js

@@ -53,7 +53,7 @@ describe('queryId', () => {
 
   test('should throw if orderNumber is invalid', () => {
     expect(() => service.byOrderNumber('hi')).toThrow(
-      /Required argument for `byCustomerId` is missing/
+      /Required argument for `byOrderNumber` is missing/
     )
   })
 

types/sdk.js

@@ -377,6 +377,9 @@ export type ServiceBuilderParams = {
 
   // container
   container?: ?string,
+
+  // params
+  orderNumber?: number,
 }
 export type ServiceBuilder = {
   type: string,