@hubspot/cli 8.1.0 → 8.1.1-experimental.0

package/commands/testAccount/__tests__/create.test.js

@@ -1,106 +0,0 @@
-import yargs from 'yargs';
-import { addAccountOptions, addConfigOptions, addUseEnvironmentOptions, addTestingOptions, addJSONOutputOptions, } from '../../../lib/commonOpts.js';
-import testAccountCreateCommand from '../create.js';
-import { ACCOUNT_LEVEL_CHOICES, ACCOUNT_LEVELS, } from '../../../lib/constants.js';
-vi.mock('../../../lib/commonOpts');
-describe('commands/testAccount/create', () => {
-    const yargsMock = yargs;
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(testAccountCreateCommand.command).toEqual('create');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(testAccountCreateCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            testAccountCreateCommand.builder(yargsMock);
-            expect(yargsMock.example).toHaveBeenCalledTimes(1);
-            expect(addTestingOptions).toHaveBeenCalledTimes(1);
-            expect(addTestingOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addJSONOutputOptions).toHaveBeenCalledTimes(1);
-            expect(addJSONOutputOptions).toHaveBeenCalledWith(yargsMock);
-        });
-        it('should add account-name option', () => {
-            testAccountCreateCommand.builder(yargsMock);
-            expect(yargsMock.option).toHaveBeenCalledWith('name', {
-                type: 'string',
-                description: 'Name for the test account',
-            });
-        });
-        it('should add description option', () => {
-            testAccountCreateCommand.builder(yargsMock);
-            expect(yargsMock.option).toHaveBeenCalledWith('description', {
-                type: 'string',
-                description: 'Description for the test account',
-            });
-        });
-        it('should add hub level options', () => {
-            testAccountCreateCommand.builder(yargsMock);
-            expect(yargsMock.option).toHaveBeenCalledWith('marketing-level', {
-                type: 'string',
-                description: 'Marketing Hub tier. Options: FREE, STARTER, PROFESSIONAL, ENTERPRISE',
-                choices: ACCOUNT_LEVEL_CHOICES,
-            });
-            expect(yargsMock.option).toHaveBeenCalledWith('ops-level', {
-                type: 'string',
-                description: 'Operations Hub tier. Options: FREE, STARTER, PROFESSIONAL, ENTERPRISE',
-                choices: ACCOUNT_LEVEL_CHOICES,
-            });
-            expect(yargsMock.option).toHaveBeenCalledWith('service-level', {
-                type: 'string',
-                description: 'Service Hub tier. Options: FREE, STARTER, PROFESSIONAL, ENTERPRISE',
-                choices: ACCOUNT_LEVEL_CHOICES,
-            });
-            expect(yargsMock.option).toHaveBeenCalledWith('sales-level', {
-                type: 'string',
-                description: 'Sales Hub tier. Options: FREE, STARTER, PROFESSIONAL, ENTERPRISE',
-                choices: ACCOUNT_LEVEL_CHOICES,
-            });
-            expect(yargsMock.option).toHaveBeenCalledWith('content-level', {
-                type: 'string',
-                description: 'CMS Hub tier. Options: FREE, STARTER, PROFESSIONAL, ENTERPRISE',
-                choices: ACCOUNT_LEVEL_CHOICES,
-            });
-            expect(yargsMock.option).toHaveBeenCalledWith('commerce-level', {
-                type: 'string',
-                description: 'Commerce Hub tier. Options: FREE, PROFESSIONAL, ENTERPRISE',
-                choices: ACCOUNT_LEVEL_CHOICES.filter(level => level !== ACCOUNT_LEVELS.STARTER),
-            });
-        });
-        it('should add examples for all usage scenarios', () => {
-            testAccountCreateCommand.builder(yargsMock);
-            expect(yargsMock.example).toHaveBeenCalledWith([
-                [
-                    '$0 test-account create',
-                    'Interactive mode - prompts for all options',
-                ],
-                [
-                    '$0 test-account create --name "MyTestAccount"',
-                    'Provide name via flag, prompt for description and tier selection',
-                ],
-                [
-                    '$0 test-account create --name "MyTestAccount" --description "Test account"',
-                    'Provide name and description, prompt for tier selection',
-                ],
-                [
-                    '$0 test-account create --name "MyTestAccount" --marketing-level PROFESSIONAL',
-                    'Specify marketing tier, other tiers default to ENTERPRISE',
-                ],
-                [
-                    '$0 test-account create --config-path ./test-account-config.json',
-                    'Create from config file (mutually exclusive with other flags)',
-                ],
-            ]);
-        });
-    });
-});

package/commands/testAccount/__tests__/createConfig.test.d.ts

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

package/commands/testAccount/__tests__/createConfig.test.js

@@ -1,32 +0,0 @@
-import yargs from 'yargs';
-import testAccountCreateConfigCommand from '../createConfig.js';
-vi.mock('../../../lib/commonOpts');
-describe('commands/testAccount/createConfig', () => {
-    const yargsMock = yargs;
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(testAccountCreateConfigCommand.command).toEqual('create-config');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(testAccountCreateConfigCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            const optionSpy = vi.spyOn(yargsMock, 'option');
-            testAccountCreateConfigCommand.builder(yargsMock);
-            expect(yargsMock.example).toHaveBeenCalledTimes(1);
-            expect(optionSpy).toHaveBeenCalledWith('name', expect.objectContaining({
-                type: 'string',
-            }));
-            expect(optionSpy).toHaveBeenCalledWith('description', expect.objectContaining({
-                type: 'string',
-            }));
-            expect(optionSpy).toHaveBeenCalledWith('path', expect.objectContaining({
-                type: 'string',
-            }));
-        });
-    });
-});

package/commands/testAccount/__tests__/delete.test.d.ts

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

package/commands/testAccount/__tests__/delete.test.js

@@ -1,29 +0,0 @@
-import yargs from 'yargs';
-import { addConfigOptions, addUseEnvironmentOptions, addTestingOptions, } from '../../../lib/commonOpts.js';
-import testAccountDeleteCommand from '../delete.js';
-vi.mock('../../../lib/commonOpts');
-describe('commands/testAccount/delete', () => {
-    const yargsMock = yargs;
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(testAccountDeleteCommand.command).toEqual('delete [test-account]');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(testAccountDeleteCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            testAccountDeleteCommand.builder(yargsMock);
-            expect(yargsMock.example).toHaveBeenCalledTimes(1);
-            expect(addTestingOptions).toHaveBeenCalledTimes(1);
-            expect(addTestingOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
-        });
-    });
-});

package/commands/testAccount/__tests__/importData.test.d.ts

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

package/commands/testAccount/__tests__/importData.test.js

@@ -1,92 +0,0 @@
-import yargs from 'yargs';
-import { addAccountOptions, addConfigOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
-import testAccountImportDataCommand from '../importData.js';
-import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
-import { handleImportData, handleTargetTestAccountSelectionFlow, } from '../../../lib/importData.js';
-import { trackCommandUsage } from '../../../lib/usageTracking.js';
-import { getImportDataRequest } from '@hubspot/local-dev-lib/crm';
-import { logError } from '../../../lib/errorHandlers/index.js';
-import { importDataFilePathPrompt } from '../../../lib/prompts/importDataFilePathPrompt.js';
-import { confirmImportDataPrompt } from '../../../lib/prompts/confirmImportDataPrompt.js';
-vi.mock('../../../lib/commonOpts');
-vi.mock('../../../lib/importData');
-vi.mock('@hubspot/local-dev-lib/crm');
-vi.mock('../../../lib/errorHandlers/index');
-vi.mock('../../../lib/prompts/importDataFilePathPrompt');
-vi.mock('../../../lib/prompts/confirmImportDataPrompt');
-describe('commands/testAccount/importData', () => {
-    const yargsMock = yargs;
-    const mockExit = vi
-        .spyOn(process, 'exit')
-        .mockImplementation(() => undefined);
-    const mockHandleImportData = vi.mocked(handleImportData);
-    const mockHandleTargetTestAccountSelectionFlow = vi.mocked(handleTargetTestAccountSelectionFlow);
-    const mockTrackCommandUsage = vi.mocked(trackCommandUsage);
-    const mockGetImportDataRequest = vi.mocked(getImportDataRequest);
-    const mockLogError = vi.mocked(logError);
-    const mockImportDataFilePathPrompt = vi.mocked(importDataFilePathPrompt);
-    const mockConfirmImportDataPrompt = vi.mocked(confirmImportDataPrompt);
-    beforeEach(() => {
-        mockExit.mockReset();
-        mockHandleImportData.mockReset();
-        mockHandleTargetTestAccountSelectionFlow.mockReset();
-        mockTrackCommandUsage.mockReset();
-        mockGetImportDataRequest.mockReset();
-        mockLogError.mockReset();
-        mockImportDataFilePathPrompt.mockReset();
-        mockConfirmImportDataPrompt.mockReset();
-    });
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(testAccountImportDataCommand.command).toEqual('import-data');
-        });
-    });
-    // describe('describe', () => {
-    //   it('should provide a description', () => {
-    //     expect(testAccountImportDataCommand.describe).toBeDefined();
-    //   });
-    // });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            testAccountImportDataCommand.builder(yargsMock);
-            expect(yargsMock.example).toHaveBeenCalledTimes(1);
-            expect(yargsMock.options).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargsMock);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargsMock);
-        });
-    });
-    describe('handler', () => {
-        it('should complete the flow given the correct args', async () => {
-            const targetAccountId = 123456789;
-            const mockArgs = {
-                d: false,
-                debug: false,
-                _: [],
-                $0: 'hs',
-                derivedAccountId: targetAccountId,
-                userProvidedAccount: 'test-account',
-                filePath: 'test-file.json',
-                skipConfirm: true,
-            };
-            mockHandleTargetTestAccountSelectionFlow.mockResolvedValue(targetAccountId);
-            mockGetImportDataRequest.mockReturnValue({
-                importRequest: {},
-                dataFileNames: ['test-file.json'],
-            });
-            mockHandleImportData.mockResolvedValue();
-            await testAccountImportDataCommand.handler(mockArgs);
-            expect(mockHandleTargetTestAccountSelectionFlow).toHaveBeenCalledWith(123456789, 'test-account');
-            expect(mockGetImportDataRequest).toHaveBeenCalledWith('test-file.json');
-            expect(mockHandleImportData).toHaveBeenCalledTimes(1);
-            expect(mockTrackCommandUsage).toHaveBeenCalledTimes(1);
-            expect(mockLogError).not.toHaveBeenCalled();
-            expect(mockImportDataFilePathPrompt).not.toHaveBeenCalled();
-            expect(mockConfirmImportDataPrompt).not.toHaveBeenCalled();
-            expect(mockExit).toHaveBeenCalledWith(EXIT_CODES.SUCCESS);
-        });
-    });
-});

package/lib/__tests__/CLIWebSocketServer.test.d.ts

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

package/lib/__tests__/CLIWebSocketServer.test.js

@@ -1,252 +0,0 @@
-import { WebSocketServer } from 'ws';
-import { isPortManagerServerRunning, requestPorts, } from '@hubspot/local-dev-lib/portManager';
-import { uiLogger } from '../ui/logger.js';
-import CLIWebSocketServer from '../CLIWebSocketServer.js';
-import { lib } from '../../lang/en.js';
-import { pkg } from '../jsonLoader.js';
-vi.mock('ws');
-vi.mock('@hubspot/local-dev-lib/portManager');
-const INSTANCE_ID = 'test-websocket-server';
-const LOG_PREFIX = '[TestWebSocketServer]';
-const PORT = 1234;
-describe('CLIWebSocketServer', () => {
-    let mockWebSocket;
-    let mockWebSocketServer;
-    let server;
-    beforeEach(() => {
-        mockWebSocket = {
-            on: vi.fn(),
-            send: vi.fn(),
-            close: vi.fn(),
-        };
-        mockWebSocketServer = {
-            on: vi.fn(),
-            close: vi.fn(),
-        };
-        WebSocketServer.mockImplementation(() => mockWebSocketServer);
-        server = new CLIWebSocketServer({
-            instanceId: INSTANCE_ID,
-            logPrefix: LOG_PREFIX,
-            debug: true,
-        });
-    });
-    describe('start()', () => {
-        it('should throw error if port manager is not running', async () => {
-            isPortManagerServerRunning.mockResolvedValue(false);
-            await expect(server.start({})).rejects.toThrow(lib.CLIWebsocketServer.errors.portManagerNotRunning(LOG_PREFIX));
-        });
-        it('should start websocket server on the assigned port', async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                [INSTANCE_ID]: PORT,
-            });
-            await server.start({});
-            expect(WebSocketServer).toHaveBeenCalledWith({ port: PORT });
-            expect(mockWebSocketServer.on).toHaveBeenCalledWith('connection', expect.any(Function));
-        });
-        it('should log startup message when debug is enabled', async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                [INSTANCE_ID]: PORT,
-            });
-            await server.start({});
-            expect(uiLogger.log).toHaveBeenCalledWith(expect.stringContaining(String(PORT)));
-        });
-    });
-    describe('origin validation', () => {
-        const validOrigins = [
-            'https://app.hubspot.com',
-            'https://app.hubspotqa.com',
-            'https://local.hubspot.com',
-            'https://local.hubspotqa.com',
-            'https://app-na2.hubspot.com',
-            'https://app-na2.hubspotqa.com',
-            'https://app-na3.hubspot.com',
-            'https://app-na3.hubspotqa.com',
-            'https://app-ap1.hubspot.com',
-            'https://app-ap1.hubspotqa.com',
-            'https://app-eu1.hubspot.com',
-            'https://app-eu1.hubspotqa.com',
-        ];
-        let connectionCallback;
-        beforeEach(async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                [INSTANCE_ID]: PORT,
-            });
-            await server.start({});
-            connectionCallback = mockWebSocketServer.on.mock
-                .calls[0][1];
-        });
-        validOrigins.forEach(origin => {
-            it(`should accept connection from ${origin}`, () => {
-                connectionCallback(mockWebSocket, { headers: { origin } });
-                expect(mockWebSocket.close).not.toHaveBeenCalled();
-                expect(mockWebSocket.on).toHaveBeenCalledWith('message', expect.any(Function));
-            });
-        });
-        const invalidOrigins = [
-            'https://malicious-site.com',
-            'https://app.malicious-site.com',
-            'https://app.hubspot.com.evil.com',
-        ];
-        invalidOrigins.forEach(origin => {
-            it(`should reject connection from "${origin}"`, () => {
-                connectionCallback(mockWebSocket, { headers: { origin } });
-                expect(mockWebSocket.close).toHaveBeenCalledWith(1008, lib.CLIWebsocketServer.errors.originNotAllowed(origin));
-                expect(mockWebSocket.on).not.toHaveBeenCalled();
-            });
-        });
-        it('should reject connection with no origin header', () => {
-            connectionCallback(mockWebSocket, { headers: {} });
-            expect(mockWebSocket.close).toHaveBeenCalledWith(1008, lib.CLIWebsocketServer.errors.originNotAllowed());
-            expect(mockWebSocket.on).not.toHaveBeenCalled();
-        });
-    });
-    describe('sendCliMetadata', () => {
-        let connectionCallback;
-        beforeEach(async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                [INSTANCE_ID]: PORT,
-            });
-        });
-        it('should send cliVersion on connection', async () => {
-            await server.start({});
-            connectionCallback = mockWebSocketServer.on.mock
-                .calls[0][1];
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://app.hubspot.com' },
-            });
-            expect(mockWebSocket.send).toHaveBeenCalledWith(JSON.stringify({
-                type: 'server:cliMetadata',
-                data: { cliVersion: pkg.version },
-            }));
-        });
-        it('should merge additional metadata when provided', async () => {
-            await server.start({
-                metadata: { customField: 'customValue', version: 2 },
-            });
-            connectionCallback = mockWebSocketServer.on.mock
-                .calls[0][1];
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://app.hubspot.com' },
-            });
-            expect(mockWebSocket.send).toHaveBeenCalledWith(JSON.stringify({
-                type: 'server:cliMetadata',
-                data: {
-                    cliVersion: pkg.version,
-                    customField: 'customValue',
-                    version: 2,
-                },
-            }));
-        });
-    });
-    describe('callbacks', () => {
-        let connectionCallback;
-        beforeEach(async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                [INSTANCE_ID]: PORT,
-            });
-        });
-        it('should call onConnection for valid connections', async () => {
-            const onConnection = vi.fn();
-            await server.start({ onConnection });
-            connectionCallback = mockWebSocketServer.on.mock
-                .calls[0][1];
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://app.hubspot.com' },
-            });
-            expect(onConnection).toHaveBeenCalledWith(mockWebSocket);
-        });
-        it('should not call onConnection for rejected connections', async () => {
-            const onConnection = vi.fn();
-            await server.start({ onConnection });
-            connectionCallback = mockWebSocketServer.on.mock
-                .calls[0][1];
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://malicious.com' },
-            });
-            expect(onConnection).not.toHaveBeenCalled();
-        });
-        it('should call onMessage when a valid message is received', async () => {
-            const onMessage = vi.fn().mockReturnValue(true);
-            await server.start({ onMessage });
-            connectionCallback = mockWebSocketServer.on.mock
-                .calls[0][1];
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://app.hubspot.com' },
-            });
-            const messageCallback = mockWebSocket.on.mock.calls.find(call => call[0] === 'message')[1];
-            messageCallback(JSON.stringify({ type: 'test:action', data: { key: 'value' } }));
-            expect(onMessage).toHaveBeenCalledWith(mockWebSocket, {
-                type: 'test:action',
-                data: { key: 'value' },
-            });
-        });
-        it('should log error when onMessage returns false', async () => {
-            const onMessage = vi.fn().mockReturnValue(false);
-            await server.start({ onMessage });
-            connectionCallback = mockWebSocketServer.on.mock
-                .calls[0][1];
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://app.hubspot.com' },
-            });
-            const messageCallback = mockWebSocket.on.mock.calls.find(call => call[0] === 'message')[1];
-            messageCallback(JSON.stringify({ type: 'UNKNOWN_TYPE' }));
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('UNKNOWN_TYPE'));
-        });
-        it('should call onClose when server closes', async () => {
-            const onClose = vi.fn();
-            await server.start({ onClose });
-            const closeCallback = mockWebSocketServer.on.mock.calls.find(call => call[0] === 'close')[1];
-            closeCallback();
-            expect(onClose).toHaveBeenCalled();
-        });
-    });
-    describe('message parsing', () => {
-        let messageCallback;
-        beforeEach(async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                [INSTANCE_ID]: PORT,
-            });
-            await server.start({});
-            const connectionCallback = mockWebSocketServer.on.mock.calls[0][1];
-            connectionCallback(mockWebSocket, {
-                headers: { origin: 'https://app.hubspot.com' },
-            });
-            messageCallback = mockWebSocket.on.mock.calls.find(call => call[0] === 'message')[1];
-        });
-        it('should log error for messages missing a type field', () => {
-            messageCallback(JSON.stringify({}));
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('Missing type field'));
-        });
-        it('should log error for invalid JSON', () => {
-            messageCallback('not valid json');
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('Invalid JSON'));
-        });
-    });
-    describe('sendMessage()', () => {
-        it('should serialize and send a message to the websocket', () => {
-            const message = { type: 'test:message', data: { foo: 'bar' } };
-            server.sendMessage(mockWebSocket, message);
-            expect(mockWebSocket.send).toHaveBeenCalledWith(JSON.stringify(message));
-        });
-    });
-    describe('shutdown()', () => {
-        it('should close the websocket server', async () => {
-            isPortManagerServerRunning.mockResolvedValue(true);
-            requestPorts.mockResolvedValue({
-                [INSTANCE_ID]: PORT,
-            });
-            await server.start({});
-            server.shutdown();
-            expect(mockWebSocketServer.close).toHaveBeenCalled();
-        });
-        it('should handle shutdown when server was never started', () => {
-            expect(() => server.shutdown()).not.toThrow();
-        });
-    });
-});

package/lib/__tests__/accountAuth.test.d.ts

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