@hubspot/cli 8.3.0 → 8.4.0-beta.0

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

@@ -1,231 +0,0 @@
-import { vi } from 'vitest';
-import { validateBuildIdForDeploy, logDeployErrors, handleProjectDeploy, } from '../deploy.js';
-import { uiLogger } from '../../ui/logger.js';
-import { commands } from '../../../lang/en.js';
-import { PROJECT_ERROR_TYPES } from '../../constants.js';
-import { deployProjectV1, deployProjectV2, } from '@hubspot/local-dev-lib/api/projects';
-import { pollDeployStatus } from '../pollProjectBuildAndDeploy.js';
-// Mock external dependencies
-vi.mock('@hubspot/local-dev-lib/api/projects');
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('../pollProjectBuildAndDeploy.js');
-const mockUiLogger = vi.mocked(uiLogger);
-const mockDeployProjectV1 = vi.mocked(deployProjectV1);
-const mockDeployProjectV2 = vi.mocked(deployProjectV2);
-const mockPollDeployStatus = vi.mocked(pollDeployStatus);
-describe('lib/projects/deploy', () => {
-    beforeEach(() => {
-        vi.resetAllMocks();
-    });
-    describe('validateBuildIdForDeploy()', () => {
-        const accountId = 12345;
-        const projectName = 'test-project';
-        it('returns true when build ID is valid for deployment', () => {
-            const buildId = 5;
-            const deployedBuildId = 3;
-            const latestBuildId = 10;
-            const result = validateBuildIdForDeploy(buildId, deployedBuildId, latestBuildId, projectName, accountId);
-            expect(result).toBe(true);
-        });
-        it('returns error message when build ID does not exist', () => {
-            const buildId = 15;
-            const deployedBuildId = 3;
-            const latestBuildId = 10;
-            const result = validateBuildIdForDeploy(buildId, deployedBuildId, latestBuildId, projectName, accountId);
-            expect(result).toBe(commands.project.deploy.errors.buildIdDoesNotExist(accountId, buildId, projectName));
-        });
-        it('returns error message when build is already deployed', () => {
-            const buildId = 3;
-            const deployedBuildId = 3;
-            const latestBuildId = 10;
-            const result = validateBuildIdForDeploy(buildId, deployedBuildId, latestBuildId, projectName, accountId);
-            expect(result).toBe(commands.project.deploy.errors.buildAlreadyDeployed(accountId, buildId, projectName));
-        });
-        it('handles edge case when deployedBuildId is undefined', () => {
-            const buildId = 5;
-            const deployedBuildId = undefined;
-            const latestBuildId = 10;
-            const result = validateBuildIdForDeploy(buildId, deployedBuildId, latestBuildId, projectName, accountId);
-            expect(result).toBe(true);
-        });
-    });
-    describe('logDeployErrors()', () => {
-        it('logs main error message and individual error messages', () => {
-            const errorData = {
-                message: 'Deploy failed with errors',
-                errors: [
-                    {
-                        message: 'Component error 1',
-                        subCategory: 'SOME_ERROR',
-                        context: { COMPONENT_NAME: 'test-component' },
-                    },
-                    {
-                        message: 'Component error 2',
-                        subCategory: 'ANOTHER_ERROR',
-                        context: { COMPONENT_NAME: 'another-component' },
-                    },
-                ],
-            };
-            logDeployErrors(errorData);
-            expect(mockUiLogger.error).toHaveBeenCalledWith('Deploy failed with errors');
-            expect(mockUiLogger.log).toHaveBeenCalledWith('Component error 1');
-            expect(mockUiLogger.log).toHaveBeenCalledWith('Component error 2');
-        });
-        it('handles DEPLOY_CONTAINS_REMOVALS error type specially', () => {
-            const errorData = {
-                message: 'Deploy contains removals',
-                errors: [
-                    {
-                        message: 'Component will be removed',
-                        subCategory: PROJECT_ERROR_TYPES.DEPLOY_CONTAINS_REMOVALS,
-                        context: { COMPONENT_NAME: 'removed-component' },
-                    },
-                ],
-            };
-            logDeployErrors(errorData);
-            expect(mockUiLogger.error).toHaveBeenCalledWith('Deploy contains removals');
-            expect(mockUiLogger.log).toHaveBeenCalledWith(commands.project.deploy.errors.deployContainsRemovals('removed-component'));
-        });
-        it('handles empty errors array', () => {
-            const errorData = {
-                message: 'No specific errors',
-                errors: [],
-            };
-            logDeployErrors(errorData);
-            expect(mockUiLogger.error).toHaveBeenCalledWith('No specific errors');
-            expect(mockUiLogger.log).not.toHaveBeenCalled();
-        });
-    });
-    describe('handleProjectDeploy()', () => {
-        const targetAccountId = 12345;
-        const projectName = 'test-project';
-        const buildId = 5;
-        const useV2Api = true;
-        const force = false;
-        it('successfully deploys and returns deploy result', async () => {
-            const mockDeployResponseData = {
-                id: 'deploy-123',
-                buildResultType: 'DEPLOY_QUEUED',
-                links: {
-                    status: 'http://status-url',
-                },
-            };
-            const mockDeployResult = {
-                deployId: 123,
-                buildId: 5,
-                status: 'SUCCESS',
-                enqueuedAt: '2023-01-01T00:00:00Z',
-                startedAt: '2023-01-01T00:01:00Z',
-                finishedAt: '2023-01-01T00:05:00Z',
-                portalId: targetAccountId,
-                projectName: 'test-project',
-                userId: 456,
-                source: 'HUBSPOT_USER',
-                subdeployStatuses: [],
-            };
-            mockDeployProjectV2.mockResolvedValue({
-                data: mockDeployResponseData,
-            });
-            mockPollDeployStatus.mockResolvedValue(mockDeployResult);
-            const deploy = await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
-            expect(mockDeployProjectV2).toHaveBeenCalledWith(targetAccountId, projectName, buildId, force);
-            expect(deploy).toEqual(mockDeployResult);
-        });
-        it('handles blocked deploy with warnings', async () => {
-            const mockBlockedResponse = {
-                buildResultType: 'DEPLOY_BLOCKED',
-                issues: [
-                    {
-                        uid: 'component-1',
-                        componentTypeName: 'module',
-                        errorMessages: [],
-                        blockingMessages: [
-                            {
-                                message: 'This is a warning',
-                                isWarning: true,
-                            },
-                        ],
-                    },
-                ],
-            };
-            mockDeployProjectV2.mockResolvedValue({
-                data: mockBlockedResponse,
-            });
-            await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
-            expect(mockUiLogger.warn).toHaveBeenCalledWith(commands.project.deploy.errors.deployWarningsHeader);
-        });
-        it('handles blocked deploy with errors (cannot be forced)', async () => {
-            const mockBlockedResponse = {
-                buildResultType: 'DEPLOY_BLOCKED',
-                issues: [
-                    {
-                        uid: 'component-1',
-                        componentTypeName: 'module',
-                        errorMessages: [],
-                        blockingMessages: [
-                            {
-                                message: 'This is an error',
-                                isWarning: false,
-                            },
-                        ],
-                    },
-                ],
-            };
-            mockDeployProjectV2.mockResolvedValue({
-                data: mockBlockedResponse,
-            });
-            await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
-            expect(mockUiLogger.error).toHaveBeenCalledWith(commands.project.deploy.errors.deployBlockedHeader);
-            expect(mockUiLogger.log).toHaveBeenCalledWith(commands.project.deploy.errors.deployIssueComponentWarning('component-1', 'module', 'This is an error'));
-        });
-        it('handles blocked deploy with no blocking messages', async () => {
-            const mockBlockedResponse = {
-                buildResultType: 'DEPLOY_BLOCKED',
-                issues: [
-                    {
-                        uid: 'component-1',
-                        componentTypeName: 'module',
-                        errorMessages: [],
-                        blockingMessages: [],
-                    },
-                ],
-            };
-            mockDeployProjectV2.mockResolvedValue({
-                data: mockBlockedResponse,
-            });
-            await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
-            expect(mockUiLogger.warn).toHaveBeenCalledWith(commands.project.deploy.errors.deployWarningsHeader);
-            expect(mockUiLogger.log).toHaveBeenCalledWith(commands.project.deploy.errors.deployIssueComponentGeneric('component-1', 'module'));
-        });
-        it('handles general deploy failure', async () => {
-            mockDeployProjectV2.mockResolvedValue({ data: null });
-            const deploy = await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
-            expect(mockUiLogger.error).toHaveBeenCalledWith(commands.project.deploy.errors.deploy);
-            expect(deploy).toBeUndefined();
-        });
-        it('handles undefined deploy response', async () => {
-            mockDeployProjectV2.mockResolvedValue({ data: undefined });
-            const deploy = await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
-            expect(mockUiLogger.error).toHaveBeenCalledWith(commands.project.deploy.errors.deploy);
-            expect(deploy).toBeUndefined();
-        });
-        it('passes correct parameters to deployProjectV1', async () => {
-            const mockDeployResponseData = {
-                id: 'deploy-123',
-                buildResultType: 'DEPLOY_QUEUED',
-                links: {
-                    status: 'http://status-url',
-                },
-            };
-            mockDeployProjectV1.mockResolvedValue({
-                data: mockDeployResponseData,
-            });
-            mockPollDeployStatus.mockResolvedValue({});
-            await handleProjectDeploy(targetAccountId, projectName, buildId, false, // isV2Project
-            true // force
-            );
-            expect(mockDeployProjectV1).toHaveBeenCalledWith(targetAccountId, projectName, buildId, true // force
-            );
-        });
-    });
-});

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

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

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

@@ -1,120 +0,0 @@
-import fs from 'fs-extra';
-import os from 'os';
-import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { downloadProject } from '@hubspot/local-dev-lib/api/projects';
-import { extractZipArchive } from '@hubspot/local-dev-lib/archive';
-import { isDeepEqual } from '@hubspot/local-dev-lib/isDeepEqual';
-import { translate, } from '@hubspot/project-parsing-lib/translate';
-import { isDeployedProjectUpToDateWithLocal } from '../localDev/helpers/project.js';
-// Mock all external dependencies
-vi.mock('@hubspot/local-dev-lib/api/projects');
-vi.mock('@hubspot/local-dev-lib/archive');
-vi.mock('@hubspot/project-parsing-lib/translate');
-vi.mock('@hubspot/local-dev-lib/isDeepEqual');
-vi.mock('fs-extra');
-vi.mock('../../utils/isDeepEqual.js');
-describe('isDeployedProjectUpToDateWithLocal', () => {
-    const mockProjectName = 'test-project';
-    const mockAccountId = 123456;
-    const mockBuildId = 789;
-    const mockProjectConfig = {
-        name: mockProjectName,
-        srcDir: 'src',
-        platformVersion: '1.0.0',
-    };
-    const mockLocalNode = {
-        uid: 'component1',
-        componentType: 'APP',
-        localDev: {
-            componentRoot: '/local/path',
-            componentConfigPath: '/local/path/config.json',
-            configUpdatedSinceLastUpload: false,
-            removed: false,
-            parsingErrors: [],
-        },
-        componentDeps: {},
-        metaFilePath: '/local/path',
-        config: { name: 'Component 1' },
-        files: [],
-    };
-    const mockLocalProjectNodes = {
-        component1: mockLocalNode,
-    };
-    const mockTempDir = '/tmp/test-temp-dir';
-    const mockZippedProject = Buffer.from('fake-zip-data');
-    beforeEach(() => {
-        // Mock fs.mkdtemp
-        fs.mkdtemp.mockResolvedValue(mockTempDir);
-        // Mock fs.pathExists
-        fs.pathExists.mockResolvedValue(true);
-        // Mock fs.remove
-        fs.remove.mockResolvedValue(undefined);
-        // Mock os.tmpdir
-        vi.spyOn(os, 'tmpdir').mockReturnValue('/tmp');
-    });
-    afterEach(() => {
-        vi.restoreAllMocks();
-    });
-    describe('when projects are identical', () => {
-        it('should return true for identical projects', async () => {
-            // Mock downloadProject
-            downloadProject.mockResolvedValue({
-                data: mockZippedProject,
-            });
-            // Mock extractZipArchive
-            extractZipArchive.mockResolvedValue(undefined);
-            // Mock translate to return identical nodes
-            translate.mockResolvedValue({
-                intermediateNodesIndexedByUid: mockLocalProjectNodes,
-            });
-            // Mock isDeepEqual to return true for identical projects
-            isDeepEqual.mockReturnValue(true);
-            const result = await isDeployedProjectUpToDateWithLocal(mockProjectConfig, mockAccountId, mockBuildId, mockLocalProjectNodes);
-            expect(result).toBe(true);
-            expect(isDeepEqual).toHaveBeenCalledWith(mockLocalProjectNodes, mockLocalProjectNodes, ['localDev']);
-            expect(fs.remove).toHaveBeenCalledWith(mockTempDir);
-        });
-    });
-    describe('when projects are different', () => {
-        it('should return false for different projects', async () => {
-            // Mock downloadProject
-            downloadProject.mockResolvedValue({
-                data: mockZippedProject,
-            });
-            // Mock extractZipArchive
-            extractZipArchive.mockResolvedValue(undefined);
-            // Mock translate to return different nodes
-            const differentDeployedNodes = {};
-            translate.mockResolvedValue({
-                intermediateNodesIndexedByUid: differentDeployedNodes,
-            });
-            // Mock isDeepEqual to return false for different projects
-            isDeepEqual.mockReturnValue(false);
-            const result = await isDeployedProjectUpToDateWithLocal(mockProjectConfig, mockAccountId, mockBuildId, mockLocalProjectNodes);
-            expect(result).toBe(false);
-            expect(isDeepEqual).toHaveBeenCalledWith(mockLocalProjectNodes, differentDeployedNodes, ['localDev']);
-        });
-    });
-    describe('error handling', () => {
-        it('should clean up temp directory even when errors occur', async () => {
-            // Mock downloadProject to throw an error after temp dir is created
-            downloadProject.mockRejectedValue(new Error('Download Error'));
-            const result = await isDeployedProjectUpToDateWithLocal(mockProjectConfig, mockAccountId, mockBuildId, mockLocalProjectNodes);
-            expect(result).toBe(false);
-            expect(fs.remove).toHaveBeenCalledWith(mockTempDir);
-        });
-        it('should handle translateForLocalDev errors', async () => {
-            // Mock downloadProject
-            downloadProject.mockResolvedValue({
-                data: mockZippedProject,
-            });
-            // Mock extractZipArchive
-            extractZipArchive.mockResolvedValue(undefined);
-            // Mock translate to throw an error
-            translate.mockRejectedValue(new Error('Translation Error'));
-            const result = await isDeployedProjectUpToDateWithLocal(mockProjectConfig, mockAccountId, mockBuildId, mockLocalProjectNodes);
-            expect(result).toBe(false);
-            expect(fs.remove).toHaveBeenCalledWith(mockTempDir);
-        });
-    });
-});

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

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

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

@@ -1,63 +0,0 @@
-import { isV2Project, isUnsupportedPlatformVersion, } from '../platformVersion.js';
-describe('platformVersion', () => {
-    describe('isV2Project', () => {
-        it('returns true if platform version is UNSTABLE', () => {
-            expect(isV2Project('UNSTABLE')).toBe(true);
-        });
-        it('returns true if platform version is equal to the minimum', () => {
-            expect(isV2Project('2025.2')).toBe(true);
-        });
-        it('returns true if platform version is greater than the minimum', () => {
-            expect(isV2Project('2025.3')).toBe(true);
-            expect(isV2Project('2026.03')).toBe(true);
-            expect(isV2Project('2026.03-beta')).toBe(true);
-        });
-        it('returns false if platform version is less than the minimum', () => {
-            expect(isV2Project('2025.0')).toBe(false);
-            expect(isV2Project('2025.01')).toBe(false);
-            expect(isV2Project('2025.01-beta')).toBe(false);
-        });
-        it('returns false if platform version is invalid', () => {
-            expect(isV2Project(null)).toBe(false);
-        });
-        it('returns false for an invalid platform version', () => {
-            expect(isV2Project('notplaformversion')).toBe(false);
-        });
-    });
-    describe('isUnsupportedPlatformVersion', () => {
-        it('returns false for platform version 2026.03 (boundary)', () => {
-            expect(isUnsupportedPlatformVersion('2026.03')).toBe(false);
-        });
-        it('returns false for platform versions less than 2026.03', () => {
-            expect(isUnsupportedPlatformVersion('2025.2')).toBe(false);
-            expect(isUnsupportedPlatformVersion('2026.01')).toBe(false);
-            expect(isUnsupportedPlatformVersion('2026.02')).toBe(false);
-        });
-        it('returns true for platform version 2026.04', () => {
-            expect(isUnsupportedPlatformVersion('2026.04')).toBe(true);
-        });
-        it('returns true for platform versions greater than 2026.03', () => {
-            expect(isUnsupportedPlatformVersion('2026.4')).toBe(true);
-            expect(isUnsupportedPlatformVersion('2027.01')).toBe(true);
-            expect(isUnsupportedPlatformVersion('2027.1')).toBe(true);
-            expect(isUnsupportedPlatformVersion('2028.10')).toBe(true);
-        });
-        it('returns false for UNSTABLE', () => {
-            expect(isUnsupportedPlatformVersion('UNSTABLE')).toBe(false);
-            expect(isUnsupportedPlatformVersion('unstable')).toBe(false);
-        });
-        it('returns false for null or undefined', () => {
-            expect(isUnsupportedPlatformVersion(null)).toBe(false);
-            expect(isUnsupportedPlatformVersion(undefined)).toBe(false);
-        });
-        it('returns false for invalid platform versions', () => {
-            expect(isUnsupportedPlatformVersion('notaversion')).toBe(false);
-            expect(isUnsupportedPlatformVersion('abc.def')).toBe(false);
-        });
-        it('handles beta versions correctly', () => {
-            expect(isUnsupportedPlatformVersion('2026.03-beta')).toBe(false);
-            expect(isUnsupportedPlatformVersion('2026.04-beta')).toBe(true);
-            expect(isUnsupportedPlatformVersion('2027.01-beta')).toBe(true);
-        });
-    });
-});

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

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