From 308b99cc5d40a984b8d1d65ac065ebcda7091236 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Frane=20Poli=C4=87?=
 <16856471+fPolic@users.noreply.github.com>
Date: Tue, 11 Oct 2022 12:28:24 +0200
Subject: [PATCH] fix(medusa-react): OrderEdit add item hook renaming (#2392)

---
 .../src/hooks/admin/order-edits/mutations.ts  | 46 ++++++++++++++-----
 .../hooks/admin/order-edits/mutations.test.ts |  9 ++--
 2 files changed, 38 insertions(+), 17 deletions(-)

diff --git a/packages/medusa-react/src/hooks/admin/order-edits/mutations.ts b/packages/medusa-react/src/hooks/admin/order-edits/mutations.ts
index 4ae7ebe2c2..418fb13f17 100644
--- a/packages/medusa-react/src/hooks/admin/order-edits/mutations.ts
+++ b/packages/medusa-react/src/hooks/admin/order-edits/mutations.ts
@@ -13,6 +13,7 @@ import {
 
 import { buildOptions } from "../../utils/buildOptions"
 import { useMedusa } from "../../../contexts"
+import { adminOrderKeys } from "../orders"
 import { adminOrderEditsKeys } from "."
 
 export const useAdminCreateOrderEdit = (
@@ -27,7 +28,11 @@ export const useAdminCreateOrderEdit = (
   return useMutation(
     (payload: AdminPostOrderEditsReq) =>
       client.admin.orderEdits.create(payload),
-    buildOptions(queryClient, adminOrderEditsKeys.lists(), options)
+    buildOptions(
+      queryClient,
+      [adminOrderEditsKeys.lists(), adminOrderKeys.details()],
+      options
+    )
   )
 }
 
@@ -42,7 +47,11 @@ export const useAdminDeleteOrderEdit = (
     () => client.admin.orderEdits.delete(id),
     buildOptions(
       queryClient,
-      [adminOrderEditsKeys.detail(id), adminOrderEditsKeys.lists()],
+      [
+        adminOrderEditsKeys.detail(id),
+        adminOrderEditsKeys.lists(),
+        adminOrderKeys.details(),
+      ],
       options
     )
   )
@@ -96,16 +105,13 @@ export const useAdminOrderEditUpdateLineItem = (
 export const useAdminOrderEditDeleteLineItem = (
   orderEditId: string,
   itemId: string,
-  options?: UseMutationOptions<
-    Response<AdminOrderEditsRes>,
-    Error
-  >
+  options?: UseMutationOptions<Response<AdminOrderEditsRes>, Error>
 ) => {
   const { client } = useMedusa()
   const queryClient = useQueryClient()
 
   return useMutation(
-    (() => client.admin.orderEdits.removeLineItem(orderEditId, itemId)),
+    () => client.admin.orderEdits.removeLineItem(orderEditId, itemId),
     buildOptions(
       queryClient,
       [adminOrderEditsKeys.detail(orderEditId), adminOrderEditsKeys.lists()],
@@ -130,13 +136,17 @@ export const useAdminUpdateOrderEdit = (
       client.admin.orderEdits.update(id, payload),
     buildOptions(
       queryClient,
-      [adminOrderEditsKeys.lists(), adminOrderEditsKeys.detail(id)],
+      [
+        adminOrderEditsKeys.lists(),
+        adminOrderEditsKeys.detail(id),
+        adminOrderKeys.details(),
+      ],
       options
     )
   )
 }
 
-export const useAdminOrderEditLineItem = (
+export const useAdminOrderEditAddLineItem = (
   id: string,
   options?: UseMutationOptions<
     Response<AdminOrderEditsRes>,
@@ -168,7 +178,11 @@ export const useAdminRequestOrderEditConfirmation = (
     () => client.admin.orderEdits.requestConfirmation(id),
     buildOptions(
       queryClient,
-      [adminOrderEditsKeys.lists(), adminOrderEditsKeys.detail(id)],
+      [
+        adminOrderEditsKeys.lists(),
+        adminOrderEditsKeys.detail(id),
+        adminOrderKeys.details(),
+      ],
       options
     )
   )
@@ -185,7 +199,11 @@ export const useAdminCancelOrderEdit = (
     () => client.admin.orderEdits.cancel(id),
     buildOptions(
       queryClient,
-      [adminOrderEditsKeys.lists(), adminOrderEditsKeys.detail(id)],
+      [
+        adminOrderEditsKeys.lists(),
+        adminOrderEditsKeys.detail(id),
+        adminOrderKeys.details(),
+      ],
       options
     )
   )
@@ -202,7 +220,11 @@ export const useAdminConfirmOrderEdit = (
     () => client.admin.orderEdits.confirm(id),
     buildOptions(
       queryClient,
-      [adminOrderEditsKeys.lists(), adminOrderEditsKeys.detail(id)],
+      [
+        adminOrderEditsKeys.lists(),
+        adminOrderEditsKeys.detail(id),
+        adminOrderKeys.details(),
+      ],
       options
     )
   )
diff --git a/packages/medusa-react/test/hooks/admin/order-edits/mutations.test.ts b/packages/medusa-react/test/hooks/admin/order-edits/mutations.test.ts
index 939efbb3aa..6d14effbb1 100644
--- a/packages/medusa-react/test/hooks/admin/order-edits/mutations.test.ts
+++ b/packages/medusa-react/test/hooks/admin/order-edits/mutations.test.ts
@@ -7,7 +7,7 @@ import {
   useAdminDeleteOrderEditItemChange,
   useAdminOrderEditUpdateLineItem,
   useAdminRequestOrderEditConfirmation,
-  useAdminOrderEditLineItem,
+  useAdminOrderEditAddLineItem,
   useAdminCancelOrderEdit,
   useAdminUpdateOrderEdit,
   useAdminOrderEditDeleteLineItem,
@@ -167,10 +167,10 @@ describe("useAdminRequestOrderEditConfirmation hook", () => {
   })
 })
 
-describe("useAdminOrderEditLineItem hook", () => {
+describe("useAdminOrderEditAddLineItem hook", () => {
   test("Created an order edit line item", async () => {
     const { result, waitFor } = renderHook(
-      () => useAdminOrderEditLineItem(fixtures.get("order_edit").id),
+      () => useAdminOrderEditAddLineItem(fixtures.get("order_edit").id),
       {
         wrapper: createWrapper(),
       }
@@ -249,7 +249,6 @@ describe("useAdminConfirmOrderEdit hook", () => {
   })
 })
 
-
 describe("useAdminOrderEditDeleteLineItem hook", () => {
   test("Remove line item of an order edit and create an item change", async () => {
     const id = "oe_1"
@@ -270,7 +269,7 @@ describe("useAdminOrderEditDeleteLineItem hook", () => {
         ...fixtures.get("order_edit"),
         changes: expect.arrayContaining([
           expect.objectContaining({
-            type: 'item_remove'
+            type: "item_remove",
           }),
         ]),
       })