feat(medusa): Nested Categories Admin Update Endpoint

.changeset/hungry-starfishes-count.md

@@ -0,0 +1,5 @@
+---
+"@medusajs/medusa": patch
+---
+
+feat(medusa): added admin create endpoint for product categories

integration-tests/api/__tests__/admin/product-category.ts

@@ -18,6 +18,7 @@ describe("/admin/product-categories", () => {
   let medusaProcess
   let dbConnection
   let productCategory = null
+  let productCategory2 = null
   let productCategoryChild = null
   let productCategoryParent = null
   let productCategoryChild2 = null
@@ -234,6 +235,71 @@ describe("/admin/product-categories", () => {
     })
   })
 
+  describe("POST /admin/product-categories", () => {
+    beforeEach(async () => {
+      await adminSeeder(dbConnection)
+    })
+
+    afterEach(async () => {
+      const db = useDb()
+      return await db.teardown()
+    })
+
+    it("throws an error if required fields are missing", async () => {
+      const api = useApi()
+
+      const error = await api.post(
+        `/admin/product-categories`,
+        {},
+        adminHeaders
+      ).catch(e => e)
+
+      expect(error.response.status).toEqual(400)
+      expect(error.response.data.type).toEqual("invalid_data")
+      expect(error.response.data.message).toEqual(
+        "name should not be empty, name must be a string, handle should not be empty, handle must be a string"
+      )
+    })
+
+    it("successfully creates a product category", async () => {
+      productCategoryParent = await simpleProductCategoryFactory(dbConnection, {
+        name: "category parent",
+        handle: "category-parent",
+      })
+
+      const api = useApi()
+
+      const response = await api.post(
+        `/admin/product-categories`,
+        {
+          name: "test",
+          handle: "test",
+          is_internal: true,
+          parent_category_id: productCategoryParent.id,
+        },
+        adminHeaders
+      )
+
+      expect(response.status).toEqual(200)
+      expect(response.data).toEqual(
+        expect.objectContaining({
+          product_category: expect.objectContaining({
+            name: "test",
+            handle: "test",
+            is_internal: true,
+            is_active: false,
+            created_at: expect.any(String),
+            updated_at: expect.any(String),
+            parent_category: expect.objectContaining({
+              id: productCategoryParent.id
+            }),
+            category_children: []
+          }),
+        })
+      )
+    })
+  })
+
   describe("DELETE /admin/product-categories/:id", () => {
     beforeEach(async () => {
       await adminSeeder(dbConnection)
@@ -305,4 +371,95 @@ describe("/admin/product-categories", () => {
       expect(errorFetchingDeleted.response.status).toEqual(404)
     })
   })
+
+  describe("POST /admin/product-categories/:id", () => {
+    beforeEach(async () => {
+      await adminSeeder(dbConnection)
+
+      productCategory = await simpleProductCategoryFactory(dbConnection, {
+        name: "skinny jeans",
+        handle: "skinny-jeans",
+      })
+
+      productCategory2 = await simpleProductCategoryFactory(dbConnection, {
+        name: "sweater",
+        handle: "sweater",
+      })
+    })
+
+    afterEach(async () => {
+      const db = useDb()
+      return await db.teardown()
+    })
+
+    it("throws an error if invalid ID is sent", async () => {
+      const api = useApi()
+
+      const error = await api.post(
+        `/admin/product-categories/not-found-id`,
+        {
+          name: 'testing'
+        },
+        adminHeaders
+      ).catch(e => e)
+
+      expect(error.response.status).toEqual(404)
+      expect(error.response.data.type).toEqual("not_found")
+      expect(error.response.data.message).toEqual(
+        "ProductCategory with id: not-found-id was not found"
+      )
+    })
+
+    it("throws an error if invalid attribute is sent", async () => {
+      const api = useApi()
+
+      const error = await api.post(
+        `/admin/product-categories/${productCategory.id}`,
+        {
+          invalid_property: 'string'
+        },
+        adminHeaders
+      ).catch(e => e)
+
+      expect(error.response.status).toEqual(400)
+      expect(error.response.data.type).toEqual("invalid_data")
+      expect(error.response.data.message).toEqual(
+        "property invalid_property should not exist"
+      )
+    })
+
+    it("successfully updates a product category", async () => {
+      const api = useApi()
+
+      const response = await api.post(
+        `/admin/product-categories/${productCategory.id}`,
+        {
+          name: "test",
+          handle: "test",
+          is_internal: true,
+          is_active: true,
+          parent_category_id: productCategory2.id,
+        },
+        adminHeaders
+      )
+
+      expect(response.status).toEqual(200)
+      expect(response.data).toEqual(
+        expect.objectContaining({
+          product_category: expect.objectContaining({
+            name: "test",
+            handle: "test",
+            is_internal: true,
+            is_active: true,
+            created_at: expect.any(String),
+            updated_at: expect.any(String),
+            parent_category: expect.objectContaining({
+              id: productCategory2.id,
+            }),
+            category_children: []
+          }),
+        })
+      )
+    })
+  })
 })

packages/medusa/src/api/routes/admin/product-categories/create-product-category.ts

@@ -0,0 +1,133 @@
+import { IsNotEmpty, IsOptional, IsString, IsBoolean } from "class-validator"
+import { Request, Response } from "express"
+import { EntityManager } from "typeorm"
+
+import { ProductCategoryService } from "../../../../services"
+import { AdminProductCategoriesReqBase } from "../../../../types/product-category"
+import { FindParams } from "../../../../types/common"
+
+/**
+ * @oas [post] /product-categories
+ * operationId: "PostProductCategories"
+ * summary: "Create a Product Category"
+ * description: "Creates a Product Category."
+ * x-authenticated: true
+ * parameters:
+ *   - (query) expand {string} (Comma separated) Which fields should be expanded in each product category.
+ *   - (query) fields {string} (Comma separated) Which fields should be retrieved in each product category.
+ * requestBody:
+ *   content:
+ *     application/json:
+ *       schema:
+ *         $ref: "#/components/schemas/AdminPostProductCategoriesReq"
+ * x-codeSamples:
+ *   - lang: JavaScript
+ *     label: JS Client
+ *     source: |
+ *       import Medusa from "@medusajs/medusa-js"
+ *       const medusa = new Medusa({ baseUrl: MEDUSA_BACKEND_URL, maxRetries: 3 })
+ *       // must be previously logged in or use api token
+ *       medusa.admin.productCategories.create({
+ *         name: 'Jeans',
+ *         handle: 'jeans',
+ *       })
+ *       .then(({ productCategory }) => {
+ *         console.log(productCategory.id);
+ *       });
+ *   - lang: Shell
+ *     label: cURL
+ *     source: |
+ *       curl --location --request POST 'https://medusa-url.com/admin/product-categories' \
+ *       --header 'Authorization: Bearer {api_token}' \
+ *       --header 'Content-Type: application/json' \
+ *       --data-raw '{
+ *           "name": "Jeans",
+*            "handle": "jeans",
+ *       }'
+ * security:
+ *   - api_token: []
+ *   - cookie_auth: []
+ * tags:
+ *   - Product Category
+ * responses:
+ *  "200":
+ *    description: OK
+ *    content:
+ *      application/json:
+ *        schema:
+ *          type: object
+ *          properties:
+ *            productCategory:
+ *              $ref: "#/components/schemas/ProductCategory"
+ *  "400":
+ *    $ref: "#/components/responses/400_error"
+ *  "401":
+ *    $ref: "#/components/responses/unauthorized"
+ *  "404":
+ *    $ref: "#/components/responses/not_found_error"
+ *  "409":
+ *    $ref: "#/components/responses/invalid_state_error"
+ *  "422":
+ *    $ref: "#/components/responses/invalid_request_error"
+ *  "500":
+ *    $ref: "#/components/responses/500_error"
+ */
+export default async (req: Request, res: Response) => {
+  const { validatedBody } = req as {
+    validatedBody: AdminPostProductCategoriesReq
+  }
+
+  const productCategoryService: ProductCategoryService = req.scope.resolve(
+    "productCategoryService"
+  )
+
+  const manager: EntityManager = req.scope.resolve("manager")
+  const created = await manager.transaction(async (transactionManager) => {
+    return await productCategoryService
+      .withTransaction(transactionManager)
+      .create(validatedBody)
+  })
+
+  const productCategory = await productCategoryService.retrieve(
+    created.id,
+    req.retrieveConfig,
+  )
+
+  res.status(200).json({ product_category: productCategory })
+}
+
+/**
+ * @schema AdminPostProductCategoriesReq
+ * type: object
+ * required:
+ *   - name
+ *   - handle
+ * properties:
+ *   name:
+ *     type: string
+ *     description:  The name to identify the Product Category by.
+ *   handle:
+ *     type: string
+ *     description:  An optional handle to be used in slugs, if none is provided we will kebab-case the name.
+ *   is_internal:
+ *     type: boolean
+ *     description: A flag to make product category an internal category for admins
+ *   is_active:
+ *     type: boolean
+ *     description: A flag to make product category visible/hidden in the store front
+ *   parent_category_id:
+ *     type: string
+ *     description: The ID of the parent product category
+ */
+// eslint-disable-next-line max-len
+export class AdminPostProductCategoriesReq extends AdminProductCategoriesReqBase {
+  @IsString()
+  @IsNotEmpty()
+  name: string
+
+  @IsString()
+  @IsNotEmpty()
+  handle: string
+}
+
+export class AdminPostProductCategoriesParams extends FindParams {}

packages/medusa/src/api/routes/admin/product-categories/index.ts

@@ -1,6 +1,9 @@
 import { Router } from "express"
 
-import middlewares, { transformQuery } from "../../../middlewares"
+import middlewares, {
+  transformQuery,
+  transformBody,
+} from "../../../middlewares"
 import { isFeatureFlagEnabled } from "../../../middlewares/feature-flag-enabled"
 import deleteProductCategory from "./delete-product-category"
 
@@ -12,6 +15,16 @@ import listProductCategories, {
   AdminGetProductCategoriesParams,
 } from "./list-product-categories"
 
+import createProductCategory, {
+  AdminPostProductCategoriesReq,
+  AdminPostProductCategoriesParams,
+} from "./create-product-category"
+
+import updateProductCategory, {
+  AdminPostProductCategoriesCategoryReq,
+  AdminPostProductCategoriesCategoryParams,
+} from "./update-product-category"
+
 const route = Router()
 
 export default (app) => {
@@ -21,6 +34,17 @@ export default (app) => {
     route
   )
 
+  route.post(
+    "/",
+    transformQuery(AdminPostProductCategoriesParams, {
+      defaultFields: defaultProductCategoryFields,
+      defaultRelations: defaultAdminProductCategoryRelations,
+      isList: false,
+    }),
+    transformBody(AdminPostProductCategoriesReq),
+    middlewares.wrap(createProductCategory)
+  )
+
   route.get(
     "/",
     transformQuery(AdminGetProductCategoriesParams, {
@@ -40,6 +64,17 @@ export default (app) => {
     middlewares.wrap(getProductCategory)
   )
 
+  route.post(
+    "/:id",
+    transformQuery(AdminPostProductCategoriesCategoryParams, {
+      defaultFields: defaultProductCategoryFields,
+      defaultRelations: defaultAdminProductCategoryRelations,
+      isList: false,
+    }),
+    transformBody(AdminPostProductCategoriesCategoryReq),
+    middlewares.wrap(updateProductCategory)
+  )
+
   route.delete("/:id", middlewares.wrap(deleteProductCategory))
 
   return app
@@ -48,6 +83,7 @@ export default (app) => {
 export * from "./get-product-category"
 export * from "./delete-product-category"
 export * from "./list-product-categories"
+export * from "./create-product-category"
 
 export const defaultAdminProductCategoryRelations = [
   "parent_category",
@@ -60,4 +96,6 @@ export const defaultProductCategoryFields = [
   "handle",
   "is_active",
   "is_internal",
+  "created_at",
+  "updated_at",
 ]