feat(medusa,inventory,types): Expand list-reservation capabilities

.changeset/smooth-numbers-whisper.md

@@ -0,0 +1,8 @@
+---
+"@medusajs/client-types": patch
+"@medusajs/inventory": patch
+"@medusajs/medusa": patch
+"@medusajs/types": patch
+---
+
+feat(client-types, inventory, medusa, types): add additional filtering capabilities to list-reservations

integration-tests/plugins/__tests__/inventory/reservation-items/index.js

@@ -204,6 +204,185 @@ describe("Inventory Items endpoints", () => {
       })
     })
 
+    describe("List reservation items", () => {
+      let item2
+      let location2
+      let reservation2
+
+      beforeEach(async () => {
+        const api = useApi()
+        const stockRes = await api.post(
+          `/admin/stock-locations`,
+          {
+            name: "Fake Warehouse 1",
+          },
+          adminHeaders
+        )
+
+        location2 = stockRes.data.stock_location.id
+
+        await salesChannelLocationService.associateLocation(
+          "test-channel",
+          location2
+        )
+
+        const inventoryItem1 = await inventoryService.createInventoryItem({
+          sku: "12345",
+        })
+        item2 = inventoryItem1.id
+
+        await inventoryService.createInventoryLevel({
+          inventory_item_id: item2,
+          location_id: location2,
+          stocked_quantity: 100,
+        })
+
+        order = await simpleOrderFactory(dbConnection, {
+          sales_channel: "test-channel",
+          line_items: [
+            {
+              variant_id: variantId,
+              quantity: 2,
+              id: "line-item-id-2",
+            },
+          ],
+          shipping_methods: [
+            {
+              shipping_option: {
+                region_id: regionId,
+              },
+            },
+          ],
+        })
+
+        const orderRes = await api.get(
+          `/admin/orders/${order.id}`,
+          adminHeaders
+        )
+
+        reservation2 = await inventoryService.createReservationItem({
+          line_item_id: "line-item-id-2",
+          inventory_item_id: item2,
+          location_id: location2,
+          description: "test description",
+          quantity: 1,
+        })
+      })
+
+      it("lists reservation items", async () => {
+        const api = useApi()
+
+        const reservationsRes = await api.get(
+          `/admin/reservations`,
+          adminHeaders
+        )
+        expect(reservationsRes.data.reservations.length).toBe(2)
+        expect(reservationsRes.data.reservations).toEqual(
+          expect.arrayContaining([
+            expect.objectContaining({
+              id: reservationItem.id,
+            }),
+            expect.objectContaining({
+              id: reservation2.id,
+            }),
+          ])
+        )
+      })
+
+      describe("Filters reservation items", () => {
+        it("filters by location", async () => {
+          const api = useApi()
+
+          const reservationsRes = await api.get(
+            `/admin/reservations?location_id[]=${locationId}`,
+            adminHeaders
+          )
+          expect(reservationsRes.data.reservations.length).toBe(1)
+          expect(reservationsRes.data.reservations[0].location_id).toBe(
+            locationId
+          )
+        })
+
+        it("filters by itemID", async () => {
+          const api = useApi()
+
+          const reservationsRes = await api.get(
+            `/admin/reservations?inventory_item_id[]=${item2}`,
+            adminHeaders
+          )
+          expect(reservationsRes.data.reservations.length).toBe(1)
+          expect(reservationsRes.data.reservations[0].inventory_item_id).toBe(
+            item2
+          )
+        })
+
+        it("filters by quantity", async () => {
+          const api = useApi()
+
+          const reservationsRes = await api.get(
+            `/admin/reservations?quantity[gt]=1`,
+            adminHeaders
+          )
+
+          expect(reservationsRes.data.reservations.length).toBe(1)
+          expect(reservationsRes.data.reservations[0].id).toBe(
+            reservationItem.id
+          )
+        })
+
+        it("filters by date", async () => {
+          const api = useApi()
+
+          const reservationsRes = await api.get(
+            `/admin/reservations?created_at[gte]=${new Date(
+              reservation2.created_at
+            ).toISOString()}`,
+            adminHeaders
+          )
+
+          expect(reservationsRes.data.reservations.length).toBe(1)
+          expect(reservationsRes.data.reservations[0].id).toBe(reservation2.id)
+        })
+
+        it("filters by description using equals", async () => {
+          const api = useApi()
+
+          const reservationsRes = await api
+            .get(
+              `/admin/reservations?description=test%20description`,
+              adminHeaders
+            )
+            .catch(console.log)
+
+          expect(reservationsRes.data.reservations.length).toBe(1)
+          expect(reservationsRes.data.reservations[0].id).toBe(reservation2.id)
+        })
+
+        it("filters by description using equals removes results", async () => {
+          const api = useApi()
+
+          const reservationsRes = await api.get(
+            `/admin/reservations?description=description`,
+            adminHeaders
+          )
+
+          expect(reservationsRes.data.reservations.length).toBe(0)
+        })
+
+        it("filters by description using contains", async () => {
+          const api = useApi()
+
+          const reservationsRes = await api.get(
+            `/admin/reservations?description[contains]=descri`,
+            adminHeaders
+          )
+
+          expect(reservationsRes.data.reservations.length).toBe(1)
+          expect(reservationsRes.data.reservations[0].id).toBe(reservation2.id)
+        })
+      })
+    })
+
     it("lists reservations with inventory_items and line items", async () => {
       const api = useApi()
 

packages/generated/client-types/src/lib/models/AdminGetReservationsParams.ts

@@ -37,6 +37,50 @@ export interface AdminGetReservationsParams {
      */
     gte?: number
   }
+  /**
+   * A param for search reservation descriptions
+   */
+  description?:
+    | string
+    | {
+        /**
+         * filter by reservation description containing search string.
+         */
+        contains?: string
+        /**
+         * filter by reservation description equals search string.
+         */
+        equals?: string
+        /**
+         * filter by reservation description starting with search string.
+         */
+        starts_with?: string
+        /**
+         * filter by reservation description ending with search string.
+         */
+        ends_with?: string
+      }
+  /**
+   * Date comparison for when resulting reservations were created.
+   */
+  created_at?: {
+    /**
+     * filter by dates less than this date
+     */
+    lt?: string
+    /**
+     * filter by dates greater than this date
+     */
+    gt?: string
+    /**
+     * filter by dates less than or equal to this date
+     */
+    lte?: string
+    /**
+     * filter by dates greater than or equal to this date
+     */
+    gte?: string
+  }
   /**
    * How many Reservations to skip in the result.
    */

packages/inventory/src/services/reservation-item.ts

@@ -60,6 +60,7 @@ export default class ReservationItemService {
     const itemRepository = manager.getRepository(ReservationItem)
 
     const query = buildQuery(selector, config) as FindManyOptions
+
     return await itemRepository.find(query)
   }
 
@@ -79,6 +80,7 @@ export default class ReservationItemService {
     const itemRepository = manager.getRepository(ReservationItem)
 
     const query = buildQuery(selector, config) as FindManyOptions
+
     return await itemRepository.findAndCount(query)
   }
 

packages/inventory/src/utils/query.ts

@@ -1,11 +1,11 @@
+import { EntityManager, FindOptionsWhere, ILike } from "typeorm"
 import {
   ExtendedFindConfig,
   FilterableInventoryItemProps,
   FindConfig,
 } from "@medusajs/types"
-import { objectToStringPath, buildQuery } from "@medusajs/utils"
-import { EntityManager, FindOptionsWhere, ILike } from "typeorm"
-import { InventoryItem } from "../models"
+import { InventoryItem, ReservationItem } from "../models"
+import { buildQuery, objectToStringPath } from "@medusajs/utils"
 
 export function getListQuery(
   manager: EntityManager,

packages/medusa/src/api/routes/admin/reservations/list-reservations.ts

@@ -1,8 +1,10 @@
-import { IsArray, IsOptional, IsString, ValidateNested } from "class-validator"
 import {
+  DateComparisonOperator,
   NumericalComparisonOperator,
+  StringComparisonOperator,
   extendedFindParamsMixin,
 } from "../../../../types/common"
+import { IsArray, IsOptional, IsString, ValidateNested } from "class-validator"
 import { Request, Response } from "express"
 
 import { EntityManager } from "typeorm"
@@ -65,6 +67,48 @@ import { joinLineItems } from "./utils/join-line-items"
  *         gte:
  *           type: number
  *           description: filter by reservation quantity greater than or equal to this number
+ *   - in: query
+ *     name: description
+ *     description: A param for search reservation descriptions
+ *     schema:
+ *       oneOf:
+ *         - type: string
+ *         - type: object
+ *           properties:
+ *             contains:
+ *               type: string
+ *               description: filter by reservation description containing search string.
+ *             equals:
+ *               type: string
+ *               description: filter by reservation description equals search string.
+ *             starts_with:
+ *               type: string
+ *               description: filter by reservation description starting with search string.
+ *             ends_with:
+ *               type: string
+ *               description: filter by reservation description ending with search string.
+ *   - in: query
+ *     name: created_at
+ *     description: Date comparison for when resulting reservations were created.
+ *     schema:
+ *       type: object
+ *       properties:
+ *         lt:
+ *            type: string
+ *            description: filter by dates less than this date
+ *            format: date
+ *         gt:
+ *            type: string
+ *            description: filter by dates greater than this date
+ *            format: date
+ *         lte:
+ *            type: string
+ *            description: filter by dates less than or equal to this date
+ *            format: date
+ *         gte:
+ *            type: string
+ *            description: filter by dates greater than or equal to this date
+ *            format: date
  *   - (query) offset=0 {integer} How many Reservations to skip in the result.
  *   - (query) limit=20 {integer} Limit the number of Reservations returned.
  *   - (query) expand {string} (Comma separated) Which fields should be expanded in the product category.
@@ -184,4 +228,13 @@ export class AdminGetReservationsParams extends extendedFindParamsMixin({
   @ValidateNested()
   @Type(() => NumericalComparisonOperator)
   quantity?: NumericalComparisonOperator
+
+  @IsOptional()
+  @ValidateNested()
+  @Type(() => DateComparisonOperator)
+  created_at?: DateComparisonOperator
+
+  @IsOptional()
+  @IsType([StringComparisonOperator, String])
+  description?: string | StringComparisonOperator
 }