@hubspot/cli 7.7.19-experimental.2 → 7.7.20-experimental.0

package/api/__tests__/migrate.test.d.ts

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

package/api/__tests__/migrate.test.js

@@ -0,0 +1,183 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const http_1 = require("@hubspot/local-dev-lib/http");
+const Migration_1 = require("@hubspot/local-dev-lib/types/Migration");
+const migrate_1 = require("../migrate");
+vi.mock('@hubspot/local-dev-lib/http');
+const httpMock = http_1.http;
+describe('api/migrate', () => {
+    const mockAccountId = 12345;
+    const mockPortalId = 12345;
+    const mockAppId = 67890;
+    const mockMigrationId = 54321;
+    const mockPlatformVersion = '2025.2';
+    const convertedPlatformVersion = 'V2025_2';
+    const mockProjectName = 'Test Project';
+    const mockComponentUids = { 'component-1': 'uid-1', 'component-2': 'uid-2' };
+    describe('listAppsForMigration', () => {
+        it('should call http.get with correct parameters', async () => {
+            const mockResponse = {
+                migratableApps: [
+                    {
+                        appId: 1,
+                        appName: 'App 1',
+                        isMigratable: true,
+                        migrationComponents: [
+                            { id: 'comp1', componentType: 'type1', isSupported: true },
+                        ],
+                    },
+                ],
+                unmigratableApps: [
+                    {
+                        appId: 2,
+                        appName: 'App 2',
+                        isMigratable: false,
+                        unmigratableReason: 'UP_TO_DATE',
+                        migrationComponents: [
+                            { id: 'comp2', componentType: 'type2', isSupported: false },
+                        ],
+                    },
+                ],
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await (0, migrate_1.listAppsForMigration)(mockAccountId, mockPlatformVersion);
+            expect(http_1.http.get).toHaveBeenCalledWith(mockAccountId, {
+                url: 'dfs/migrations/v2/list-apps',
+                params: {
+                    platformVersion: convertedPlatformVersion,
+                },
+            });
+            expect(result).toEqual(mockResponse);
+        });
+    });
+    describe('initializeMigration', () => {
+        it('should call http.post with correct parameters', async () => {
+            const mockResponse = { migrationId: mockMigrationId };
+            // @ts-expect-error Mock
+            httpMock.post.mockResolvedValue(mockResponse);
+            const result = await (0, migrate_1.initializeMigration)(mockAccountId, mockAppId, mockPlatformVersion);
+            expect(http_1.http.post).toHaveBeenCalledWith(mockAccountId, {
+                url: 'dfs/migrations/v2/migrations',
+                data: {
+                    applicationId: mockAppId,
+                    platformVersion: convertedPlatformVersion,
+                },
+            });
+            expect(result).toEqual(mockResponse);
+        });
+    });
+    describe('continueMigration', () => {
+        it('should call http.post with correct parameters', async () => {
+            const mockResponse = { migrationId: mockMigrationId };
+            // @ts-expect-error Mock
+            httpMock.post.mockResolvedValue(mockResponse);
+            const result = await (0, migrate_1.continueMigration)(mockPortalId, mockMigrationId, mockComponentUids, mockProjectName);
+            expect(http_1.http.post).toHaveBeenCalledWith(mockPortalId, {
+                url: 'dfs/migrations/v2/migrations/continue',
+                data: {
+                    migrationId: mockMigrationId,
+                    projectName: mockProjectName,
+                    componentUids: mockComponentUids,
+                },
+            });
+            expect(result).toEqual(mockResponse);
+        });
+    });
+    describe('checkMigrationStatusV2', () => {
+        it('should call http.get with correct parameters for in-progress status', async () => {
+            const mockResponse = {
+                id: mockMigrationId,
+                status: Migration_1.MIGRATION_STATUS.IN_PROGRESS,
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await (0, migrate_1.checkMigrationStatusV2)(mockAccountId, mockMigrationId);
+            expect(http_1.http.get).toHaveBeenCalledWith(mockAccountId, {
+                url: `dfs/migrations/v2/migrations/${mockMigrationId}/status`,
+            });
+            expect(result).toEqual(mockResponse);
+        });
+        it('should handle input required status', async () => {
+            const mockResponse = {
+                id: mockMigrationId,
+                status: Migration_1.MIGRATION_STATUS.INPUT_REQUIRED,
+                componentsRequiringUids: {
+                    'component-1': {
+                        componentType: 'type1',
+                        componentHint: 'hint1',
+                    },
+                },
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await (0, migrate_1.checkMigrationStatusV2)(mockAccountId, mockMigrationId);
+            expect(result).toEqual(mockResponse);
+        });
+        it('should handle success status', async () => {
+            const mockResponse = {
+                id: mockMigrationId,
+                status: Migration_1.MIGRATION_STATUS.SUCCESS,
+                buildId: 98765,
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await (0, migrate_1.checkMigrationStatusV2)(mockAccountId, mockMigrationId);
+            expect(result).toEqual(mockResponse);
+        });
+        it('should handle failure status', async () => {
+            const mockResponse = {
+                id: mockMigrationId,
+                status: Migration_1.MIGRATION_STATUS.FAILURE,
+                projectErrorDetail: 'Error details',
+                componentErrors: [],
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await (0, migrate_1.checkMigrationStatusV2)(mockAccountId, mockMigrationId);
+            expect(result).toEqual(mockResponse);
+        });
+    });
+});
+describe('isMigrationStatus', () => {
+    it.each([
+        {
+            id: 123,
+            status: Migration_1.MIGRATION_STATUS.IN_PROGRESS,
+        },
+        {
+            id: 456,
+            status: Migration_1.MIGRATION_STATUS.INPUT_REQUIRED,
+            componentsRequiringUids: {
+                'component-1': {
+                    componentType: 'type1',
+                    componentHint: 'hint1',
+                },
+            },
+        },
+        {
+            id: 789,
+            status: Migration_1.MIGRATION_STATUS.SUCCESS,
+            buildId: 98765,
+        },
+        {
+            id: 101,
+            status: Migration_1.MIGRATION_STATUS.FAILURE,
+            projectErrorDetail: 'Error details',
+            componentErrors: [],
+        },
+    ])('should return true for valid MigrationStatus object %j', status => {
+        expect((0, migrate_1.isMigrationStatus)(status)).toBe(true);
+    });
+    it.each([null, undefined, 123, 'string', true, false, []])('should return false for non-object value %j', value => {
+        expect((0, migrate_1.isMigrationStatus)(value)).toBe(false);
+    });
+    it.each([
+        {},
+        { id: 123 },
+        { status: Migration_1.MIGRATION_STATUS.IN_PROGRESS },
+        { foo: 'bar' },
+    ])('should return false for invalid object %j', obj => {
+        expect((0, migrate_1.isMigrationStatus)(obj)).toBe(false);
+    });
+});

package/commands/__tests__/account.test.d.ts

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

package/commands/__tests__/account.test.js

@@ -0,0 +1,74 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const auth_1 = __importDefault(require("../account/auth"));
+const list_1 = __importDefault(require("../account/list"));
+const rename_1 = __importDefault(require("../account/rename"));
+const use_1 = __importDefault(require("../account/use"));
+const info_1 = __importDefault(require("../account/info"));
+const remove_1 = __importDefault(require("../account/remove"));
+const clean_1 = __importDefault(require("../account/clean"));
+const createOverride_1 = __importDefault(require("../account/createOverride"));
+const removeOverride_1 = __importDefault(require("../account/removeOverride"));
+const account_1 = __importDefault(require("../account"));
+vi.mock('../account/auth');
+vi.mock('../account/list');
+vi.mock('../account/rename');
+vi.mock('../account/use');
+vi.mock('../account/info');
+vi.mock('../account/remove');
+vi.mock('../account/clean');
+vi.mock('../account/createOverride');
+vi.mock('../account/removeOverride');
+vi.mock('../../lib/commonOpts');
+const commandSpy = vi
+    .spyOn(yargs_1.default, 'command')
+    .mockReturnValue(yargs_1.default);
+const demandCommandSpy = vi
+    .spyOn(yargs_1.default, 'demandCommand')
+    .mockReturnValue(yargs_1.default);
+describe('commands/account', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(account_1.default.command).toEqual(['account', 'accounts']);
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(account_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        beforeEach(() => {
+            commandSpy.mockClear();
+            demandCommandSpy.mockClear();
+        });
+        const subcommands = [
+            auth_1.default,
+            list_1.default,
+            rename_1.default,
+            use_1.default,
+            info_1.default,
+            remove_1.default,
+            clean_1.default,
+            createOverride_1.default,
+            removeOverride_1.default,
+        ];
+        it('should demand the command takes one positional argument', () => {
+            account_1.default.builder(yargs_1.default);
+            expect(demandCommandSpy).toHaveBeenCalledTimes(1);
+            expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
+        });
+        it('should add the correct number of sub commands', () => {
+            account_1.default.builder(yargs_1.default);
+            expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
+        });
+        it.each(subcommands)('should attach the %s subcommand', module => {
+            account_1.default.builder(yargs_1.default);
+            expect(commandSpy).toHaveBeenCalledWith(module);
+        });
+    });
+});

package/commands/__tests__/auth.test.d.ts

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

package/commands/__tests__/auth.test.js

@@ -0,0 +1,43 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const commonOpts_1 = require("../../lib/commonOpts");
+const auth_1 = __importDefault(require("../auth"));
+vi.mock('../../lib/commonOpts');
+const optionsSpy = vi
+    .spyOn(mockYargs, 'options')
+    .mockReturnValue(yargs_1.default);
+describe('commands/auth', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(auth_1.default.command).toEqual('auth');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(auth_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            auth_1.default.builder(yargs_1.default);
+            expect(optionsSpy).toHaveBeenCalledTimes(1);
+            expect(optionsSpy).toHaveBeenCalledWith({
+                'auth-type': expect.objectContaining({
+                    type: 'string',
+                    choices: ['personalaccesskey', 'oauth2'],
+                    default: 'personalaccesskey',
+                }),
+                account: expect.objectContaining({ type: 'string' }),
+                'personal-access-key': expect.objectContaining({ type: 'string' }),
+            });
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargs_1.default);
+            expect(commonOpts_1.addTestingOptions).toHaveBeenCalledTimes(1);
+            expect(commonOpts_1.addTestingOptions).toHaveBeenCalledWith(yargs_1.default);
+        });
+    });
+});

package/commands/__tests__/cms.test.d.ts

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

package/commands/__tests__/cms.test.js

@@ -0,0 +1,49 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const lighthouseScore_1 = __importDefault(require("../cms/lighthouseScore"));
+const convertFields_1 = __importDefault(require("../cms/convertFields"));
+const getReactModule_1 = __importDefault(require("../cms/getReactModule"));
+const cms_1 = __importDefault(require("../cms"));
+vi.mock('../cms/lighthouseScore');
+vi.mock('../cms/convertFields');
+vi.mock('../cms/getReactModule');
+vi.mock('../../lib/commonOpts');
+const commandSpy = vi
+    .spyOn(yargs_1.default, 'command')
+    .mockReturnValue(yargs_1.default);
+const demandCommandSpy = vi
+    .spyOn(yargs_1.default, 'demandCommand')
+    .mockReturnValue(yargs_1.default);
+describe('commands/cms', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(cms_1.default.command).toEqual('cms');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(cms_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        const subcommands = [lighthouseScore_1.default, convertFields_1.default, getReactModule_1.default];
+        it('should demand the command takes one positional argument', () => {
+            cms_1.default.builder(yargs_1.default);
+            expect(demandCommandSpy).toHaveBeenCalledTimes(1);
+            expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
+        });
+        it('should add the correct number of sub commands', () => {
+            cms_1.default.builder(yargs_1.default);
+            expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
+        });
+        it.each(subcommands)('should attach the %s subcommand', module => {
+            cms_1.default.builder(yargs_1.default);
+            expect(module).toBeDefined();
+            expect(commandSpy).toHaveBeenCalledWith(module);
+        });
+    });
+});

package/commands/__tests__/config.test.d.ts

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

package/commands/__tests__/config.test.js

@@ -0,0 +1,49 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const set_1 = __importDefault(require("../config/set"));
+const migrate_1 = __importDefault(require("../config/migrate"));
+const config_1 = __importDefault(require("../config"));
+vi.mock('../config/set');
+vi.mock('../config/set');
+const commandSpy = vi
+    .spyOn(yargs_1.default, 'command')
+    .mockReturnValue(yargs_1.default);
+const demandCommandSpy = vi
+    .spyOn(yargs_1.default, 'demandCommand')
+    .mockReturnValue(yargs_1.default);
+describe('commands/config', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(config_1.default.command).toEqual('config');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(config_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        beforeEach(() => {
+            commandSpy.mockClear();
+            demandCommandSpy.mockClear();
+        });
+        const subcommands = [set_1.default, migrate_1.default];
+        it('should demand the command takes one positional argument', () => {
+            config_1.default.builder(yargs_1.default);
+            expect(demandCommandSpy).toHaveBeenCalledTimes(1);
+            expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
+        });
+        it('should add the correct number of sub commands', () => {
+            config_1.default.builder(yargs_1.default);
+            expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
+        });
+        it.each(subcommands)('should attach the %s subcommand', module => {
+            config_1.default.builder(yargs_1.default);
+            expect(commandSpy).toHaveBeenCalledWith(module);
+        });
+    });
+});

package/commands/__tests__/create.test.d.ts

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

package/commands/__tests__/create.test.js

@@ -0,0 +1,38 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const create_1 = __importDefault(require("../create"));
+const positionalSpy = vi
+    .spyOn(yargs_1.default, 'positional')
+    .mockReturnValue(yargs_1.default);
+const optionSpy = vi
+    .spyOn(yargs_1.default, 'option')
+    .mockReturnValue(yargs_1.default);
+describe('commands/create', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(create_1.default.command).toEqual('create <type> [name] [dest]');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(create_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct positional arguments', () => {
+            create_1.default.builder(yargs_1.default);
+            expect(positionalSpy).toHaveBeenCalledTimes(3);
+            expect(positionalSpy).toHaveBeenCalledWith('type', expect.objectContaining({ type: 'string' }));
+            expect(positionalSpy).toHaveBeenCalledWith('name', expect.objectContaining({ type: 'string' }));
+            expect(positionalSpy).toHaveBeenCalledWith('dest', expect.objectContaining({ type: 'string' }));
+        });
+        it('should support the correct options', () => {
+            create_1.default.builder(yargs_1.default);
+            expect(optionSpy).toHaveBeenCalledWith('internal', expect.objectContaining({ type: 'boolean', hidden: true }));
+        });
+    });
+});

package/commands/__tests__/customObject.test.d.ts

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

package/commands/__tests__/customObject.test.js

@@ -0,0 +1,54 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const yargs_1 = __importDefault(require("yargs"));
+const create_1 = __importDefault(require("../customObject/create"));
+const schema_1 = __importDefault(require("../customObject/schema"));
+const customObject_1 = __importDefault(require("../customObject"));
+vi.mock('../customObject/create');
+vi.mock('../customObject/schema');
+vi.mock('../../lib/commonOpts');
+const commandSpy = vi
+    .spyOn(yargs_1.default, 'command')
+    .mockReturnValue(yargs_1.default);
+const demandCommandSpy = vi
+    .spyOn(yargs_1.default, 'demandCommand')
+    .mockReturnValue(yargs_1.default);
+describe('commands/customObject', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(customObject_1.default.command).toEqual([
+                'custom-object',
+                'custom-objects',
+                'co',
+            ]);
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(customObject_1.default.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        beforeEach(() => {
+            commandSpy.mockClear();
+            demandCommandSpy.mockClear();
+        });
+        const subcommands = [create_1.default, schema_1.default];
+        it('should demand the command takes one positional argument', () => {
+            customObject_1.default.builder(yargs_1.default);
+            expect(demandCommandSpy).toHaveBeenCalledTimes(1);
+            expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
+        });
+        it('should add the correct number of sub commands', () => {
+            customObject_1.default.builder(yargs_1.default);
+            expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
+        });
+        it.each(subcommands)('should attach the %s subcommand', module => {
+            customObject_1.default.builder(yargs_1.default);
+            expect(commandSpy).toHaveBeenCalledWith(module);
+        });
+    });
+});

package/commands/__tests__/doctor.test.d.ts

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

package/commands/__tests__/doctor.test.js

@@ -0,0 +1,139 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const exitCodes_1 = require("../../lib/enums/exitCodes");
+const yargs_1 = __importDefault(require("yargs"));
+const doctor_1 = __importDefault(require("../doctor"));
+const usageTracking_1 = require("../../lib/usageTracking");
+const Doctor_1 = require("../../lib/doctor/Doctor");
+const logger_1 = require("@hubspot/local-dev-lib/logger");
+const fs_1 = __importDefault(require("fs"));
+const path_1 = require("@hubspot/local-dev-lib/path");
+vi.mock('../../lib/usageTracking');
+vi.mock('../../lib/doctor/Doctor');
+vi.mock('@hubspot/local-dev-lib/logger');
+vi.mock('@hubspot/local-dev-lib/path');
+vi.mock('fs');
+// @ts-expect-error Doesn't match the actual signature because then the linter complains about unused variables
+const DoctorMock = Doctor_1.Doctor;
+const mockedFs = vi.mocked(fs_1.default);
+const getCwd = path_1.getCwd;
+const optionSpy = vi
+    .spyOn(yargs_1.default, 'option')
+    .mockReturnValue(yargs_1.default);
+const date = new Date('2022-02-22');
+vi.useFakeTimers().setSystemTime(date);
+describe('doctor', () => {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    let processExitSpy;
+    const accountId = 123456;
+    beforeEach(() => {
+        // @ts-expect-error Doesn't match the actual signature because then the linter complains about unused variables
+        processExitSpy = vi.spyOn(process, 'exit').mockImplementation(() => { });
+    });
+    describe('command', () => {
+        it('should have the proper command name', () => {
+            expect(doctor_1.default.command).toEqual('doctor');
+        });
+    });
+    describe('describe', () => {
+        it('should have a description', () => {
+            expect(doctor_1.default.describe).toEqual('Retrieve diagnostic information about your local HubSpot configurations.');
+        });
+    });
+    describe('builder', () => {
+        it('should apply the correct options', () => {
+            doctor_1.default.builder(yargs_1.default);
+            expect(optionSpy).toHaveBeenCalledWith('output-dir', {
+                describe: 'Directory to save a detailed diagnosis JSON file in',
+                type: 'string',
+            });
+        });
+    });
+    describe('handler', () => {
+        let diagnosis;
+        beforeEach(() => {
+            diagnosis = 'Yooooooooooooooo';
+            DoctorMock.mockImplementation(() => {
+                return {
+                    diagnose: vi.fn().mockResolvedValue({ diagnosis }),
+                    accountId,
+                };
+            });
+        });
+        it('should track the command usage', async () => {
+            await doctor_1.default.handler({});
+            expect(usageTracking_1.trackCommandUsage).toHaveBeenCalledTimes(1);
+            expect(usageTracking_1.trackCommandUsage).toHaveBeenCalledWith('doctor', undefined, accountId);
+        });
+        it('should log the diagnosis if it is defined', async () => {
+            await doctor_1.default.handler({});
+            expect(logger_1.logger.log).toHaveBeenCalledTimes(1);
+            expect(logger_1.logger.log).toHaveBeenCalledWith(diagnosis);
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.SUCCESS);
+        });
+        it('should log an error if the diagnosis is undefined', async () => {
+            DoctorMock.mockImplementationOnce(() => {
+                return {
+                    diagnose: vi.fn().mockResolvedValue(undefined),
+                    accountId,
+                };
+            });
+            await doctor_1.default.handler({});
+            expect(logger_1.logger.error).toHaveBeenCalledTimes(1);
+            expect(logger_1.logger.error).toHaveBeenCalledWith('Error generating diagnosis');
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.ERROR);
+        });
+        it('should write the output to a file if output-dir is defined', async () => {
+            mockedFs.writeFileSync.mockImplementationOnce(() => { });
+            const expectedOutputFile = `/foo/hubspot-doctor-${date.toISOString()}.json`;
+            await doctor_1.default.handler({
+                outputDir: '/foo',
+            });
+            expect(logger_1.logger.log).not.toHaveBeenCalled();
+            expect(logger_1.logger.error).not.toHaveBeenCalled();
+            expect(mockedFs.writeFileSync).toHaveBeenCalledTimes(1);
+            expect(mockedFs.writeFileSync).toHaveBeenCalledWith(expectedOutputFile, expect.stringContaining(diagnosis));
+            expect(logger_1.logger.success).toHaveBeenCalledTimes(1);
+            expect(logger_1.logger.success).toHaveBeenCalledWith(expect.stringMatching(/Output written to /));
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.SUCCESS);
+        });
+        it('should handle absolute paths', async () => {
+            mockedFs.writeFileSync.mockImplementationOnce(() => { });
+            const cwd = '/some/path/to';
+            getCwd.mockImplementationOnce(() => cwd);
+            const expectedOutputFile = `${cwd}/foo/hubspot-doctor-${date.toISOString()}.json`;
+            await doctor_1.default.handler({
+                outputDir: './foo',
+            });
+            expect(logger_1.logger.log).not.toHaveBeenCalled();
+            expect(logger_1.logger.error).not.toHaveBeenCalled();
+            expect(mockedFs.writeFileSync).toHaveBeenCalledTimes(1);
+            expect(mockedFs.writeFileSync).toHaveBeenCalledWith(expectedOutputFile, expect.stringContaining(diagnosis));
+            expect(logger_1.logger.success).toHaveBeenCalledTimes(1);
+            expect(logger_1.logger.success).toHaveBeenCalledWith(expect.stringMatching(/Output written to /));
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.SUCCESS);
+        });
+        it('should log an error message when writing the output fails', async () => {
+            const errorMessage = 'Something bad happened';
+            mockedFs.writeFileSync.mockImplementationOnce(() => {
+                throw new Error(errorMessage);
+            });
+            await doctor_1.default.handler({
+                outputDir: '/foo',
+            });
+            expect(logger_1.logger.error).toHaveBeenCalledTimes(1);
+            expect(logger_1.logger.error).toHaveBeenCalledWith(expect.stringMatching(/Unable to write output to/));
+            expect(mockedFs.writeFileSync).toHaveBeenCalledTimes(1);
+            expect(mockedFs.writeFileSync).toHaveBeenCalledWith(`/foo/hubspot-doctor-${date.toISOString()}.json`, expect.stringContaining(diagnosis));
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.ERROR);
+        });
+    });
+});

package/commands/__tests__/feedback.test.d.ts

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