@app-connect/core 1.7.37 → 1.7.38

package/test/lib/generalErrorMessage.test.js

@@ -1,82 +0,0 @@
-const {
-  authorizationErrorMessage,
-  rateLimitErrorMessage,
-} = require('../../lib/generalErrorMessage');
-
-describe('generalErrorMessage', () => {
-  describe('rateLimitErrorMessage', () => {
-    test('builds the standard rate limit warning message', () => {
-      const result = rateLimitErrorMessage({ platform: 'Clio' });
-
-      expect(result).toEqual({
-        message: 'Rate limit exceeded',
-        messageType: 'warning',
-        details: [
-          {
-            title: 'Details',
-            items: [
-              {
-                id: '1',
-                type: 'text',
-                text: 'You have exceeded the maximum number of requests allowed by Clio. Please try again in the next minute. If the problem persists please contact support.',
-              },
-            ],
-          },
-        ],
-        ttl: 5000,
-      });
-    });
-
-    test('keeps current interpolation behavior for empty platform input', () => {
-      const result = rateLimitErrorMessage({ platform: '' });
-
-      expect(result.details[0].items[0].text).toContain('allowed by .');
-      expect(result.ttl).toBe(5000);
-    });
-
-    test('keeps current interpolation behavior for missing platform input', () => {
-      const result = rateLimitErrorMessage({});
-
-      expect(result.details[0].items[0].text).toContain('allowed by undefined.');
-    });
-  });
-
-  describe('authorizationErrorMessage', () => {
-    test('builds the standard authorization warning message', () => {
-      const result = authorizationErrorMessage({ platform: 'Bullhorn' });
-
-      expect(result).toEqual({
-        message: 'Authorization error',
-        messageType: 'warning',
-        details: [
-          {
-            title: 'Details',
-            items: [
-              {
-                id: '1',
-                type: 'text',
-                text: "It seems like there's something wrong with your authorization of Bullhorn. Please Logout and then Connect your Bullhorn account within this extension.",
-              },
-            ],
-          },
-        ],
-        ttl: 5000,
-      });
-    });
-
-    test('keeps current interpolation behavior for empty platform input', () => {
-      const result = authorizationErrorMessage({ platform: '' });
-
-      expect(result.details[0].items[0].text).toContain('authorization of .');
-      expect(result.details[0].items[0].text).toContain('Connect your  account');
-    });
-
-    test('keeps current interpolation behavior for missing platform input', () => {
-      const result = authorizationErrorMessage({});
-
-      expect(result.details[0].items[0].text).toContain('authorization of undefined.');
-      expect(result.details[0].items[0].text).toContain('Connect your undefined account');
-    });
-  });
-});
-

package/test/lib/jwt.test.js

@@ -1,176 +0,0 @@
-const jwt = require('../../lib/jwt');
-
-describe('JWT Utility', () => {
-  beforeEach(() => {
-    // Reset environment
-    process.env.APP_SERVER_SECRET_KEY = 'test-secret-key';
-  });
-
-  describe('generateJwt', () => {
-    test('should generate JWT token from payload', () => {
-      // Arrange
-      const payload = {
-        id: 'test-user-id',
-        platform: 'testCRM'
-      };
-
-      // Act
-      const token = jwt.generateJwt(payload);
-
-      // Assert
-      expect(token).toBeDefined();
-      expect(typeof token).toBe('string');
-      expect(token.split('.')).toHaveLength(3); // JWT has 3 parts
-    });
-
-    test('should generate different tokens for different payloads', () => {
-      // Arrange
-      const payload1 = { id: 'user1', platform: 'testCRM' };
-      const payload2 = { id: 'user2', platform: 'testCRM' };
-
-      // Act
-      const token1 = jwt.generateJwt(payload1);
-      const token2 = jwt.generateJwt(payload2);
-
-      // Assert
-      expect(token1).not.toBe(token2);
-    });
-
-    test('should generate token with about 2 weeks lifetime', () => {
-      // Arrange
-      const payload = { id: 'user-ttl', platform: 'testCRM' };
-
-      // Act
-      const token = jwt.generateJwt(payload);
-      const decoded = jwt.decodeJwt(token);
-      const lifetimeSeconds = decoded.exp - decoded.iat;
-
-      // Assert
-      // Keep a tiny tolerance to avoid timing flakiness.
-      expect(lifetimeSeconds).toBeGreaterThanOrEqual((14 * 24 * 60 * 60) - 2);
-      expect(lifetimeSeconds).toBeLessThanOrEqual((14 * 24 * 60 * 60) + 2);
-    });
-  });
-
-  describe('decodeJwt', () => {
-    test('should decode valid JWT token', () => {
-      // Arrange
-      const payload = {
-        id: 'test-user-id',
-        platform: 'testCRM'
-      };
-      const token = jwt.generateJwt(payload);
-
-      // Act
-      const decoded = jwt.decodeJwt(token);
-
-      // Assert
-      expect(decoded).toMatchObject(payload);
-      expect(decoded).toHaveProperty('exp');
-      expect(decoded).toHaveProperty('iat');
-    });
-
-    test('should return null for invalid token', () => {
-      // Arrange
-      const invalidToken = 'invalid.jwt.token';
-
-      // Act
-      const decoded = jwt.decodeJwt(invalidToken);
-
-      // Assert
-      expect(decoded).toBeNull();
-    });
-
-    test('should return null for malformed token', () => {
-      // Arrange
-      const malformedToken = 'not-a-jwt-token';
-
-      // Act
-      const decoded = jwt.decodeJwt(malformedToken);
-
-      // Assert
-      expect(decoded).toBeNull();
-    });
-
-    test('should return null for token with wrong secret', () => {
-      // Arrange
-      const payload = { id: 'test-user-id', platform: 'testCRM' };
-      const token = jwt.generateJwt(payload);
-      
-      // Change secret temporarily
-      const originalSecret = process.env.APP_SERVER_SECRET_KEY;
-      process.env.APP_SERVER_SECRET_KEY = 'different-secret';
-
-      // Act
-      const decoded = jwt.decodeJwt(token);
-
-      // Restore secret
-      process.env.APP_SERVER_SECRET_KEY = originalSecret;
-
-      // Assert
-      expect(decoded).toBeNull();
-    });
-  });
-
-  describe('generateJwt and decodeJwt round trip', () => {
-    test('should successfully generate and decode complex payload', () => {
-      // Arrange
-      const complexPayload = {
-        id: 'test-user-id',
-        platform: 'testCRM',
-        timestamp: Date.now(),
-        metadata: {
-          timezone: 'America/Los_Angeles',
-          preferences: {
-            autoLog: true,
-            callPop: false
-          }
-        }
-      };
-
-      // Act
-      const token = jwt.generateJwt(complexPayload);
-      const decoded = jwt.decodeJwt(token);
-
-      // Assert
-      expect(decoded).toMatchObject(complexPayload);
-      expect(decoded).toHaveProperty('exp');
-      expect(decoded).toHaveProperty('iat');
-    });
-
-    test('should handle empty payload', () => {
-      // Arrange
-      const emptyPayload = {};
-
-      // Act
-      const token = jwt.generateJwt(emptyPayload);
-      const decoded = jwt.decodeJwt(token);
-
-      // Assert
-      expect(decoded).toMatchObject(emptyPayload);
-      expect(decoded).toHaveProperty('exp');
-      expect(decoded).toHaveProperty('iat');
-    });
-  });
-
-  describe('error handling', () => {
-    test('should handle missing secret key', () => {
-      // Arrange
-      const payload = { id: 'test-user-id' };
-      delete process.env.APP_SERVER_SECRET_KEY;
-
-      // Act & Assert
-      expect(() => jwt.generateJwt(payload)).toThrow();
-    });
-
-    test('should handle null payload', () => {
-      // Act & Assert
-      expect(() => jwt.generateJwt(null)).toThrow();
-    });
-
-    test('should handle undefined payload', () => {
-      // Act & Assert
-      expect(() => jwt.generateJwt(undefined)).toThrow();
-    });
-  });
-});

package/test/lib/logger.test.js

@@ -1,206 +0,0 @@
-const { Logger, LOG_LEVELS } = require('../../lib/logger');
-
-describe('Logger', () => {
-  let originalEnv;
-  let consoleSpy;
-  let consoleErrorSpy;
-
-  beforeEach(() => {
-    originalEnv = process.env.NODE_ENV;
-    consoleSpy = jest.spyOn(console, 'log').mockImplementation();
-    consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
-  });
-
-  afterEach(() => {
-    process.env.NODE_ENV = originalEnv;
-    consoleSpy.mockRestore();
-    consoleErrorSpy.mockRestore();
-  });
-
-  describe('Log Levels', () => {
-    test('should only log messages at or above the configured level', () => {
-      const logger = new Logger({ level: 'WARN' });
-
-      logger.debug('debug message');
-      logger.info('info message');
-      logger.warn('warn message');
-      logger.error('error message');
-
-      // Only warn and error should be logged
-      expect(consoleSpy).not.toHaveBeenCalled();
-      expect(consoleErrorSpy).toHaveBeenCalledTimes(2);
-    });
-
-    test('should log all messages when level is DEBUG', () => {
-      const logger = new Logger({ level: 'DEBUG' });
-
-      logger.debug('debug message');
-      logger.info('info message');
-      logger.warn('warn message');
-      logger.error('error message');
-
-      expect(consoleSpy).toHaveBeenCalledTimes(2); // debug and info
-      expect(consoleErrorSpy).toHaveBeenCalledTimes(2); // warn and error
-    });
-  });
-
-  describe('Production vs Development', () => {
-    test('should output JSON in production', () => {
-      process.env.NODE_ENV = 'production';
-      const logger = new Logger({ level: 'INFO' });
-
-      logger.info('test message', { userId: '123' });
-
-      const logOutput = consoleSpy.mock.calls[0][0];
-      const parsed = JSON.parse(logOutput);
-
-      expect(parsed).toHaveProperty('timestamp');
-      expect(parsed).toHaveProperty('level', 'INFO');
-      expect(parsed).toHaveProperty('message', 'test message');
-      expect(parsed).toHaveProperty('userId', '123');
-    });
-
-    test('should output human-readable format in development', () => {
-      process.env.NODE_ENV = 'development';
-      const logger = new Logger({ level: 'INFO' });
-
-      logger.info('test message', { userId: '123' });
-
-      const logOutput = consoleSpy.mock.calls[0][0];
-      expect(logOutput).toContain('[INFO]');
-      expect(logOutput).toContain('test message');
-    });
-  });
-
-  describe('Error Handling', () => {
-    test('should extract error information from Error objects', () => {
-      process.env.NODE_ENV = 'production';
-      const logger = new Logger({ level: 'ERROR' });
-      const error = new Error('Test error');
-      error.response = {
-        status: 500,
-        data: { message: 'Server error' },
-      };
-
-      logger.error('Operation failed', { error });
-
-      const logOutput = consoleErrorSpy.mock.calls[0][0];
-      const parsed = JSON.parse(logOutput);
-
-      expect(parsed).toHaveProperty('errorMessage', 'Test error');
-      expect(parsed).toHaveProperty('errorStack');
-      expect(parsed).toHaveProperty('errorStatus', 500);
-      expect(parsed).toHaveProperty('errorResponse');
-      expect(parsed).not.toHaveProperty('error'); // Should be removed
-    });
-  });
-
-  describe('Child Logger', () => {
-    test('should create child logger with default context', () => {
-      process.env.NODE_ENV = 'production';
-      const logger = new Logger({ level: 'INFO' });
-      const childLogger = logger.child({ platform: 'clio', userId: '123' });
-
-      childLogger.info('test message', { operation: 'createLog' });
-
-      const logOutput = consoleSpy.mock.calls[0][0];
-      const parsed = JSON.parse(logOutput);
-
-      expect(parsed).toHaveProperty('platform', 'clio');
-      expect(parsed).toHaveProperty('userId', '123');
-      expect(parsed).toHaveProperty('operation', 'createLog');
-    });
-
-    test('should support nested child loggers', () => {
-      process.env.NODE_ENV = 'production';
-      const logger = new Logger({ level: 'INFO' });
-      const child1 = logger.child({ platform: 'clio' });
-      const child2 = child1.child({ userId: '123' });
-
-      child2.info('test message');
-
-      const logOutput = consoleSpy.mock.calls[0][0];
-      const parsed = JSON.parse(logOutput);
-
-      expect(parsed).toHaveProperty('platform', 'clio');
-      expect(parsed).toHaveProperty('userId', '123');
-    });
-  });
-
-  describe('API Request Logging', () => {
-    test('should log successful API requests at DEBUG level', () => {
-      const logger = new Logger({ level: 'DEBUG' });
-
-      logger.logApiRequest({
-        method: 'GET',
-        url: 'https://api.example.com/users',
-        status: 200,
-        duration: 150,
-        platform: 'clio',
-      });
-
-      expect(consoleSpy).toHaveBeenCalledTimes(1);
-      expect(consoleErrorSpy).not.toHaveBeenCalled();
-    });
-
-    test('should log failed API requests at ERROR level', () => {
-      const logger = new Logger({ level: 'ERROR' });
-      const error = new Error('Request failed');
-
-      logger.logApiRequest({
-        method: 'POST',
-        url: 'https://api.example.com/logs',
-        status: 500,
-        duration: 300,
-        platform: 'clio',
-        error,
-      });
-
-      expect(consoleErrorSpy).toHaveBeenCalledTimes(1);
-      expect(consoleSpy).not.toHaveBeenCalled();
-    });
-
-    test('should log 4xx errors at WARN level', () => {
-      const logger = new Logger({ level: 'WARN' });
-
-      logger.logApiRequest({
-        method: 'GET',
-        url: 'https://api.example.com/contact/999',
-        status: 404,
-        duration: 100,
-        platform: 'clio',
-      });
-
-      expect(consoleErrorSpy).toHaveBeenCalledTimes(1);
-    });
-  });
-
-  describe('Database Query Logging', () => {
-    test('should log successful queries at DEBUG level', () => {
-      const logger = new Logger({ level: 'DEBUG' });
-
-      logger.logDatabaseQuery({
-        operation: 'SELECT',
-        table: 'users',
-        duration: 50,
-      });
-
-      expect(consoleSpy).toHaveBeenCalledTimes(1);
-    });
-
-    test('should log failed queries at ERROR level', () => {
-      const logger = new Logger({ level: 'ERROR' });
-      const error = new Error('Constraint violation');
-
-      logger.logDatabaseQuery({
-        operation: 'INSERT',
-        table: 'call_logs',
-        duration: 20,
-        error,
-      });
-
-      expect(consoleErrorSpy).toHaveBeenCalledTimes(1);
-    });
-  });
-});
-