test: add routing and validation tests for existing-endpoint flow

Author: aidandaly24

src/cli/commands/add/__tests__/actions.test.ts

@@ -1,6 +1,15 @@
 import { buildGatewayTargetConfig } from '../actions.js';
 import type { ValidatedAddGatewayTargetOptions } from '../actions.js';
-import { describe, expect, it } from 'vitest';
+import { afterEach, describe, expect, it, vi } from 'vitest';
+
+const mockCreateToolFromWizard = vi.fn().mockResolvedValue({ toolName: 'test', projectPath: '/tmp' });
+const mockCreateExternalGatewayTarget = vi.fn().mockResolvedValue({ toolName: 'test', projectPath: '' });
+
+vi.mock('../../../operations/mcp/create-mcp', () => ({
+  createToolFromWizard: (...args: unknown[]) => mockCreateToolFromWizard(...args),
+  createExternalGatewayTarget: (...args: unknown[]) => mockCreateExternalGatewayTarget(...args),
+  createGatewayFromWizard: vi.fn(),
+}));
 
 describe('buildGatewayTargetConfig', () => {
   it('maps name, gateway, language correctly', () => {
@@ -66,3 +75,26 @@ describe('buildGatewayTargetConfig', () => {
     expect(config.outboundAuth).toBeUndefined();
   });
 });
+
+// Dynamic import to pick up mocks
+const { handleAddGatewayTarget } = await import('../actions.js');
+
+describe('handleAddGatewayTarget', () => {
+  afterEach(() => vi.clearAllMocks());
+
+  it('routes existing-endpoint to createExternalGatewayTarget', async () => {
+    const options: ValidatedAddGatewayTargetOptions = {
+      name: 'test-tool',
+      language: 'Other',
+      host: 'Lambda',
+      source: 'existing-endpoint',
+      endpoint: 'https://example.com/mcp',
+      gateway: 'my-gw',
+    };
+
+    await handleAddGatewayTarget(options);
+
+    expect(mockCreateExternalGatewayTarget).toHaveBeenCalledOnce();
+    expect(mockCreateToolFromWizard).not.toHaveBeenCalled();
+  });
+});

src/cli/commands/add/__tests__/validate.test.ts

@@ -422,6 +422,18 @@ describe('validate', () => {
       expect(result.valid).toBe(false);
       expect(result.error).toContain('--credential-name is required');
     });
+
+    it('rejects --host with existing-endpoint', async () => {
+      const options: AddGatewayTargetOptions = {
+        name: 'test-tool',
+        source: 'existing-endpoint',
+        endpoint: 'https://example.com/mcp',
+        host: 'Lambda',
+      };
+      const result = await validateAddGatewayTargetOptions(options);
+      expect(result.valid).toBe(false);
+      expect(result.error).toBe('--host is not applicable for existing endpoint targets');
+    });
   });
 
   describe('validateAddMemoryOptions', () => {