@app-connect/core 1.7.36 → 1.7.38

package/test/mcp/tools/createMessageLog.test.js

@@ -1,595 +0,0 @@
-const createMessageLog = require('../../../mcp/tools/createMessageLog');
-const jwt = require('../../../lib/jwt');
-const connectorRegistry = require('../../../connector/registry');
-const logCore = require('../../../handlers/log');
-
-// Mock dependencies
-jest.mock('../../../lib/jwt');
-jest.mock('../../../connector/registry');
-jest.mock('../../../handlers/log');
-
-describe('MCP Tool: createMessageLog', () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  describe('tool definition', () => {
-    test('should have correct tool definition', () => {
-      expect(createMessageLog.definition).toBeDefined();
-      expect(createMessageLog.definition.name).toBe('createMessageLog');
-      expect(createMessageLog.definition.description).toContain('REQUIRES AUTHENTICATION');
-      expect(createMessageLog.definition.inputSchema).toBeDefined();
-    });
-
-    test('should require jwtToken and incomingData parameters', () => {
-      expect(createMessageLog.definition.inputSchema.required).toContain('jwtToken');
-      expect(createMessageLog.definition.inputSchema.required).toContain('incomingData');
-    });
-
-    test('should have detailed inputSchema for incomingData', () => {
-      const incomingDataSchema = createMessageLog.definition.inputSchema.properties.incomingData;
-      expect(incomingDataSchema.properties).toHaveProperty('conversation');
-      expect(incomingDataSchema.properties).toHaveProperty('contactId');
-      expect(incomingDataSchema.properties).toHaveProperty('contactName');
-      expect(incomingDataSchema.required).toContain('conversation');
-      expect(incomingDataSchema.required).toContain('contactId');
-      expect(incomingDataSchema.required).toContain('contactName');
-    });
-  });
-
-  describe('execute', () => {
-    test('should create message log successfully', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationId: 'conv-123',
-          conversationLogId: 'conv-log-123',
-          type: 'SMS',
-          messages: [
-            {
-              id: 'msg-123',
-              conversationId: 'conv-123',
-              phoneNumber: '+1234567890',
-              direction: 'Inbound',
-              creationTime: 1704110400000,
-              subject: 'Hello',
-              from: { phoneNumber: '+1234567890', name: 'John Doe' },
-              to: [{ phoneNumber: '+0987654321' }]
-            }
-          ],
-          correspondents: [
-            { phoneNumber: '+1234567890' }
-          ]
-        },
-        contactId: 'contact-123',
-        contactName: 'John Doe',
-        contactType: 'Contact'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {
-        createMessageLog: jest.fn()
-      };
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      logCore.createMessageLog.mockResolvedValue({
-        successful: true,
-        logIds: ['crm-msg-log-123'],
-        returnMessage: { message: 'Message logs created successfully' }
-      });
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result).toEqual({
-        success: true,
-        data: {
-          logIds: ['crm-msg-log-123'],
-          message: 'Message logs created successfully'
-        }
-      });
-      expect(jwt.decodeJwt).toHaveBeenCalledWith('mock-jwt-token');
-      expect(logCore.createMessageLog).toHaveBeenCalledWith({
-        platform: 'testCRM',
-        userId: 'user-123',
-        incomingData: expect.objectContaining({
-          contactId: 'contact-123',
-          contactName: 'John Doe',
-          logInfo: mockIncomingData.conversation
-        })
-      });
-    });
-
-    test('should create message log with multiple messages', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationId: 'conv-456',
-          conversationLogId: 'conv-log-456',
-          type: 'SMS',
-          messages: [
-            {
-              id: 'msg-456-1',
-              conversationId: 'conv-456',
-              phoneNumber: '+1234567890',
-              direction: 'Outbound',
-              from: { phoneNumber: '+0987654321' },
-              to: [{ phoneNumber: '+1234567890' }]
-            },
-            {
-              id: 'msg-456-2',
-              conversationId: 'conv-456',
-              phoneNumber: '+1234567890',
-              direction: 'Inbound',
-              from: { phoneNumber: '+1234567890' },
-              to: [{ phoneNumber: '+0987654321' }]
-            }
-          ],
-          correspondents: [
-            { phoneNumber: '+1234567890' }
-          ]
-        },
-        contactId: 'contact-456',
-        contactName: 'Jane Smith'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {
-        createMessageLog: jest.fn()
-      };
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      logCore.createMessageLog.mockResolvedValue({
-        successful: true,
-        logIds: ['crm-msg-log-456-1', 'crm-msg-log-456-2']
-      });
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(true);
-      expect(result.data.logIds).toHaveLength(2);
-    });
-
-    test('should create message log with attachments', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationId: 'conv-789',
-          conversationLogId: 'conv-log-789',
-          type: 'MMS',
-          messages: [
-            {
-              id: 'msg-789',
-              conversationId: 'conv-789',
-              phoneNumber: '+1234567890',
-              direction: 'Inbound',
-              from: { phoneNumber: '+1234567890' },
-              to: [{ phoneNumber: '+0987654321' }],
-              attachments: [
-                {
-                  type: 'image',
-                  link: 'https://example.com/image.jpg',
-                  contentType: 'image/jpeg'
-                }
-              ]
-            }
-          ],
-          correspondents: [
-            { phoneNumber: '+1234567890' }
-          ]
-        },
-        contactId: 'contact-789',
-        contactName: 'Bob Johnson'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {
-        createMessageLog: jest.fn()
-      };
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      logCore.createMessageLog.mockResolvedValue({
-        successful: true,
-        logIds: ['crm-msg-log-789']
-      });
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(true);
-    });
-
-    test('should create fax message log', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationId: 'conv-fax',
-          conversationLogId: 'conv-log-fax',
-          type: 'Fax',
-          messages: [
-            {
-              id: 'msg-fax',
-              conversationId: 'conv-fax',
-              phoneNumber: '+1234567890',
-              direction: 'Inbound',
-              messageStatus: 'Received',
-              faxPageCount: 5,
-              from: { phoneNumber: '+1234567890' },
-              to: [{ phoneNumber: '+0987654321' }]
-            }
-          ],
-          correspondents: [
-            { phoneNumber: '+1234567890' }
-          ]
-        },
-        contactId: 'contact-fax',
-        contactName: 'Fax Sender'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {
-        createMessageLog: jest.fn()
-      };
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      logCore.createMessageLog.mockResolvedValue({
-        successful: true,
-        logIds: ['crm-msg-log-fax']
-      });
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(true);
-    });
-
-    test('should create message log with additional submission', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationId: 'conv-submit',
-          conversationLogId: 'conv-log-submit',
-          messages: [
-            {
-              id: 'msg-submit',
-              conversationId: 'conv-submit',
-              phoneNumber: '+1234567890',
-              direction: 'Inbound',
-              from: { phoneNumber: '+1234567890' },
-              to: [{ phoneNumber: '+0987654321' }]
-            }
-          ],
-          correspondents: [
-            { phoneNumber: '+1234567890' }
-          ]
-        },
-        contactId: 'contact-submit',
-        contactName: 'Test User',
-        additionalSubmission: {
-          isAssignedToUser: true
-        }
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {
-        createMessageLog: jest.fn()
-      };
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      logCore.createMessageLog.mockResolvedValue({
-        successful: true,
-        logIds: ['crm-msg-log-submit']
-      });
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(true);
-    });
-
-    test('should handle logInfo alias for conversation', async () => {
-      // Arrange
-      const mockIncomingData = {
-        logInfo: {
-          conversationId: 'conv-alias',
-          conversationLogId: 'conv-log-alias',
-          messages: [
-            {
-              id: 'msg-alias',
-              conversationId: 'conv-alias',
-              phoneNumber: '+1234567890',
-              direction: 'Inbound',
-              from: { phoneNumber: '+1234567890' },
-              to: [{ phoneNumber: '+0987654321' }]
-            }
-          ],
-          correspondents: [
-            { phoneNumber: '+1234567890' }
-          ]
-        },
-        contactId: 'contact-alias',
-        contactName: 'Alias Test'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {
-        createMessageLog: jest.fn()
-      };
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      logCore.createMessageLog.mockResolvedValue({
-        successful: true,
-        logIds: ['crm-msg-log-alias']
-      });
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(true);
-    });
-
-    test('should return error when jwtToken is missing', async () => {
-      // Act
-      const result = await createMessageLog.execute({
-        incomingData: { conversation: {}, contactId: 'test', contactName: 'Test' }
-      });
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('authorize CRM platform');
-    });
-
-    test('should return error when incomingData is missing', async () => {
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token'
-      });
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('Incoming data must be provided');
-    });
-
-    test('should return error when JWT is invalid', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationLogId: 'conv-123',
-          messages: [],
-          correspondents: []
-        },
-        contactId: 'contact-123',
-        contactName: 'Test'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        platform: 'testCRM'
-        // id is missing
-      });
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'invalid-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('Invalid JWT token');
-    });
-
-    test('should return error when decodeJwt returns null', async () => {
-      jwt.decodeJwt.mockReturnValue(null);
-
-      const result = await createMessageLog.execute({
-        jwtToken: 'invalid-token',
-        incomingData: {
-          sessionId: 'session-123',
-          messageInfo: { from: { phoneNumber: '+1234567890' }, to: [{ phoneNumber: '+1098765432' }] }
-        }
-      });
-
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('Invalid JWT token');
-    });
-
-    test('should return error when platform connector not found', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationLogId: 'conv-123',
-          messages: [],
-          correspondents: []
-        },
-        contactId: 'contact-123',
-        contactName: 'Test'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'unknownCRM'
-      });
-
-      connectorRegistry.getConnector.mockReturnValue(null);
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('Platform connector not found');
-    });
-
-    test('should return error when createMessageLog is not implemented', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationLogId: 'conv-123',
-          messages: [],
-          correspondents: []
-        },
-        contactId: 'contact-123',
-        contactName: 'Test'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {}; // No createMessageLog method
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('not implemented');
-    });
-
-    test('should return error when creation fails', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationId: 'conv-fail',
-          conversationLogId: 'conv-log-fail',
-          messages: [
-            {
-              id: 'msg-fail',
-              conversationId: 'conv-fail',
-              phoneNumber: '+1234567890',
-              direction: 'Inbound',
-              from: { phoneNumber: '+1234567890' },
-              to: [{ phoneNumber: '+0987654321' }]
-            }
-          ],
-          correspondents: [
-            { phoneNumber: '+1234567890' }
-          ]
-        },
-        contactId: 'contact-fail',
-        contactName: 'Fail Test'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {
-        createMessageLog: jest.fn()
-      };
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      logCore.createMessageLog.mockResolvedValue({
-        successful: false,
-        returnMessage: { message: 'Failed to create message logs in CRM' }
-      });
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toBe('Failed to create message logs in CRM');
-    });
-
-    test('should handle unexpected errors gracefully', async () => {
-      // Arrange
-      const mockIncomingData = {
-        conversation: {
-          conversationLogId: 'conv-error',
-          messages: [],
-          correspondents: []
-        },
-        contactId: 'contact-error',
-        contactName: 'Error Test'
-      };
-
-      jwt.decodeJwt.mockReturnValue({
-        id: 'user-123',
-        platform: 'testCRM'
-      });
-
-      const mockConnector = {
-        createMessageLog: jest.fn()
-      };
-      connectorRegistry.getConnector.mockReturnValue(mockConnector);
-
-      logCore.createMessageLog.mockRejectedValue(
-        new Error('Network timeout')
-      );
-
-      // Act
-      const result = await createMessageLog.execute({
-        jwtToken: 'mock-jwt-token',
-        incomingData: mockIncomingData
-      });
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toBe('Network timeout');
-      expect(result.errorDetails).toBeDefined();
-    });
-  });
-});
-

package/test/mcp/tools/doAuth.test.js

@@ -1,113 +0,0 @@
-const doAuth = require('../../../mcp/tools/doAuth');
-const { createAuthSession } = require('../../../lib/authSession');
-
-jest.mock('../../../lib/authSession');
-
-describe('MCP Tool: doAuth', () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  describe('tool definition', () => {
-    test('should have correct tool definition', () => {
-      expect(doAuth.definition).toBeDefined();
-      expect(doAuth.definition.name).toBe('doAuth');
-      expect(doAuth.definition.description).toContain('OAuth session');
-      expect(doAuth.definition.inputSchema).toBeDefined();
-    });
-
-    test('should require connectorName and have optional hostname', () => {
-      expect(doAuth.definition.inputSchema.required).toContain('connectorName');
-      expect(doAuth.definition.inputSchema.properties).toHaveProperty('connectorName');
-      expect(doAuth.definition.inputSchema.properties).toHaveProperty('hostname');
-    });
-  });
-
-  describe('execute', () => {
-    test('should create auth session successfully', async () => {
-      // Arrange
-      createAuthSession.mockResolvedValue(undefined);
-
-      // Act
-      const result = await doAuth.execute({
-        sessionId: 'session-abc',
-        connectorName: 'pipedrive',
-        hostname: 'mycompany.pipedrive.com'
-      });
-
-      // Assert
-      expect(result).toEqual({ success: true });
-      expect(createAuthSession).toHaveBeenCalledWith('session-abc', {
-        platform: 'pipedrive',
-        hostname: 'mycompany.pipedrive.com'
-      });
-    });
-
-    test('should create auth session with empty hostname when not provided', async () => {
-      // Arrange
-      createAuthSession.mockResolvedValue(undefined);
-
-      // Act
-      const result = await doAuth.execute({
-        sessionId: 'session-xyz',
-        connectorName: 'clio'
-      });
-
-      // Assert
-      expect(result).toEqual({ success: true });
-      expect(createAuthSession).toHaveBeenCalledWith('session-xyz', {
-        platform: 'clio',
-        hostname: ''
-      });
-    });
-
-    test('should return error when sessionId is missing', async () => {
-      // Act
-      const result = await doAuth.execute({ connectorName: 'pipedrive' });
-
-      // Assert
-      expect(result).toEqual({
-        success: false,
-        error: 'Missing required fields: sessionId, connectorName'
-      });
-      expect(createAuthSession).not.toHaveBeenCalled();
-    });
-
-    test('should return error when connectorName is missing', async () => {
-      // Act
-      const result = await doAuth.execute({ sessionId: 'session-abc' });
-
-      // Assert
-      expect(result).toEqual({
-        success: false,
-        error: 'Missing required fields: sessionId, connectorName'
-      });
-      expect(createAuthSession).not.toHaveBeenCalled();
-    });
-
-    test('should return error when both sessionId and connectorName are missing', async () => {
-      // Act
-      const result = await doAuth.execute({});
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toContain('Missing required fields');
-    });
-
-    test('should handle unexpected errors gracefully', async () => {
-      // Arrange
-      createAuthSession.mockRejectedValue(new Error('DB write failed'));
-
-      // Act
-      const result = await doAuth.execute({
-        sessionId: 'session-abc',
-        connectorName: 'pipedrive'
-      });
-
-      // Assert
-      expect(result.success).toBe(false);
-      expect(result.error).toBe('DB write failed');
-      expect(result.errorDetails).toBeDefined();
-    });
-  });
-});