chore(order): dml

.changeset/fair-ants-heal.md

@@ -0,0 +1,41 @@
+---
+"@medusajs/locking-postgres": minor
+"@medusajs/workflow-engine-inmemory": minor
+"@medusajs/workflow-engine-redis": minor
+"@medusajs/stock-location": minor
+"@medusajs/sales-channel": minor
+"@medusajs/link-modules": minor
+"@medusajs/notification": minor
+"@medusajs/fulfillment": minor
+"@medusajs/orchestration": minor
+"@medusajs/workflows-sdk": minor
+"@medusajs/test-utils": minor
+"@medusajs/inventory": minor
+"@medusajs/promotion": minor
+"@medusajs/modules-sdk": minor
+"@medusajs/currency": minor
+"@medusajs/customer": minor
+"@medusajs/core-flows": minor
+"@medusajs/api-key": minor
+"@medusajs/locking": minor
+"@medusajs/payment": minor
+"@medusajs/pricing": minor
+"@medusajs/product": minor
+"@medusajs/cli": minor
+"@medusajs/framework": minor
+"@medusajs/region": minor
+"integration-tests-http": minor
+"@medusajs/index": minor
+"@medusajs/order": minor
+"@medusajs/store": minor
+"@medusajs/auth": minor
+"@medusajs/cart": minor
+"@medusajs/file": minor
+"@medusajs/user": minor
+"@medusajs/tax": minor
+"@medusajs/types": minor
+"@medusajs/utils": minor
+"@medusajs/medusa": minor
+---
+
+chore: upgrade to mikro-orm 6

integration-tests/http/__tests__/cart/store/cart.spec.ts

@@ -386,7 +386,7 @@ medusaIntegrationTestRunner({
             rules: [
               {
                 attribute: "enabled_in_store",
-                value: '"true"',
+                value: "true",
                 operator: "eq",
               },
               {
@@ -1717,7 +1717,7 @@ medusaIntegrationTestRunner({
                 rules: [
                   {
                     attribute: "enabled_in_store",
-                    value: '"true"',
+                    value: "true",
                     operator: "eq",
                   },
                   {
@@ -2088,7 +2088,7 @@ medusaIntegrationTestRunner({
                 rules: [
                   {
                     attribute: "enabled_in_store",
-                    value: '"true"',
+                    value: "true",
                     operator: "eq",
                   },
                   {

integration-tests/http/__tests__/customer-group/admin/customer-group.spec.ts

@@ -69,7 +69,6 @@ medusaIntegrationTestRunner({
         await api
           .post("/admin/customer-groups", payload, adminHeaders)
           .catch((err) => {
-            console.log(err)
             // BREAKING: Duplicate error is now 400
             expect(err.response.status).toEqual(400)
             expect(err.response.data.type).toEqual("invalid_data")

integration-tests/http/__tests__/exchanges/exchanges.spec.ts

@@ -1,16 +1,16 @@
+import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 import {
   ContainerRegistrationKeys,
   Modules,
   RuleOperator,
 } from "@medusajs/utils"
-import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 import {
   adminHeaders,
   createAdminUser,
 } from "../../../helpers/create-admin-user"
 import { setupTaxStructure } from "../../../modules/__tests__/fixtures/tax"
 
-jest.setTimeout(30000)
+jest.setTimeout(300000)
 
 medusaIntegrationTestRunner({
   testSuite: ({ dbConnection, getContainer, api }) => {

integration-tests/http/__tests__/order/admin/rma-flows.spec.ts

@@ -1,13 +1,13 @@
-import { ClaimType, Modules, RuleOperator } from "@medusajs/utils"
 import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
+import { ClaimType, Modules, RuleOperator } from "@medusajs/utils"
 import {
   adminHeaders,
   createAdminUser,
 } from "../../../../helpers/create-admin-user"
 import { setupTaxStructure } from "../../../../modules/__tests__/fixtures"
 import { createOrderSeeder } from "../../fixtures/order"
 
-jest.setTimeout(30000)
+jest.setTimeout(300000)
 
 medusaIntegrationTestRunner({
   testSuite: ({ dbConnection, getContainer, api }) => {
@@ -268,6 +268,7 @@ medusaIntegrationTestRunner({
           {
             items: [
               {
+                title: "new ITEM",
                 variant_id: order.items[0].variant_id,
                 quantity: 1,
               },
@@ -480,6 +481,7 @@ medusaIntegrationTestRunner({
           {
             items: [
               {
+                title: "Test item 2",
                 variant_id: order.items[0].variant_id,
                 quantity: 1,
               },
@@ -551,7 +553,7 @@ medusaIntegrationTestRunner({
           })
         )
 
-        inboundItem = orderResult.items[0]
+        inboundItem = orderResult.items[1]
 
         await api.post(
           `/admin/claims/${claimWithInboundAndOutbound.id}/inbound/items`,

integration-tests/modules/__tests__/link-modules/index.ts

@@ -1,7 +1,7 @@
 import { getMigrationPlanner, initialize } from "@medusajs/link-modules"
 import { MedusaModule } from "@medusajs/modules-sdk"
-import { ModuleJoinerConfig } from "@medusajs/types"
 import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
+import { ModuleJoinerConfig } from "@medusajs/types"
 
 jest.setTimeout(5000000)
 
@@ -73,7 +73,11 @@ medusaIntegrationTestRunner({
       }) as any)
 
       const planner = getMigrationPlanner(dbConfig, linkDefinition)
-      await planner.executePlan(await planner.createPlan())
+      let plans = await planner.createPlan()
+
+      // TODO: review update plan of Order module tables
+      plans = plans.filter((p) => p.action !== "update")
+      await planner.executePlan(plans)
       links = await initialize(dbConfig, linkDefinition)
     })
 

integration-tests/modules/__tests__/order/draft-order.spec.ts

@@ -236,7 +236,7 @@ medusaIntegrationTestRunner({
               summary: expect.objectContaining({
                 // TODO: add summary fields
               }),
-              items: [
+              items: expect.arrayContaining([
                 expect.objectContaining({
                   title: "Test variant",
                   subtitle: "Test product",
@@ -335,7 +335,7 @@ medusaIntegrationTestRunner({
                     value: "1",
                   }),
                 }),
-              ],
+              ]),
               shipping_address: expect.objectContaining({
                 last_name: "Test",
                 address_1: "Test",

integration-tests/modules/__tests__/order/order.spec.ts

@@ -362,6 +362,7 @@ medusaIntegrationTestRunner({
             metadata: null,
             created_at: expect.any(String),
             updated_at: expect.any(String),
+            deleted_at: null,
           },
           billing_address: {
             id: expect.any(String),
@@ -379,6 +380,7 @@ medusaIntegrationTestRunner({
             metadata: null,
             created_at: expect.any(String),
             updated_at: expect.any(String),
+            deleted_at: null,
           },
           shipping_methods: [
             expect.objectContaining({

integration-tests/modules/__tests__/order/workflows/begin-order-claim.spec.ts

@@ -3,6 +3,7 @@ import {
   createShippingOptionsWorkflow,
   orderClaimRequestItemReturnWorkflow,
 } from "@medusajs/core-flows"
+import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 import {
   FulfillmentWorkflow,
   IOrderModuleService,
@@ -19,7 +20,6 @@ import {
   RuleOperator,
   remoteQueryObjectFromString,
 } from "@medusajs/utils"
-import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 
 jest.setTimeout(500000)
 
@@ -186,7 +186,7 @@ async function prepareDataFixtures({ container }) {
         {
           attribute: "is_return",
           operator: RuleOperator.EQ,
-          value: '"true"',
+          value: "true",
         },
       ],
     }

integration-tests/modules/__tests__/order/workflows/begin-order-exchange.spec.ts

@@ -4,6 +4,7 @@ import {
   orderExchangeAddNewItemWorkflow,
   orderExchangeRequestItemReturnWorkflow,
 } from "@medusajs/core-flows"
+import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 import {
   FulfillmentWorkflow,
   IRegionModuleService,
@@ -21,7 +22,6 @@ import {
   RuleOperator,
   remoteQueryObjectFromString,
 } from "@medusajs/utils"
-import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 
 jest.setTimeout(500000)
 
@@ -234,7 +234,7 @@ async function prepareDataFixtures({ container }) {
         {
           attribute: "is_return",
           operator: RuleOperator.EQ,
-          value: '"true"',
+          value: "true",
         },
       ],
     }

integration-tests/modules/__tests__/order/workflows/begin-order-return.spec.ts

@@ -2,6 +2,7 @@ import {
   beginReturnOrderWorkflow,
   createShippingOptionsWorkflow,
 } from "@medusajs/core-flows"
+import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 import {
   FulfillmentWorkflow,
   IOrderModuleService,
@@ -17,7 +18,6 @@ import {
   RuleOperator,
   remoteQueryObjectFromString,
 } from "@medusajs/utils"
-import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 
 jest.setTimeout(500000)
 
@@ -187,7 +187,7 @@ async function prepareDataFixtures({ container }) {
         {
           attribute: "is_return",
           operator: RuleOperator.EQ,
-          value: '"true"',
+          value: "true",
         },
       ],
     }

integration-tests/modules/__tests__/order/workflows/create-complete-return.spec.ts

@@ -3,6 +3,7 @@ import {
   createShippingOptionsWorkflow,
 } from "@medusajs/core-flows"
 import { RemoteLink } from "@medusajs/modules-sdk"
+import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 import {
   FulfillmentSetDTO,
   FulfillmentWorkflow,
@@ -21,7 +22,6 @@ import {
   RuleOperator,
   remoteQueryObjectFromString,
 } from "@medusajs/utils"
-import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 
 jest.setTimeout(500000)
 
@@ -190,7 +190,7 @@ async function prepareDataFixtures({ container }) {
         {
           attribute: "is_return",
           operator: RuleOperator.EQ,
-          value: '"true"',
+          value: "true",
         },
       ],
     }

integration-tests/modules/__tests__/order/workflows/create-fulfillment.spec.ts

@@ -3,6 +3,7 @@ import {
   createOrderFulfillmentWorkflow,
   createShippingOptionsWorkflow,
 } from "@medusajs/core-flows"
+import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 import {
   FulfillmentWorkflow,
   IOrderModuleService,
@@ -20,7 +21,6 @@ import {
   Modules,
   remoteQueryObjectFromString,
 } from "@medusajs/utils"
-import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 
 jest.setTimeout(500000)
 
@@ -305,12 +305,15 @@ async function createOrderFixture({ container, product, location }) {
 
   const inventoryModule = container.resolve(Modules.INVENTORY)
 
+  const itemWithInventory = order.items!.find(
+    (o) => o.variant_sku === variantSkuWithInventory
+  )!
   const reservation = await inventoryModule.createReservationItems([
     {
-      line_item_id: order.items![0].id,
+      line_item_id: itemWithInventory.id,
       inventory_item_id: inventoryItem.id,
       location_id: location.id,
-      quantity: order.items![0].quantity,
+      quantity: itemWithInventory.quantity,
     },
   ])
 
@@ -440,7 +443,6 @@ medusaIntegrationTestRunner({
             id: order.id,
           },
           fields: [
-            "*",
             "items.*",
             "shipping_methods.*",
             "total",
@@ -458,14 +460,15 @@ medusaIntegrationTestRunner({
         )!
 
         expect(orderFulfillAfterCancelled.fulfillments).toHaveLength(1)
-        expect(orderFulfillItemWithInventory.detail.fulfilled_quantity).toEqual(
-          0
-        )
+        expect(
+          orderFulfillItemWithInventory.detail.fulfilled_quantity.valueOf()
+        ).toEqual(0)
 
         const stockAvailabilityAfterCancelled =
           await inventoryModule.retrieveStockedQuantity(inventoryItem.id, [
             location.id,
           ])
+
         expect(stockAvailabilityAfterCancelled.valueOf()).toEqual(2)
       })
 

integration-tests/modules/__tests__/order/workflows/create-shipment.spec.ts

@@ -3,6 +3,7 @@ import {
   createOrderShipmentWorkflow,
   createShippingOptionsWorkflow,
 } from "@medusajs/core-flows"
+import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 import {
   FulfillmentWorkflow,
   IOrderModuleService,
@@ -20,7 +21,6 @@ import {
   RuleOperator,
   remoteQueryObjectFromString,
 } from "@medusajs/utils"
-import { medusaIntegrationTestRunner } from "@medusajs/test-utils"
 
 jest.setTimeout(500000)
 
@@ -177,7 +177,7 @@ async function prepareDataFixtures({ container }) {
         {
           attribute: "is_return",
           operator: RuleOperator.EQ,
-          value: '"true"',
+          value: "true",
         },
       ],
     }

integration-tests/modules/__tests__/order/workflows/return/items.spec.ts

@@ -145,7 +145,9 @@ medusaIntegrationTestRunner({
             throwOnError: false,
           })
 
-          expect(error.error.message).toEqual(`Order id not found: ${order.id}`)
+          expect(error.error.message).toEqual(
+            `Return id not found: ${returnOrder.id}`
+          )
         })
 
         it("should throw an error if order change does not exist", async () => {