test(core): adding test cases for unwrapping logic from the factory

adding test cases for unwrapping logic from the factory

GH-14

Author: vinaygupta-sourcefuse

src/__tests__/integration/mcp-tool.integration.ts

@@ -50,4 +50,148 @@ describe('McpServerFactory (integration)', () => {
       sinon.match.func,
     );
   });
+
+  describe('parameter unwrapping', () => {
+    it('unwraps double-wrapped object parameters', async () => {
+      const receivedParams: Record<string, unknown> = {};
+
+      // Simulate the unwrapping logic directly
+      const parameters = {
+        currency: {
+          currency: {
+            currencyCode: 'CAD',
+            currencyName: 'Canadian Dollar',
+            symbol: 'C$',
+            country: 'Canada',
+          },
+        },
+      };
+
+      // Apply the same unwrapping logic from the factory
+      for (const [key, value] of Object.entries(parameters)) {
+        if (value && typeof value === 'object') {
+          const valueObj = value as Record<string, unknown>;
+          if (key in valueObj && Object.keys(valueObj).length === 1) {
+            receivedParams[key] = valueObj[key];
+          } else {
+            receivedParams[key] = value;
+          }
+        } else {
+          receivedParams[key] = value;
+        }
+      }
+
+      expect(receivedParams).to.deepEqual({
+        currency: {
+          currencyCode: 'CAD',
+          currencyName: 'Canadian Dollar',
+          symbol: 'C$',
+          country: 'Canada',
+        },
+      });
+    });
+
+    it('keeps normal object parameters unchanged', () => {
+      const receivedParams: Record<string, unknown> = {};
+
+      const normalParams = {
+        currency: {
+          currencyCode: 'USD',
+          currencyName: 'US Dollar',
+          symbol: '$',
+          country: 'USA',
+        },
+        amount: 100,
+      };
+
+      for (const [key, value] of Object.entries(normalParams)) {
+        if (value && typeof value === 'object') {
+          const valueObj = value as Record<string, unknown>;
+          if (key in valueObj && Object.keys(valueObj).length === 1) {
+            receivedParams[key] = valueObj[key];
+          } else {
+            receivedParams[key] = value;
+          }
+        } else {
+          receivedParams[key] = value;
+        }
+      }
+
+      expect(receivedParams).to.deepEqual(normalParams);
+    });
+
+    it('preserves primitive parameters unchanged', () => {
+      const receivedParams: Record<string, unknown> = {};
+
+      const primitiveParams = {
+        name: 'John Doe',
+        age: 30,
+        active: true,
+      };
+
+      for (const [key, value] of Object.entries(primitiveParams)) {
+        if (value && typeof value === 'object') {
+          const valueObj = value as Record<string, unknown>;
+          if (key in valueObj && Object.keys(valueObj).length === 1) {
+            receivedParams[key] = valueObj[key];
+          } else {
+            receivedParams[key] = value;
+          }
+        } else {
+          receivedParams[key] = value;
+        }
+      }
+
+      expect(receivedParams).to.deepEqual(primitiveParams);
+    });
+
+    it('handles array parameters correctly', () => {
+      const receivedParams: Record<string, unknown> = {};
+
+      const arrayParams: Record<string, unknown> = {
+        items: ['item1', 'item2', 'item3'],
+        numbers: [1, 2, 3],
+      };
+
+      for (const [key, value] of Object.entries(arrayParams)) {
+        if (value && typeof value === 'object') {
+          const valueObj = value as Record<string, unknown>;
+          if (key in valueObj && Object.keys(valueObj).length === 1) {
+            receivedParams[key] = valueObj[key];
+          } else {
+            receivedParams[key] = value;
+          }
+        } else {
+          receivedParams[key] = value;
+        }
+      }
+
+      expect(receivedParams).to.deepEqual(arrayParams);
+    });
+
+    it('handles null and undefined parameters', () => {
+      const receivedParams: Record<string, unknown> = {};
+
+      const nullParams = {
+        name: 'John Doe',
+        age: null,
+        address: undefined,
+      };
+
+      for (const [key, value] of Object.entries(nullParams)) {
+        if (value && typeof value === 'object') {
+          const valueObj = value as Record<string, unknown>;
+          if (key in valueObj && Object.keys(valueObj).length === 1) {
+            receivedParams[key] = valueObj[key];
+          } else {
+            receivedParams[key] = value;
+          }
+        } else {
+          receivedParams[key] = value;
+        }
+      }
+
+      expect(receivedParams).to.deepEqual(nullParams);
+    });
+  });
 });