@hubspot/cli 7.6.0-beta.2 → 7.6.0-beta.4

package/lib/projects/__tests__/LocalDevProcess.test.js

@@ -0,0 +1,314 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const path_1 = __importDefault(require("path"));
+const project_parsing_lib_1 = require("@hubspot/project-parsing-lib");
+const upload_1 = require("../upload");
+const config_1 = require("../config");
+const LocalDevProcess_1 = __importDefault(require("../localDev/LocalDevProcess"));
+const LocalDevLogger_1 = __importDefault(require("../localDev/LocalDevLogger"));
+const DevServerManagerV2_1 = __importDefault(require("../localDev/DevServerManagerV2"));
+const environments_1 = require("@hubspot/local-dev-lib/constants/environments");
+const vitest_1 = require("vitest");
+// Mock @hubspot/ui-extensions-dev-server
+vitest_1.vi.mock('@hubspot/ui-extensions-dev-server', () => ({
+    DevModeUnifiedInterface: {
+        setup: vitest_1.vi.fn().mockResolvedValue(undefined),
+        start: vitest_1.vi.fn().mockResolvedValue(undefined),
+        fileChange: vitest_1.vi.fn().mockResolvedValue(undefined),
+        cleanup: vitest_1.vi.fn().mockResolvedValue(undefined),
+    },
+}));
+vitest_1.vi.mock('@hubspot/project-parsing-lib');
+vitest_1.vi.mock('../upload');
+vitest_1.vi.mock('../config');
+vitest_1.vi.mock('../localDev/LocalDevLogger');
+vitest_1.vi.mock('../localDev/DevServerManagerV2');
+// Tests for LocalDevProcess and LocalDevState
+describe('LocalDevProcess', () => {
+    let mockLocalDevLogger;
+    let mockDevServerManager;
+    let process;
+    const mockProjectConfig = {
+        name: 'test-project',
+        srcDir: 'src',
+        platformVersion: '1.0.0',
+    };
+    const mockSubbuildStatus = {
+        buildName: 'component1',
+        buildType: 'APP',
+        errorMessage: '',
+        finishedAt: new Date().toISOString(),
+        rootPath: '/test/path',
+        startedAt: new Date().toISOString(),
+        status: 'SUCCESS',
+        id: '123',
+        standardError: null,
+        visible: true,
+    };
+    const mockBuild = {
+        activitySource: { type: 'HUBSPOT' },
+        projectName: 'test-project',
+        uploadMessage: 'test-upload-message',
+        autoDeployId: 123,
+        buildId: 123,
+        createdAt: new Date().toISOString(),
+        deployableState: 'DEPLOYED',
+        finishedAt: new Date().toISOString(),
+        startedAt: new Date().toISOString(),
+        status: 'SUCCESS',
+        subbuildStatuses: [
+            { ...mockSubbuildStatus, buildName: 'component1' },
+            { ...mockSubbuildStatus, buildName: 'component2' },
+        ],
+        deployStatusTaskLocator: {
+            id: '123',
+            links: [],
+        },
+        enqueuedAt: new Date().toISOString(),
+        isAutoDeployEnabled: false,
+        portalId: 123,
+    };
+    const mockOptions = {
+        projectDir: '/test/project',
+        projectConfig: mockProjectConfig,
+        targetProjectAccountId: 123,
+        targetTestingAccountId: 456,
+        projectId: 789,
+        isGithubLinked: false,
+        initialProjectNodes: {},
+        env: environments_1.ENVIRONMENTS.PROD,
+        deployedBuild: mockBuild,
+        projectName: 'test-project',
+    };
+    beforeEach(() => {
+        vitest_1.vi.clearAllMocks();
+        mockLocalDevLogger = {
+            resetSpinnies: vitest_1.vi.fn(),
+            devServerSetupError: vitest_1.vi.fn(),
+            devServerStartError: vitest_1.vi.fn(),
+            devServerCleanupError: vitest_1.vi.fn(),
+            missingComponentsWarning: vitest_1.vi.fn(),
+            noDeployedBuild: vitest_1.vi.fn(),
+            startupMessage: vitest_1.vi.fn(),
+            monitorConsoleOutput: vitest_1.vi.fn(),
+            cleanupStart: vitest_1.vi.fn(),
+            cleanupError: vitest_1.vi.fn(),
+            cleanupSuccess: vitest_1.vi.fn(),
+            uploadInitiated: vitest_1.vi.fn(),
+            projectConfigMismatch: vitest_1.vi.fn(),
+            uploadError: vitest_1.vi.fn(),
+            uploadSuccess: vitest_1.vi.fn(),
+            clearUploadWarnings: vitest_1.vi.fn(),
+            fileChangeError: vitest_1.vi.fn(),
+            uploadWarning: vitest_1.vi.fn(),
+        };
+        mockDevServerManager = {
+            setup: vitest_1.vi.fn().mockResolvedValue(undefined),
+            start: vitest_1.vi.fn().mockResolvedValue(undefined),
+            cleanup: vitest_1.vi.fn().mockResolvedValue(undefined),
+            fileChange: vitest_1.vi.fn().mockResolvedValue(undefined),
+        };
+        // Mock constructors
+        LocalDevLogger_1.default.mockImplementation(() => mockLocalDevLogger);
+        DevServerManagerV2_1.default.mockImplementation(() => mockDevServerManager);
+        // Create process instance
+        process = new LocalDevProcess_1.default(mockOptions);
+        // Mock process.exit
+        vitest_1.vi.spyOn(global.process, 'exit').mockImplementation((code) => {
+            throw new Error(`Process.exit called with code ${code}`);
+        });
+    });
+    describe('start()', () => {
+        it('should exit if no deployed build exists', async () => {
+            const processWithoutBuild = new LocalDevProcess_1.default({
+                ...mockOptions,
+                deployedBuild: undefined,
+            });
+            await expect(processWithoutBuild.start()).rejects.toThrow('Process.exit called with code 0');
+            expect(mockLocalDevLogger.noDeployedBuild).toHaveBeenCalled();
+        });
+        it('should exit if dev server setup fails', async () => {
+            mockDevServerManager.setup.mockRejectedValue(new Error('Setup failed'));
+            await expect(process.start()).rejects.toThrow('Process.exit called with code 1');
+            expect(mockLocalDevLogger.devServerSetupError).toHaveBeenCalled();
+        });
+        it('should start successfully and compare project nodes', async () => {
+            await process.start();
+            expect(mockLocalDevLogger.startupMessage).toHaveBeenCalled();
+            expect(mockDevServerManager.start).toHaveBeenCalled();
+            expect(mockLocalDevLogger.monitorConsoleOutput).toHaveBeenCalled();
+            expect(mockLocalDevLogger.missingComponentsWarning).not.toHaveBeenCalled();
+        });
+        it('should warn about missing components', async () => {
+            const mockNode = {
+                uid: 'component3',
+                componentType: 'APP',
+                localDev: {
+                    componentRoot: '/test/path',
+                    componentConfigPath: '/test/path/config.json',
+                    configUpdatedSinceLastUpload: false,
+                },
+                componentDeps: {},
+                metaFilePath: '/test/path',
+                config: {},
+                files: [],
+            };
+            const processWithNode = new LocalDevProcess_1.default({
+                ...mockOptions,
+                initialProjectNodes: {
+                    component3: mockNode,
+                },
+            });
+            await processWithNode.start();
+            expect(mockLocalDevLogger.missingComponentsWarning).toHaveBeenCalledWith([
+                '[App] component3',
+            ]);
+        });
+    });
+    describe('stop()', () => {
+        it('should exit with error if cleanup fails', async () => {
+            mockDevServerManager.cleanup.mockRejectedValue(new Error('Cleanup failed'));
+            await expect(process.stop()).rejects.toThrow('Process.exit called with code 1');
+            expect(mockLocalDevLogger.cleanupError).toHaveBeenCalled();
+        });
+        it('should exit successfully after cleanup', async () => {
+            await expect(process.stop()).rejects.toThrow('Process.exit called with code 0');
+            expect(mockLocalDevLogger.cleanupSuccess).toHaveBeenCalled();
+        });
+        it('should not show progress messages when showProgress is false', async () => {
+            await expect(process.stop(false)).rejects.toThrow('Process.exit called with code 0');
+            expect(mockLocalDevLogger.cleanupStart).not.toHaveBeenCalled();
+            expect(mockLocalDevLogger.cleanupSuccess).not.toHaveBeenCalled();
+        });
+    });
+    describe('updateProjectNodes()', () => {
+        it('should update project nodes with translated representation', async () => {
+            const mockNodes = { node1: { uid: 'node1' } };
+            project_parsing_lib_1.translateForLocalDev.mockResolvedValue({
+                intermediateNodesIndexedByUid: mockNodes,
+            });
+            // @ts-expect-error testing private method
+            await process.updateProjectNodes();
+            expect(project_parsing_lib_1.translateForLocalDev).toHaveBeenCalledWith({
+                projectSourceDir: path_1.default.join(mockOptions.projectDir, mockOptions.projectConfig.srcDir),
+                platformVersion: mockOptions.projectConfig.platformVersion,
+                accountId: mockOptions.targetProjectAccountId,
+            }, { projectNodesAtLastUpload: {} });
+            expect(process.projectNodes).toEqual(mockNodes);
+        });
+    });
+    describe('uploadProject()', () => {
+        it('should not upload if project config is invalid', async () => {
+            config_1.getProjectConfig.mockResolvedValue({
+                projectConfig: null,
+            });
+            await process.uploadProject();
+            expect(mockLocalDevLogger.projectConfigMismatch).toHaveBeenCalled();
+            expect(upload_1.handleProjectUpload).not.toHaveBeenCalled();
+        });
+        it('should handle upload error', async () => {
+            config_1.getProjectConfig.mockResolvedValue({
+                projectConfig: mockOptions.projectConfig,
+            });
+            upload_1.handleProjectUpload.mockResolvedValue({
+                uploadError: new Error('Upload failed'),
+            });
+            const success = await process.uploadProject();
+            expect(mockLocalDevLogger.uploadError).toHaveBeenCalledWith(new Error('Upload failed'));
+            expect(success).toBe(false);
+        });
+        it('should handle successful upload', async () => {
+            await process.handleConfigFileChange();
+            config_1.getProjectConfig.mockResolvedValue({
+                projectConfig: mockOptions.projectConfig,
+            });
+            upload_1.handleProjectUpload.mockResolvedValue({
+                uploadError: null,
+            });
+            const success = await process.uploadProject();
+            expect(mockLocalDevLogger.uploadSuccess).toHaveBeenCalled();
+            expect(mockLocalDevLogger.clearUploadWarnings).toHaveBeenCalled();
+            expect(success).toBe(true);
+        });
+        it('should reset projectNodesAtLastUpload', async () => {
+            const mockNodes = { node1: { uid: 'node1' } };
+            const initialProjectNodes = { existingNode: { uid: 'existingNode' } };
+            // @ts-expect-error accessing private property for testing
+            process.state._projectNodesAtLastUpload = initialProjectNodes;
+            config_1.getProjectConfig.mockResolvedValue({
+                projectConfig: mockOptions.projectConfig,
+            });
+            upload_1.handleProjectUpload.mockResolvedValue({
+                uploadError: null,
+            });
+            project_parsing_lib_1.translateForLocalDev.mockResolvedValue({
+                intermediateNodesIndexedByUid: mockNodes,
+            });
+            const success = await process.uploadProject();
+            // Verify translateForLocalDev was called without projectNodesAtLastUpload option
+            expect(project_parsing_lib_1.translateForLocalDev).toHaveBeenCalledWith({
+                projectSourceDir: path_1.default.join(mockOptions.projectDir, mockOptions.projectConfig.srcDir),
+                platformVersion: mockOptions.projectConfig.platformVersion,
+                accountId: mockOptions.targetProjectAccountId,
+            }, { projectNodesAtLastUpload: undefined });
+            // Verify projectNodesAtLastUpload was reset to the new nodes
+            // @ts-expect-error accessing private property for testing
+            expect(process.state.projectNodesAtLastUpload).toEqual(mockNodes);
+            expect(success).toBe(true);
+        });
+    });
+    describe('handleFileChange()', () => {
+        it('should handle file change successfully', async () => {
+            const filePath = 'test/file.js';
+            const event = 'change';
+            await process.handleFileChange(filePath, event);
+            expect(mockDevServerManager.fileChange).toHaveBeenCalledWith({
+                filePath,
+                event,
+            });
+        });
+        it('should log error if file change fails', async () => {
+            const error = new Error('File change failed');
+            mockDevServerManager.fileChange.mockImplementation(() => {
+                throw error;
+            });
+            await process.handleFileChange('test/file.js', 'change');
+            expect(mockLocalDevLogger.fileChangeError).toHaveBeenCalledWith(error);
+        });
+    });
+    describe('handleConfigFileChange()', () => {
+        beforeEach(() => {
+            project_parsing_lib_1.translateForLocalDev.mockResolvedValue({
+                intermediateNodesIndexedByUid: {},
+            });
+        });
+        it('should update project nodes and show upload warning', async () => {
+            await process.handleConfigFileChange();
+            expect(project_parsing_lib_1.translateForLocalDev).toHaveBeenCalledWith({
+                projectSourceDir: path_1.default.join(mockOptions.projectDir, mockOptions.projectConfig.srcDir),
+                platformVersion: mockOptions.projectConfig.platformVersion,
+                accountId: mockOptions.targetProjectAccountId,
+            }, { projectNodesAtLastUpload: {} });
+            expect(mockLocalDevLogger.uploadWarning).toHaveBeenCalled();
+        });
+    });
+    describe('addStateListener()', () => {
+        it('should add state listener', () => {
+            const listener = vitest_1.vi.fn();
+            const key = 'projectNodes';
+            process.addStateListener(key, listener);
+            // @ts-expect-error
+            process.state.projectNodes = {};
+            expect(listener).toHaveBeenCalled();
+        });
+        it('should call listener immediately when callOnInit is true', () => {
+            const listener = vitest_1.vi.fn();
+            const key = 'projectNodes';
+            process.addStateListener(key, listener, true);
+            expect(listener).toHaveBeenCalledWith(process.projectNodes);
+        });
+    });
+});

package/lib/projects/__tests__/LocalDevWebsocketServer.test.d.ts

@@ -0,0 +1 @@
+export {};

package/lib/projects/__tests__/LocalDevWebsocketServer.test.js

@@ -0,0 +1,175 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const ws_1 = require("ws");
+const portManager_1 = require("@hubspot/local-dev-lib/portManager");
+const logger_1 = require("@hubspot/local-dev-lib/logger");
+const constants_1 = require("../../constants");
+const LocalDevWebsocketServer_1 = __importDefault(require("../localDev/LocalDevWebsocketServer"));
+const en_1 = require("../../../lang/en");
+vi.mock('ws');
+vi.mock('@hubspot/local-dev-lib/portManager');
+vi.mock('@hubspot/local-dev-lib/logger');
+describe('LocalDevWebsocketServer', () => {
+    let mockLocalDevProcess;
+    let mockWebSocket;
+    let mockWebSocketServer;
+    let server;
+    beforeEach(() => {
+        // Reset all mocks
+        vi.clearAllMocks();
+        // Setup mock WebSocket
+        mockWebSocket = {
+            on: vi.fn(),
+            send: vi.fn(),
+            close: vi.fn(),
+        };
+        // Setup mock WebSocketServer
+        mockWebSocketServer = {
+            on: vi.fn(),
+            close: vi.fn(),
+        };
+        // Setup mock LocalDevProcess
+        mockLocalDevProcess = {
+            addStateListener: vi.fn(),
+            uploadProject: vi.fn(),
+            sendDevServerMessage: vi.fn(),
+        };
+        // Mock WebSocketServer constructor
+        ws_1.WebSocketServer.mockImplementation(() => mockWebSocketServer);
+        // Create server instance
+        server = new LocalDevWebsocketServer_1.default(mockLocalDevProcess, true);
+    });
+    describe('start()', () => {
+        it('should throw error if port manager is not running', async () => {
+            portManager_1.isPortManagerServerRunning.mockResolvedValue(false);
+            await expect(server.start()).rejects.toThrow();
+        });
+        it('should start websocket server successfully', async () => {
+            portManager_1.isPortManagerServerRunning.mockResolvedValue(true);
+            portManager_1.requestPorts.mockResolvedValue({
+                'local-dev-ui-websocket-server': 1234,
+            });
+            await server.start();
+            expect(ws_1.WebSocketServer).toHaveBeenCalledWith({ port: 1234 });
+            expect(mockWebSocketServer.on).toHaveBeenCalledWith('connection', expect.any(Function));
+            expect(logger_1.logger.log).toHaveBeenCalled();
+        });
+        it('should accept connection from valid origin', async () => {
+            portManager_1.isPortManagerServerRunning.mockResolvedValue(true);
+            portManager_1.requestPorts.mockResolvedValue({
+                'local-dev-ui-websocket-server': 1234,
+            });
+            await server.start();
+            // Get the connection callback
+            const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
+            // Simulate connection from valid origin
+            connectionCallback(mockWebSocket, {
+                headers: { origin: 'https://app.hubspot.com' },
+            });
+            expect(mockWebSocket.on).toHaveBeenCalledWith('message', expect.any(Function));
+            expect(mockLocalDevProcess.addStateListener).toHaveBeenCalledWith('projectNodes', expect.any(Function), true);
+            expect(mockWebSocket.close).not.toHaveBeenCalled();
+        });
+        it('should reject connection from invalid origin', async () => {
+            portManager_1.isPortManagerServerRunning.mockResolvedValue(true);
+            portManager_1.requestPorts.mockResolvedValue({
+                'local-dev-ui-websocket-server': 1234,
+            });
+            await server.start();
+            // Get the connection callback
+            const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
+            // Simulate connection from invalid origin
+            connectionCallback(mockWebSocket, {
+                headers: { origin: 'https://malicious-site.com' },
+            });
+            expect(mockWebSocket.close).toHaveBeenCalledWith(1008, en_1.lib.LocalDevWebsocketServer.errors.originNotAllowed('https://malicious-site.com'));
+            expect(mockWebSocket.on).not.toHaveBeenCalled();
+            expect(mockLocalDevProcess.addStateListener).not.toHaveBeenCalled();
+        });
+        it('should reject connection with no origin header', async () => {
+            portManager_1.isPortManagerServerRunning.mockResolvedValue(true);
+            portManager_1.requestPorts.mockResolvedValue({
+                'local-dev-ui-websocket-server': 1234,
+            });
+            await server.start();
+            // Get the connection callback
+            const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
+            // Simulate connection with no origin header
+            connectionCallback(mockWebSocket, {
+                headers: {},
+            });
+            expect(mockWebSocket.close).toHaveBeenCalledWith(1008, en_1.lib.LocalDevWebsocketServer.errors.originNotAllowed());
+            expect(mockWebSocket.on).not.toHaveBeenCalled();
+            expect(mockLocalDevProcess.addStateListener).not.toHaveBeenCalled();
+        });
+        it('should send WEBSOCKET_SERVER_CONNECTED message when valid connection is established', async () => {
+            portManager_1.isPortManagerServerRunning.mockResolvedValue(true);
+            portManager_1.requestPorts.mockResolvedValue({
+                'local-dev-ui-websocket-server': 1234,
+            });
+            await server.start();
+            // Get the connection callback
+            const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
+            // Simulate connection from valid origin
+            connectionCallback(mockWebSocket, {
+                headers: { origin: 'https://app.hubspot.com' },
+            });
+            expect(mockLocalDevProcess.sendDevServerMessage).toHaveBeenCalledWith(constants_1.LOCAL_DEV_SERVER_MESSAGE_TYPES.WEBSOCKET_SERVER_CONNECTED);
+        });
+    });
+    describe('message handling', () => {
+        beforeEach(async () => {
+            portManager_1.isPortManagerServerRunning.mockResolvedValue(true);
+            portManager_1.requestPorts.mockResolvedValue({
+                'local-dev-ui-websocket-server': 1234,
+            });
+            await server.start();
+            const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
+            connectionCallback(mockWebSocket, {
+                headers: { origin: 'https://app.hubspot.com' },
+            });
+        });
+        it('should handle UPLOAD message type', () => {
+            const messageCallback = mockWebSocket.on.mock.calls[0][1];
+            const message = {
+                type: constants_1.LOCAL_DEV_UI_MESSAGE_RECEIVE_TYPES.UPLOAD,
+            };
+            messageCallback(JSON.stringify(message));
+            expect(mockLocalDevProcess.uploadProject).toHaveBeenCalled();
+        });
+        it('should log error for missing message type', () => {
+            const messageCallback = mockWebSocket.on.mock.calls[0][1];
+            const message = {};
+            messageCallback(JSON.stringify(message));
+            expect(logger_1.logger.error).toHaveBeenCalled();
+        });
+        it('should log error for unknown message type', () => {
+            const messageCallback = mockWebSocket.on.mock.calls[0][1];
+            const message = {
+                type: 'UNKNOWN_TYPE',
+            };
+            messageCallback(JSON.stringify(message));
+            expect(logger_1.logger.error).toHaveBeenCalled();
+        });
+        it('should log error for invalid JSON', () => {
+            const messageCallback = mockWebSocket.on.mock.calls[0][1];
+            const invalidJson = 'invalid json';
+            messageCallback(invalidJson);
+            expect(logger_1.logger.error).toHaveBeenCalled();
+        });
+    });
+    describe('shutdown()', () => {
+        it('should close the websocket server', async () => {
+            portManager_1.isPortManagerServerRunning.mockResolvedValue(true);
+            portManager_1.requestPorts.mockResolvedValue({
+                'local-dev-ui-websocket-server': 1234,
+            });
+            await server.start();
+            server.shutdown();
+            expect(mockWebSocketServer.close).toHaveBeenCalled();
+        });
+    });
+});

package/lib/projects/__tests__/ProjectLogsManager.test.d.ts

@@ -0,0 +1 @@
+export {};