fix: add lock file, tests and changeset

Signed-off-by: Jayaraman N R <[email protected]>
reactioncommerce · brent-hoover · Oct 20, 2022 · Oct 19, 2022 · Oct 20, 2022 · Oct 20, 2022
commit 769c2185b8bb39bc0a3682b37ac8efd16aa77712
diff --git a/.changeset/modern-poems-smash.md b/.changeset/modern-poems-smash.md
@@ -0,0 +1,5 @@
+---
+"@reactioncommerce/api-plugin-shipments-flat-rate": patch
+---
+
+allow nested properties to be accessible for shipment restrictions.
diff --git a/packages/api-plugin-shipments-flat-rate/src/util/filterShippingMethods.test.js b/packages/api-plugin-shipments-flat-rate/src/util/filterShippingMethods.test.js
@@ -52,7 +52,17 @@ const mockHydratedOrderItems = {
   variantId: "tMkp5QwZog5ihYTfG",
   weight: 50,
   width: 10,
-  tags: [Array]
+  tags: [Array],
+  subtotal: {
+    amount: 12.99,
+    currencyCode: "USD"
+  },
+  parcel: {
+    height: 10,
+    weight: 50,
+    width: 10,
+    length: 10
+  }
 };
 
 const mockHydratedOrder = {
@@ -814,3 +824,82 @@ test("deny method - multiple attributes - item value is less than $100 AND item
 
   expect(allowedMethods).toEqual([]);
 });
+
+/*
+* Tests with nested item/attribute restrictions
+*/
+test("deny method - nested attribute - subtotal.amount is less than $100, item restricted", async () => {
+  const mockMethodRestrictions = [
+    {
+      _id: "allow001",
+      methodIds: [
+        "stviZaLdqRvTKW6J5"
+      ],
+      type: "allow",
+      destination: {
+        country: [
+          "US"
+        ]
+      }
+    },
+    {
+      _id: "deny001",
+      methodIds: [
+        "stviZaLdqRvTKW6J5"
+      ],
+      type: "deny",
+      attributes: [
+        {
+          property: "subtotal.amount",
+          value: 100,
+          propertyType: "int",
+          operator: "lt"
+        }
+      ]
+    }
+  ];
+
+  mockContext.collections.FlatRateFulfillmentRestrictions.toArray.mockReturnValue(Promise.resolve(mockMethodRestrictions));
+
+  const allowedMethods = await filterShippingMethods(mockContext, mockShippingMethod, mockHydratedOrder);
+
+  expect(allowedMethods).toEqual([]);
+});
+
+test("deny method - nested attributes - parcel.weight is greater than 50, no item restrictions", async () => {
+  const mockMethodRestrictions = [
+    {
+      _id: "allow001",
+      methodIds: [
+        "stviZaLdqRvTKW6J5"
+      ],
+      type: "allow",
+      destination: {
+        country: [
+          "US"
+        ]
+      }
+    },
+    {
+      _id: "deny001",
+      methodIds: [
+        "stviZaLdqRvTKW6J5"
+      ],
+      type: "deny",
+      attributes: [
+        {
+          property: "parcel.weight",
+          value: 50,
+          propertyType: "int",
+          operator: "gt"
+        }
+      ]
+    }
+  ];
+
+  mockContext.collections.FlatRateFulfillmentRestrictions.toArray.mockReturnValue(Promise.resolve(mockMethodRestrictions));
+
+  const allowedMethods = await filterShippingMethods(mockContext, mockShippingMethod, mockHydratedOrder);
+
+  expect(allowedMethods).toEqual(mockShippingMethod);
+});
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml