@hubspot/cli 8.2.0 → 8.2.1-experimental.0

package/commands/project/__tests__/deploy.test.js

@@ -1,307 +0,0 @@
-import { HttpStatusCode } from 'axios';
-import yargs from 'yargs';
-import * as configUtils from '@hubspot/local-dev-lib/config';
-import * as projectApiUtils from '@hubspot/local-dev-lib/api/projects';
-import * as ui from '../../../lib/ui/index.js';
-import { addAccountOptions, addConfigOptions, addJSONOutputOptions, addUseEnvironmentOptions, } from '../../../lib/commonOpts.js';
-import * as projectUtils from '../../../lib/projects/config.js';
-import * as projectUrlUtils from '../../../lib/projects/urls.js';
-import { pollDeployStatus } from '../../../lib/projects/pollProjectBuildAndDeploy.js';
-import * as projectNamePrompt from '../../../lib/prompts/projectNamePrompt.js';
-import * as promptUtils from '../../../lib/prompts/promptUtils.js';
-import { trackCommandUsage } from '../../../lib/usageTracking.js';
-import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
-import { loadJson } from '../../../lib/jsonLoader.js';
-import { mockHubSpotHttpResponse, mockHubSpotHttpError, } from '../../../lib/testUtils.js';
-const exampleProject = loadJson(import.meta.url, './fixtures/exampleProject.json');
-import projectDeployCommand from '../deploy.js';
-import { uiLogger } from '../../../lib/ui/logger.js';
-import chalk from 'chalk';
-vi.mock('@hubspot/local-dev-lib/api/projects');
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('../../../lib/commonOpts');
-vi.mock('../../../lib/validation');
-vi.mock('../../../lib/projects/config');
-vi.mock('../../../lib/projects/urls');
-vi.mock('../../../lib/projects/pollProjectBuildAndDeploy');
-vi.mock('../../../lib/projects/platformVersion');
-vi.mock('../../../lib/prompts/projectNamePrompt');
-vi.mock('../../../lib/prompts/promptUtils');
-vi.spyOn(ui, 'uiLine');
-const uiCommandReferenceSpy = vi.spyOn(ui, 'uiCommandReference');
-const uiAccountDescriptionSpy = vi.spyOn(ui, 'uiAccountDescription');
-const getProjectConfigSpy = vi.spyOn(projectUtils, 'getProjectConfig');
-const projectNamePromptSpy = vi.spyOn(projectNamePrompt, 'projectNamePrompt');
-const getProjectDetailUrlSpy = vi.spyOn(projectUrlUtils, 'getProjectDetailUrl');
-const fetchProjectSpy = vi.spyOn(projectApiUtils, 'fetchProject');
-const deployProjectV1Spy = vi.spyOn(projectApiUtils, 'deployProjectV1');
-const getConfigAccountByIdSpy = vi.spyOn(configUtils, 'getConfigAccountById');
-const promptUserSpy = vi.spyOn(promptUtils, 'promptUser');
-const processExitSpy = vi.spyOn(process, 'exit');
-const optionsSpy = vi
-    .spyOn(yargs, 'options')
-    .mockReturnValue(yargs);
-const exampleSpy = vi
-    .spyOn(yargs, 'example')
-    .mockReturnValue(yargs);
-const conflictsSpy = vi
-    .spyOn(yargs, 'conflicts')
-    .mockReturnValue(yargs);
-describe('commands/project/deploy', () => {
-    const projectFlag = 'project';
-    const buildFlag = 'build';
-    const deployLatestBuildFlag = 'deployLatestBuild';
-    const profileFlag = 'profile';
-    const forceFlag = 'force';
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(projectDeployCommand.command).toEqual('deploy');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(projectDeployCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should support the correct options', () => {
-            projectDeployCommand.builder(yargs);
-            expect(conflictsSpy).toHaveBeenCalledTimes(2);
-            expect(conflictsSpy).toHaveBeenNthCalledWith(1, profileFlag, projectFlag);
-            expect(conflictsSpy).toHaveBeenNthCalledWith(2, profileFlag, 'account');
-            expect(optionsSpy).toHaveBeenCalledTimes(1);
-            expect(optionsSpy).toHaveBeenCalledWith({
-                [projectFlag]: expect.objectContaining({ type: 'string' }),
-                [buildFlag]: expect.objectContaining({
-                    alias: ['build-id'],
-                    type: 'number',
-                }),
-                [deployLatestBuildFlag]: expect.objectContaining({
-                    type: 'boolean',
-                    alias: ['deploy-latest-build'],
-                    default: false,
-                }),
-                [profileFlag]: expect.objectContaining({
-                    type: 'string',
-                    alias: ['p'],
-                }),
-                [forceFlag]: expect.objectContaining({
-                    type: 'boolean',
-                    alias: ['f'],
-                }),
-            });
-            expect(addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(addConfigOptions).toHaveBeenCalledWith(yargs);
-            expect(addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(addAccountOptions).toHaveBeenCalledWith(yargs);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargs);
-            expect(addJSONOutputOptions).toHaveBeenCalledTimes(1);
-            expect(addJSONOutputOptions).toHaveBeenCalledWith(yargs);
-        });
-        it('should provide examples', () => {
-            projectDeployCommand.builder(yargs);
-            expect(exampleSpy).toHaveBeenCalledTimes(1);
-        });
-    });
-    describe('handler', () => {
-        let projectConfig;
-        const accountType = 'STANDARD';
-        let args;
-        const projectNameFromPrompt = 'project name from prompt';
-        const deployDetails = {
-            id: 123,
-            buildResultType: 'DEPLOY_QUEUED',
-        };
-        const projectDetailUrl = 'http://project-details-page-url.com';
-        beforeEach(() => {
-            args = {
-                project: 'project name from options',
-                buildId: 2,
-                derivedAccountId: 1234567890,
-            };
-            projectConfig = {
-                name: 'project name from config',
-                srcDir: 'src',
-                platformVersion: '2025',
-            };
-            getProjectConfigSpy.mockResolvedValue({
-                projectConfig,
-                projectDir: 'projectDir',
-            });
-            projectNamePromptSpy.mockResolvedValue({
-                projectName: projectNameFromPrompt,
-            });
-            getProjectDetailUrlSpy.mockReturnValue(projectDetailUrl);
-            getConfigAccountByIdSpy.mockReturnValue({
-                accountId: 123,
-                name: 'SuperCoolTestAccount',
-                accountType,
-                env: 'qa',
-            });
-            fetchProjectSpy.mockReturnValue(mockHubSpotHttpResponse(exampleProject));
-            deployProjectV1Spy.mockReturnValue(mockHubSpotHttpResponse(deployDetails));
-            // Spy on process.exit so our tests don't close when it's called
-            // @ts-expect-error Doesn't match the actual signature because then the linter complains about unused variables
-            processExitSpy.mockImplementation(() => { });
-        });
-        it('should load the account config for the correct account id', async () => {
-            await projectDeployCommand.handler(args);
-            expect(getConfigAccountByIdSpy).toHaveBeenCalledTimes(1);
-            expect(getConfigAccountByIdSpy).toHaveBeenCalledWith(args.derivedAccountId);
-        });
-        it('should track the command usage', async () => {
-            await projectDeployCommand.handler(args);
-            expect(trackCommandUsage).toHaveBeenCalledTimes(1);
-            expect(trackCommandUsage).toHaveBeenCalledWith('project-deploy', { type: accountType }, args.derivedAccountId);
-        });
-        it('should load the project config', async () => {
-            await projectDeployCommand.handler(args);
-            expect(getProjectConfigSpy).toHaveBeenCalledTimes(1);
-            expect(getProjectConfigSpy).toHaveBeenCalled();
-        });
-        it('should prompt for the project name', async () => {
-            await projectDeployCommand.handler(args);
-            expect(projectNamePromptSpy).toHaveBeenCalledTimes(1);
-            expect(projectNamePromptSpy).toHaveBeenCalledWith(args.derivedAccountId, {
-                project: args.project,
-            });
-        });
-        it('should use the project name from the config is a project args is not provided', async () => {
-            delete args.project;
-            await projectDeployCommand.handler(args);
-            expect(projectNamePromptSpy).toHaveBeenCalledTimes(1);
-            expect(projectNamePromptSpy).toHaveBeenCalledWith(args.derivedAccountId, {
-                project: projectConfig.name,
-            });
-        });
-        it('should fetch the project details', async () => {
-            await projectDeployCommand.handler(args);
-            expect(fetchProjectSpy).toHaveBeenCalledTimes(1);
-            expect(fetchProjectSpy).toHaveBeenCalledWith(args.derivedAccountId, projectNameFromPrompt);
-        });
-        it('should use the name from the prompt if no others are defined', async () => {
-            // eslint-disable-next-line @typescript-eslint/no-unused-vars
-            const { project: __project, ...argsWithoutProject } = args;
-            const promptProjectName = 'project name from the prompt';
-            projectNamePromptSpy.mockResolvedValue({
-                projectName: promptProjectName,
-            });
-            getProjectConfigSpy.mockResolvedValue({
-                projectConfig: null,
-                projectDir: null,
-            });
-            await projectDeployCommand.handler(argsWithoutProject);
-            expect(projectNamePromptSpy).toHaveBeenCalledTimes(1);
-            expect(projectNamePromptSpy).toHaveBeenCalledWith(args.derivedAccountId, {});
-            expect(fetchProjectSpy).toHaveBeenCalledTimes(1);
-            expect(fetchProjectSpy).toHaveBeenCalledWith(args.derivedAccountId, promptProjectName);
-        });
-        it('should log an error and exit when latest build is not defined', async () => {
-            fetchProjectSpy.mockReturnValue(mockHubSpotHttpResponse({}));
-            await projectDeployCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledWith('Deploy error: no builds for this project were found.');
-            expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-        it('should log an error and exit when buildId option is not a valid build', async () => {
-            args.buildId = (exampleProject.latestBuild?.buildId ?? 0) + 1;
-            await projectDeployCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringMatching(`Build ${args.buildId} does not exist for project`));
-            expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-        it('should log an error and exit when buildId option is already deployed', async () => {
-            args.buildId = exampleProject.deployedBuildId;
-            await projectDeployCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringMatching(`Build ${args.buildId} is already deployed. View project builds in HubSpot.`));
-            expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-        it('should prompt for build id if no option is provided', async () => {
-            delete args.buildId;
-            promptUserSpy.mockResolvedValue({
-                buildId: exampleProject.latestBuild?.buildId ?? 0,
-            });
-            await projectDeployCommand.handler(args);
-            expect(promptUserSpy).toHaveBeenCalledTimes(1);
-        });
-        it('should log an error and exit if the prompted value is invalid', async () => {
-            delete args.buildId;
-            promptUserSpy.mockResolvedValue({});
-            await projectDeployCommand.handler(args);
-            expect(promptUserSpy).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledWith('You must specify a build to deploy');
-            expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-        it('should deploy the project', async () => {
-            await projectDeployCommand.handler(args);
-            expect(deployProjectV1Spy).toHaveBeenCalledTimes(1);
-            expect(deployProjectV1Spy).toHaveBeenCalledWith(args.derivedAccountId, projectNameFromPrompt, args.buildId, undefined);
-        });
-        it('should log an error and exit when the deploy fails', async () => {
-            // @ts-expect-error Testing an edge case where the response is empty
-            deployProjectV1Spy.mockResolvedValue({});
-            await projectDeployCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledWith(`Deploy error: an unknown error occurred.`);
-            expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-        it('should poll the deploy status', async () => {
-            await projectDeployCommand.handler(args);
-            expect(pollDeployStatus).toHaveBeenCalledTimes(1);
-            expect(pollDeployStatus).toHaveBeenCalledWith(args.derivedAccountId, projectNameFromPrompt, deployDetails.id, args.buildId);
-        });
-        it('log an error and exit if a 404 status is returned', async () => {
-            const commandReference = 'hs project upload';
-            const accountDescription = 'SuperCoolTestAccount';
-            uiCommandReferenceSpy.mockReturnValueOnce(commandReference);
-            uiAccountDescriptionSpy.mockReturnValueOnce(accountDescription);
-            fetchProjectSpy.mockImplementation(() => {
-                throw mockHubSpotHttpError('OH NO', {
-                    status: HttpStatusCode.NotFound,
-                    data: {},
-                });
-            });
-            await projectDeployCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledWith(`The project ${chalk.bold(projectNameFromPrompt)} does not exist in account ${accountDescription}. Run ${commandReference} to upload your project files to HubSpot.`);
-            expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-        it('log an error and exit if a 400 status is returned', async () => {
-            const errorMessage = 'Something bad happened';
-            fetchProjectSpy.mockImplementation(() => {
-                throw mockHubSpotHttpError(errorMessage, {
-                    status: HttpStatusCode.BadRequest,
-                    data: {},
-                });
-            });
-            await projectDeployCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledWith('The request was bad.');
-            expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-        it('log an error another unexpected status code is returned', async () => {
-            const errorMessage = 'Something bad happened';
-            fetchProjectSpy.mockImplementation(() => {
-                throw mockHubSpotHttpError(errorMessage, {
-                    status: HttpStatusCode.MethodNotAllowed,
-                    data: {},
-                });
-            });
-            await projectDeployCommand.handler(args);
-            expect(uiLogger.error).toHaveBeenCalledTimes(1);
-            expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining(`The request for 'project deploy' in account ${args.derivedAccountId} failed due to a client error.`));
-            expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-    });
-});

package/commands/project/__tests__/dev.test.d.ts

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

package/commands/project/__tests__/dev.test.js

@@ -1,273 +0,0 @@
-import * as configLib from '@hubspot/local-dev-lib/config';
-import * as projectConfigLib from '../../../lib/projects/config.js';
-import * as platformVersionLib from '../../../lib/projects/platformVersion.js';
-import * as projectProfilesLib from '../../../lib/projects/projectProfiles.js';
-import * as projectParsingProfiles from '@hubspot/project-parsing-lib/profiles';
-import * as promptUtilsLib from '../../../lib/prompts/promptUtils.js';
-import * as usageTrackingLib from '../../../lib/usageTracking.js';
-import * as errorHandlers from '../../../lib/errorHandlers/index.js';
-import { uiLogger } from '../../../lib/ui/logger.js';
-import { EXIT_CODES } from '../../../lib/enums/exitCodes.js';
-import * as deprecatedFlowLib from '../dev/deprecatedFlow.js';
-import * as unifiedFlowLib from '../dev/unifiedFlow.js';
-import projectDevCommand from '../dev/index.js';
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('@hubspot/project-parsing-lib/profiles');
-vi.mock('../../../lib/projects/config.js');
-vi.mock('../../../lib/projects/platformVersion.js');
-vi.mock('../../../lib/projects/projectProfiles.js');
-vi.mock('../../../lib/prompts/promptUtils.js');
-vi.mock('../../../lib/errorHandlers/index.js');
-vi.mock('../../../lib/ui/index.js');
-vi.mock('../dev/deprecatedFlow.js');
-vi.mock('../dev/unifiedFlow.js');
-const getConfigAccountIfExistsSpy = vi.spyOn(configLib, 'getConfigAccountIfExists');
-const getProjectConfigSpy = vi.spyOn(projectConfigLib, 'getProjectConfig');
-const validateProjectConfigSpy = vi.spyOn(projectConfigLib, 'validateProjectConfig');
-const isV2ProjectSpy = vi.spyOn(platformVersionLib, 'isV2Project');
-const loadProfileSpy = vi.spyOn(projectProfilesLib, 'loadProfile');
-const enforceProfileUsageSpy = vi.spyOn(projectProfilesLib, 'enforceProfileUsage');
-const getAllHsProfilesSpy = vi.spyOn(projectParsingProfiles, 'getAllHsProfiles');
-const listPromptSpy = vi.spyOn(promptUtilsLib, 'listPrompt');
-const trackCommandUsageSpy = vi.spyOn(usageTrackingLib, 'trackCommandUsage');
-const logErrorSpy = vi.spyOn(errorHandlers, 'logError');
-const deprecatedProjectDevFlowSpy = vi.spyOn(deprecatedFlowLib, 'deprecatedProjectDevFlow');
-const unifiedProjectDevFlowSpy = vi.spyOn(unifiedFlowLib, 'unifiedProjectDevFlow');
-const processExitSpy = vi.spyOn(process, 'exit');
-describe('commands/project/dev', () => {
-    beforeEach(() => {
-        // @ts-expect-error Mock implementation
-        processExitSpy.mockImplementation(() => { });
-        getProjectConfigSpy.mockResolvedValue({
-            projectConfig: {
-                name: 'test-project',
-                srcDir: 'src',
-                platformVersion: 'v2',
-            },
-            projectDir: '/test/project',
-        });
-        validateProjectConfigSpy.mockImplementation(() => { });
-        isV2ProjectSpy.mockReturnValue(true);
-        trackCommandUsageSpy.mockImplementation(async () => { });
-        deprecatedProjectDevFlowSpy.mockResolvedValue(undefined);
-        unifiedProjectDevFlowSpy.mockResolvedValue(undefined);
-        enforceProfileUsageSpy.mockResolvedValue(undefined);
-        getAllHsProfilesSpy.mockResolvedValue([]);
-        listPromptSpy.mockResolvedValue('dev');
-    });
-    describe('command', () => {
-        it('should have the correct command structure', () => {
-            expect(projectDevCommand.command).toEqual('dev');
-        });
-    });
-    describe('describe', () => {
-        it('should provide a description', () => {
-            expect(projectDevCommand.describe).toBeDefined();
-        });
-    });
-    describe('builder', () => {
-        it('should be a function', () => {
-            expect(typeof projectDevCommand.builder).toBe('function');
-        });
-    });
-    describe('handler', () => {
-        let args;
-        beforeEach(() => {
-            args = {
-                derivedAccountId: 123456,
-            };
-        });
-        describe('validation', () => {
-            it('should validate project config', async () => {
-                await projectDevCommand.handler(args);
-                expect(validateProjectConfigSpy).toHaveBeenCalledWith({
-                    name: 'test-project',
-                    srcDir: 'src',
-                    platformVersion: 'v2',
-                }, '/test/project');
-            });
-            it('should exit if project config validation fails', async () => {
-                const error = new Error('Invalid config');
-                validateProjectConfigSpy.mockImplementation(() => {
-                    throw error;
-                });
-                await projectDevCommand.handler(args);
-                expect(logErrorSpy).toHaveBeenCalledWith(error);
-                expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-            });
-            it('should exit if no project directory', async () => {
-                getProjectConfigSpy.mockResolvedValue({
-                    projectConfig: {
-                        name: 'test-project',
-                        srcDir: 'src',
-                        platformVersion: 'v2',
-                    },
-                    projectDir: null,
-                });
-                // Make process.exit actually throw to stop execution
-                processExitSpy.mockImplementation((code) => {
-                    throw new Error(`process.exit called with ${code}`);
-                });
-                await expect(projectDevCommand.handler(args)).rejects.toThrow('process.exit called');
-                expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('project'));
-                expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-            });
-            it('should error if using testingAccount and projectAccount with legacy project', async () => {
-                isV2ProjectSpy.mockReturnValue(false);
-                args.testingAccount = '111111';
-                args.projectAccount = '222222';
-                await projectDevCommand.handler(args);
-                expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('--project-account and --testing-account'));
-                expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-            });
-            it('should error if using account flag with V2 project', async () => {
-                isV2ProjectSpy.mockReturnValue(true);
-                args.userProvidedAccount = '123456';
-                await projectDevCommand.handler(args);
-                expect(uiLogger.error).toHaveBeenCalledWith(expect.stringContaining('--account flag'));
-                expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-            });
-        });
-        describe('account resolution', () => {
-            it('should use projectAccount flag', async () => {
-                args.projectAccount = '999999';
-                getConfigAccountIfExistsSpy.mockReturnValue({
-                    accountId: 999999,
-                });
-                await projectDevCommand.handler(args);
-                expect(getConfigAccountIfExistsSpy).toHaveBeenCalledWith('999999');
-                expect(trackCommandUsageSpy).toHaveBeenCalledWith('project-dev', {}, 999999);
-            });
-            it('should use userProvidedAccount for legacy projects', async () => {
-                isV2ProjectSpy.mockReturnValue(false);
-                args.userProvidedAccount = '888888';
-                args.derivedAccountId = 888888;
-                await projectDevCommand.handler(args);
-                expect(trackCommandUsageSpy).toHaveBeenCalledWith('project-dev', {}, 888888);
-            });
-            it('should use profile accountId when profile specified', async () => {
-                args.profile = 'test-profile';
-                loadProfileSpy.mockReturnValue({
-                    accountId: 777777,
-                });
-                await projectDevCommand.handler(args);
-                expect(loadProfileSpy).toHaveBeenCalledWith({
-                    name: 'test-project',
-                    srcDir: 'src',
-                    platformVersion: 'v2',
-                }, '/test/project', 'test-profile');
-                expect(trackCommandUsageSpy).toHaveBeenCalledWith('project-dev', {}, 777777);
-            });
-            it('should exit if profile loading fails', async () => {
-                args.profile = 'invalid-profile';
-                const error = new Error('Profile not found');
-                loadProfileSpy.mockImplementation(() => {
-                    throw error;
-                });
-                // Make process.exit actually throw to stop execution
-                processExitSpy.mockImplementation((code) => {
-                    throw new Error(`process.exit called with ${code}`);
-                });
-                await expect(projectDevCommand.handler(args)).rejects.toThrow('process.exit called');
-                expect(logErrorSpy).toHaveBeenCalledWith(error);
-                expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-            });
-            it('should prompt for profile selection when profiles exist and no profile is specified', async () => {
-                getAllHsProfilesSpy.mockResolvedValue(['dev', 'prod']);
-                listPromptSpy.mockResolvedValue('dev');
-                loadProfileSpy.mockReturnValue({
-                    accountId: 789012,
-                    variables: {},
-                });
-                await projectDevCommand.handler(args);
-                expect(getAllHsProfilesSpy).toHaveBeenCalledWith('/test/project/src');
-                expect(listPromptSpy).toHaveBeenCalledWith(expect.any(String), {
-                    choices: ['dev', 'prod'],
-                });
-                expect(loadProfileSpy).toHaveBeenCalledWith({
-                    name: 'test-project',
-                    srcDir: 'src',
-                    platformVersion: 'v2',
-                }, '/test/project', 'dev');
-                expect(trackCommandUsageSpy).toHaveBeenCalledWith('project-dev', {}, 789012);
-            });
-            it('should exit if profile loading fails after selection', async () => {
-                getAllHsProfilesSpy.mockResolvedValue(['dev', 'prod']);
-                listPromptSpy.mockResolvedValue('dev');
-                const error = new Error('Failed to load profile');
-                loadProfileSpy.mockImplementation(() => {
-                    throw error;
-                });
-                // Make process.exit actually throw to stop execution
-                processExitSpy.mockImplementation((code) => {
-                    throw new Error(`process.exit called with ${code}`);
-                });
-                await expect(projectDevCommand.handler(args)).rejects.toThrow('process.exit called');
-                expect(logErrorSpy).toHaveBeenCalledWith(error);
-                expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-            });
-            it('should use derivedAccountId as fallback', async () => {
-                await projectDevCommand.handler(args);
-                expect(trackCommandUsageSpy).toHaveBeenCalledWith('project-dev', {}, 123456);
-            });
-        });
-        describe('dev flow execution', () => {
-            it('should track command usage', async () => {
-                await projectDevCommand.handler(args);
-                expect(trackCommandUsageSpy).toHaveBeenCalledWith('project-dev', {}, 123456);
-            });
-            it('should run unified flow for V2 projects', async () => {
-                isV2ProjectSpy.mockReturnValue(true);
-                await projectDevCommand.handler(args);
-                expect(unifiedProjectDevFlowSpy).toHaveBeenCalledWith({
-                    args,
-                    targetProjectAccountId: 123456,
-                    providedTargetTestingAccountId: undefined,
-                    projectConfig: {
-                        name: 'test-project',
-                        srcDir: 'src',
-                        platformVersion: 'v2',
-                    },
-                    projectDir: '/test/project',
-                    profileConfig: undefined,
-                });
-                expect(deprecatedProjectDevFlowSpy).not.toHaveBeenCalled();
-            });
-            it('should run deprecated flow for legacy projects', async () => {
-                isV2ProjectSpy.mockReturnValue(false);
-                await projectDevCommand.handler(args);
-                expect(deprecatedProjectDevFlowSpy).toHaveBeenCalledWith({
-                    args,
-                    accountId: 123456,
-                    projectConfig: {
-                        name: 'test-project',
-                        srcDir: 'src',
-                        platformVersion: 'v2',
-                    },
-                    projectDir: '/test/project',
-                });
-                expect(unifiedProjectDevFlowSpy).not.toHaveBeenCalled();
-            });
-            it('should pass testingAccount to unified flow', async () => {
-                isV2ProjectSpy.mockReturnValue(true);
-                args.testingAccount = '555555';
-                getConfigAccountIfExistsSpy.mockReturnValue({
-                    accountId: 555555,
-                });
-                await projectDevCommand.handler(args);
-                expect(unifiedProjectDevFlowSpy).toHaveBeenCalledWith(expect.objectContaining({
-                    providedTargetTestingAccountId: 555555,
-                }));
-            });
-            it('should pass profile config to unified flow', async () => {
-                args.profile = 'dev-profile';
-                const profileConfig = { accountId: 666666 };
-                loadProfileSpy.mockReturnValue(profileConfig);
-                await projectDevCommand.handler(args);
-                expect(unifiedProjectDevFlowSpy).toHaveBeenCalledWith(expect.objectContaining({
-                    profileConfig,
-                }));
-            });
-        });
-    });
-});

package/commands/project/__tests__/devUnifiedFlow.test.d.ts

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