@app-connect/core 1.7.22 → 1.7.24

package/test/mcp/tools/getSessionInfo.test.js

@@ -0,0 +1,127 @@
+const getSessionInfo = require('../../../mcp/tools/getSessionInfo');
+const jwt = require('../../../lib/jwt');
+const { UserModel } = require('../../../models/userModel');
+const { RingCentral } = require('../../../lib/ringcentral');
+
+jest.mock('../../../lib/jwt');
+jest.mock('../../../models/userModel');
+jest.mock('../../../lib/ringcentral');
+
+describe('MCP Tool: getSessionInfo', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+    RingCentral.mockImplementation(() => ({
+      getExtensionInfo: jest.fn().mockResolvedValue({ name: 'Demo Extension' })
+    }));
+  });
+
+  describe('tool definition', () => {
+    test('should have correct tool definition', () => {
+      expect(getSessionInfo.definition).toBeDefined();
+      expect(getSessionInfo.definition.name).toBe('getSessionInfo');
+      expect(getSessionInfo.definition.description).toContain('session info');
+      expect(getSessionInfo.definition.inputSchema).toBeDefined();
+      expect(getSessionInfo.definition.inputSchema.properties).toEqual({});
+    });
+  });
+
+  describe('execute', () => {
+    test('should return unauthenticated session info when no CRM jwtToken exists', async () => {
+      const result = await getSessionInfo.execute({
+        openaiSessionId: 'session-123',
+        rcExtensionId: 'ext-456',
+        rcAccessToken: 'rc-token'
+      });
+
+      expect(result).toEqual({
+        success: true,
+        data: {
+          openaiSessionId: 'session-123',
+          dataToShow: {
+            isCrmAuthenticated: false,
+            ringcentral: {
+              extensionId: 'ext-456',
+              name: 'Demo Extension',
+            },
+            crm: {
+              userId: null,
+              platform: null,
+              hostname: null
+            }
+          }
+        }
+      });
+      expect(jwt.decodeJwt).not.toHaveBeenCalled();
+      expect(UserModel.findByPk).not.toHaveBeenCalled();
+    });
+
+    test('should return connected CRM session info when jwtToken resolves to a saved user', async () => {
+      jwt.decodeJwt.mockReturnValue({
+        id: 'crm-user-1',
+        platform: 'clio'
+      });
+      UserModel.findByPk.mockResolvedValue({
+        id: 'crm-user-1',
+        platform: 'clio',
+        hostname: 'app.clio.com',
+        accessToken: 'crm-access-token',
+      });
+
+      const result = await getSessionInfo.execute({
+        openaiSessionId: 'session-123',
+        rcExtensionId: 'ext-456',
+        rcAccessToken: 'rc-token',
+        jwtToken: 'jwt-token'
+      });
+
+      expect(jwt.decodeJwt).toHaveBeenCalledWith('jwt-token');
+      expect(UserModel.findByPk).toHaveBeenCalledWith('crm-user-1');
+      expect(result).toEqual({
+        success: true,
+        data: {
+          openaiSessionId: 'session-123',
+          dataToShow: {
+            isCrmAuthenticated: true,
+            ringcentral: {
+              extensionId: 'ext-456',
+              name: 'Demo Extension',
+            },
+            crm: {
+              userId: 'crm-user-1',
+              platform: 'clio',
+              hostname: 'app.clio.com'
+            }
+          }
+        }
+      });
+    });
+
+    test('should report not authenticated when jwtToken is invalid', async () => {
+      jwt.decodeJwt.mockReturnValue(null);
+
+      const result = await getSessionInfo.execute({
+        jwtToken: 'bad-token'
+      });
+
+      expect(result).toEqual({
+        success: true,
+        data: {
+          openaiSessionId: null,
+          dataToShow: {
+            isCrmAuthenticated: false,
+            ringcentral: {
+              extensionId: null,
+              name: null,
+            },
+            crm: {
+              userId: null,
+              platform: null,
+              hostname: null
+            }
+          }
+        }
+      });
+      expect(UserModel.findByPk).not.toHaveBeenCalled();
+    });
+  });
+});

package/test/mcp/tools/logout.test.js

@@ -1,16 +1,22 @@
 const logout = require('../../../mcp/tools/logout');
 const jwt = require('../../../lib/jwt');
 const { UserModel } = require('../../../models/userModel');
+const { LlmSessionModel } = require('../../../models/llmSessionModel');
+const { CacheModel } = require('../../../models/cacheModel');
 const connectorRegistry = require('../../../connector/registry');
 
 // Mock dependencies
 jest.mock('../../../lib/jwt');
 jest.mock('../../../models/userModel');
+jest.mock('../../../models/llmSessionModel');
+jest.mock('../../../models/cacheModel');
 jest.mock('../../../connector/registry');
 
 describe('MCP Tool: logout', () => {
   beforeEach(() => {
     jest.clearAllMocks();
+    LlmSessionModel.destroy.mockResolvedValue(1);
+    CacheModel.destroy.mockResolvedValue(1);
   });
 
   describe('tool definition', () => {
@@ -65,6 +71,7 @@ describe('MCP Tool: logout', () => {
         }
       });
       expect(jwt.decodeJwt).toHaveBeenCalledWith('mock-jwt-token');
+      expect(LlmSessionModel.destroy).toHaveBeenCalledWith({ where: { id: 'test-user-id' } });
       expect(UserModel.findByPk).toHaveBeenCalledWith('test-user-id');
       expect(connectorRegistry.getConnector).toHaveBeenCalledWith('testCRM');
       expect(mockConnector.unAuthorize).toHaveBeenCalledWith({
@@ -170,6 +177,57 @@ describe('MCP Tool: logout', () => {
       expect(consoleSpy).toHaveBeenCalled();
       consoleSpy.mockRestore();
     });
+
+    test('should clear both CRM user and RC extension session rows when rcExtensionId is provided', async () => {
+      const mockUser = {
+        id: 'test-user-id',
+        platform: 'testCRM'
+      };
+
+      jwt.decodeJwt.mockReturnValue({
+        id: 'test-user-id',
+        platform: 'testCRM'
+      });
+      UserModel.findByPk.mockResolvedValue(mockUser);
+      connectorRegistry.getConnector.mockReturnValue({
+        unAuthorize: jest.fn().mockResolvedValue({})
+      });
+
+      const result = await logout.execute({
+        jwtToken: 'mock-jwt-token',
+        rcExtensionId: 'rc-ext-123'
+      });
+
+      expect(result.success).toBe(true);
+      expect(LlmSessionModel.destroy).toHaveBeenNthCalledWith(1, { where: { id: 'test-user-id' } });
+      expect(LlmSessionModel.destroy).toHaveBeenNthCalledWith(2, { where: { id: 'rc-ext-123' } });
+    });
+
+    test('should clear the resolved rcExtension cache for the current OpenAI session on logout', async () => {
+      const mockUser = {
+        id: 'test-user-id',
+        platform: 'testCRM'
+      };
+
+      jwt.decodeJwt.mockReturnValue({
+        id: 'test-user-id',
+        platform: 'testCRM'
+      });
+      UserModel.findByPk.mockResolvedValue(mockUser);
+      connectorRegistry.getConnector.mockReturnValue({
+        unAuthorize: jest.fn().mockResolvedValue({})
+      });
+
+      const result = await logout.execute({
+        jwtToken: 'mock-jwt-token',
+        openaiSessionId: 'oa-session-123'
+      });
+
+      expect(result.success).toBe(true);
+      expect(CacheModel.destroy).toHaveBeenCalledWith({
+        where: { id: 'oa-session-123-rcExtensionId' }
+      });
+    });
   });
 });
 

package/test/routes/managedAuthRoutes.test.js

@@ -47,7 +47,6 @@ describe('Managed Auth Routes', () => {
       adminCore.validateRcUserToken.mockResolvedValue({
         rcAccountId: 'validated-account-id',
         rcExtensionId: 'validated-extension-id',
-        rcUserName: 'Validated User',
       });
       managedAuthCore.getManagedAuthState.mockResolvedValue({
         hasManagedAuth: true,
@@ -95,7 +94,6 @@ describe('Managed Auth Routes', () => {
       adminCore.validateRcUserToken.mockResolvedValue({
         rcAccountId: 'validated-account-id',
         rcExtensionId: 'validated-extension-id',
-        rcUserName: 'Validated User',
       });
       authCore.onApiKeyLogin.mockResolvedValue({
         userInfo: {
@@ -125,7 +123,6 @@ describe('Managed Auth Routes', () => {
         platform: 'testCRM',
         rcAccountId: 'validated-account-id',
         rcExtensionId: 'validated-extension-id',
-        rcUserName: 'Validated User',
       }));
     });
   });