@hubspot/cli 8.0.10-experimental.7 → 8.0.11-experimental.0

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

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

package/api/__tests__/migrate.test.js

@@ -0,0 +1,199 @@
+import { http } from '@hubspot/local-dev-lib/http';
+import { MIGRATION_STATUS } from '@hubspot/local-dev-lib/types/Migration';
+import { listAppsForMigration, initializeAppMigration, continueAppMigration, checkMigrationStatusV2, isMigrationStatus, } from '../migrate.js';
+vi.mock('@hubspot/local-dev-lib/http');
+const httpMock = 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 listAppsForMigration(mockAccountId, mockPlatformVersion, mockAppId);
+            expect(http.get).toHaveBeenCalledWith(mockAccountId, {
+                url: 'dfs/migrations/v2/list-apps',
+                params: {
+                    platformVersion: convertedPlatformVersion,
+                    appId: 67890,
+                },
+            });
+            expect(result).toEqual(mockResponse);
+        });
+        it('should call http.get with undefined appId when not provided', async () => {
+            const mockResponse = {
+                migratableApps: [],
+                unmigratableApps: [],
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await listAppsForMigration(mockAccountId, mockPlatformVersion, undefined);
+            expect(http.get).toHaveBeenCalledWith(mockAccountId, {
+                url: 'dfs/migrations/v2/list-apps',
+                params: {
+                    platformVersion: convertedPlatformVersion,
+                    appId: undefined,
+                },
+            });
+            expect(result).toEqual(mockResponse);
+        });
+    });
+    describe('initializeAppMigration', () => {
+        it('should call http.post with correct parameters', async () => {
+            const mockResponse = { migrationId: mockMigrationId };
+            // @ts-expect-error Mock
+            httpMock.post.mockResolvedValue(mockResponse);
+            const result = await initializeAppMigration(mockAccountId, mockAppId, mockPlatformVersion);
+            expect(http.post).toHaveBeenCalledWith(mockAccountId, {
+                url: 'dfs/migrations/v2/migrations',
+                data: {
+                    applicationId: mockAppId,
+                    platformVersion: convertedPlatformVersion,
+                },
+            });
+            expect(result).toEqual(mockResponse);
+        });
+    });
+    describe('continueAppMigration', () => {
+        it('should call http.post with correct parameters', async () => {
+            const mockResponse = { migrationId: mockMigrationId };
+            // @ts-expect-error Mock
+            httpMock.post.mockResolvedValue(mockResponse);
+            const result = await continueAppMigration(mockPortalId, mockMigrationId, mockComponentUids, mockProjectName);
+            expect(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_STATUS.IN_PROGRESS,
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await checkMigrationStatusV2(mockAccountId, mockMigrationId);
+            expect(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_STATUS.INPUT_REQUIRED,
+                componentsRequiringUids: {
+                    'component-1': {
+                        componentType: 'type1',
+                        componentHint: 'hint1',
+                    },
+                },
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await checkMigrationStatusV2(mockAccountId, mockMigrationId);
+            expect(result).toEqual(mockResponse);
+        });
+        it('should handle success status', async () => {
+            const mockResponse = {
+                id: mockMigrationId,
+                status: MIGRATION_STATUS.SUCCESS,
+                buildId: 98765,
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await checkMigrationStatusV2(mockAccountId, mockMigrationId);
+            expect(result).toEqual(mockResponse);
+        });
+        it('should handle failure status', async () => {
+            const mockResponse = {
+                id: mockMigrationId,
+                status: MIGRATION_STATUS.FAILURE,
+                projectErrorDetail: 'Error details',
+                componentErrors: [],
+            };
+            // @ts-expect-error Mock
+            httpMock.get.mockResolvedValue(mockResponse);
+            const result = await checkMigrationStatusV2(mockAccountId, mockMigrationId);
+            expect(result).toEqual(mockResponse);
+        });
+    });
+});
+describe('isMigrationStatus', () => {
+    it.each([
+        {
+            id: 123,
+            status: MIGRATION_STATUS.IN_PROGRESS,
+        },
+        {
+            id: 456,
+            status: MIGRATION_STATUS.INPUT_REQUIRED,
+            componentsRequiringUids: {
+                'component-1': {
+                    componentType: 'type1',
+                    componentHint: 'hint1',
+                },
+            },
+        },
+        {
+            id: 789,
+            status: MIGRATION_STATUS.SUCCESS,
+            buildId: 98765,
+        },
+        {
+            id: 101,
+            status: MIGRATION_STATUS.FAILURE,
+            projectErrorDetail: 'Error details',
+            componentErrors: [],
+        },
+    ])('should return true for valid MigrationStatus object %j', status => {
+        expect(isMigrationStatus(status)).toBe(true);
+    });
+    it.each([null, undefined, 123, 'string', true, false, []])('should return false for non-object value %j', value => {
+        expect(isMigrationStatus(value)).toBe(false);
+    });
+    it.each([
+        {},
+        { id: 123 },
+        { status: MIGRATION_STATUS.IN_PROGRESS },
+        { foo: 'bar' },
+    ])('should return false for invalid object %j', obj => {
+        expect(isMigrationStatus(obj)).toBe(false);
+    });
+});

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

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

package/commands/__tests__/account.test.js

@@ -0,0 +1,69 @@
+import yargs from 'yargs';
+import auth from '../account/auth.js';
+import list from '../account/list.js';
+import rename from '../account/rename.js';
+import use from '../account/use.js';
+import info from '../account/info.js';
+import remove from '../account/remove.js';
+import clean from '../account/clean.js';
+import createOverride from '../account/createOverride.js';
+import removeOverride from '../account/removeOverride.js';
+import accountCommands from '../account.js';
+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, 'command')
+    .mockReturnValue(yargs);
+const demandCommandSpy = vi
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
+describe('commands/account', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(accountCommands.command).toEqual(['account', 'accounts']);
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(accountCommands.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        beforeEach(() => {
+            commandSpy.mockClear();
+            demandCommandSpy.mockClear();
+        });
+        const subcommands = [
+            auth,
+            list,
+            rename,
+            use,
+            info,
+            remove,
+            clean,
+            createOverride,
+            removeOverride,
+        ];
+        it('should demand the command takes one positional argument', () => {
+            accountCommands.builder(yargs);
+            expect(demandCommandSpy).toHaveBeenCalledTimes(1);
+            expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
+        });
+        it('should add the correct number of sub commands', () => {
+            accountCommands.builder(yargs);
+            expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
+        });
+        it.each(subcommands)('should attach the %s subcommand', module => {
+            accountCommands.builder(yargs);
+            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 @@
+import yargs from 'yargs';
+import { addConfigOptions, addTestingOptions } from '../../lib/commonOpts.js';
+import authCommand from '../auth.js';
+vi.mock('../../lib/commonOpts');
+const optionsSpy = vi
+    .spyOn(mockYargs, 'options')
+    .mockReturnValue(yargs);
+describe('commands/auth', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(authCommand.command).toEqual('auth');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(authCommand.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        it('should support the correct options', () => {
+            authCommand.builder(yargs);
+            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' }),
+                'disable-tracking': expect.objectContaining({
+                    type: 'boolean',
+                    hidden: true,
+                    default: false,
+                }),
+            });
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargs);
+            expect(addTestingOptions).toHaveBeenCalledTimes(1);
+            expect(addTestingOptions).toHaveBeenCalledWith(yargs);
+        });
+    });
+});

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

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

package/commands/__tests__/cms.test.js

@@ -0,0 +1,87 @@
+import yargs from 'yargs';
+import lighthouseScore from '../cms/lighthouseScore.js';
+import convertFields from '../cms/convertFields.js';
+import getReactModule from '../cms/getReactModule.js';
+import watchCommand from '../cms/watch.js';
+import listCommand from '../cms/list.js';
+import uploadCommand from '../cms/upload.js';
+import fetchCommand from '../cms/fetch.js';
+import deleteCommand from '../cms/delete.js';
+import mvCommand from '../cms/mv.js';
+import functionCommand from '../cms/function.js';
+import lintCommand from '../cms/lint.js';
+import themeCommand from '../cms/theme.js';
+import moduleCommand from '../cms/module.js';
+import webpackCommand from '../cms/webpack.js';
+import appCommand from '../cms/app.js';
+import templateCommand from '../cms/template.js';
+import cmsCommand from '../cms.js';
+vi.mock('../cms/lighthouseScore');
+vi.mock('../cms/convertFields');
+vi.mock('../cms/getReactModule');
+vi.mock('../cms/watch');
+vi.mock('../cms/list');
+vi.mock('../cms/upload');
+vi.mock('../cms/fetch');
+vi.mock('../cms/delete');
+vi.mock('../cms/mv');
+vi.mock('../cms/function');
+vi.mock('../cms/lint');
+vi.mock('../cms/theme');
+vi.mock('../cms/module');
+vi.mock('../cms/webpack');
+vi.mock('../cms/app');
+vi.mock('../cms/template');
+vi.mock('../../lib/commonOpts');
+const commandSpy = vi
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
+const demandCommandSpy = vi
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
+describe('commands/cms', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(cmsCommand.command).toEqual('cms');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(cmsCommand.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        const subcommands = [
+            lighthouseScore,
+            convertFields,
+            getReactModule,
+            watchCommand,
+            listCommand,
+            uploadCommand,
+            fetchCommand,
+            deleteCommand,
+            mvCommand,
+            functionCommand,
+            lintCommand,
+            themeCommand,
+            moduleCommand,
+            webpackCommand,
+            appCommand,
+            templateCommand,
+        ];
+        it('should demand the command takes one positional argument', () => {
+            cmsCommand.builder(yargs);
+            expect(demandCommandSpy).toHaveBeenCalledTimes(1);
+            expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
+        });
+        it('should add the correct number of sub commands', () => {
+            cmsCommand.builder(yargs);
+            expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
+        });
+        it.each(subcommands)('should attach the %s subcommand', module => {
+            cmsCommand.builder(yargs);
+            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,44 @@
+import yargs from 'yargs';
+import set from '../config/set.js';
+import migrate from '../config/migrate.js';
+import configCommands from '../config.js';
+vi.mock('../config/set');
+vi.mock('../config/set');
+const commandSpy = vi
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
+const demandCommandSpy = vi
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
+describe('commands/config', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(configCommands.command).toEqual('config');
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(configCommands.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        beforeEach(() => {
+            commandSpy.mockClear();
+            demandCommandSpy.mockClear();
+        });
+        const subcommands = [set, migrate];
+        it('should demand the command takes one positional argument', () => {
+            configCommands.builder(yargs);
+            expect(demandCommandSpy).toHaveBeenCalledTimes(1);
+            expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
+        });
+        it('should add the correct number of sub commands', () => {
+            configCommands.builder(yargs);
+            expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
+        });
+        it.each(subcommands)('should attach the %s subcommand', module => {
+            configCommands.builder(yargs);
+            expect(commandSpy).toHaveBeenCalledWith(module);
+        });
+    });
+});

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

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

package/commands/__tests__/customObject.test.js

@@ -0,0 +1,68 @@
+import yargs from 'yargs';
+import create from '../customObject/create.js';
+import createSchema from '../customObject/createSchema.js';
+import updateSchema from '../customObject/updateSchema.js';
+import listSchemas from '../customObject/listSchemas.js';
+import deleteSchema from '../customObject/deleteSchema.js';
+import fetchSchema from '../customObject/fetchSchema.js';
+import fetchAllSchemas from '../customObject/fetchAllSchemas.js';
+import customObjectCommands from '../customObject.js';
+vi.mock('../customObject/create');
+vi.mock('../customObject/schema');
+vi.mock('../customObject/createSchema');
+vi.mock('../customObject/updateSchema');
+vi.mock('../customObject/listSchemas');
+vi.mock('../customObject/deleteSchema');
+vi.mock('../customObject/fetchSchema');
+vi.mock('../customObject/fetchAllSchemas');
+vi.mock('../../lib/commonOpts');
+const commandSpy = vi
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
+const demandCommandSpy = vi
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
+describe('commands/customObject', () => {
+    describe('command', () => {
+        it('should have the correct command structure', () => {
+            expect(customObjectCommands.command).toEqual([
+                'custom-object',
+                'custom-objects',
+                'co',
+            ]);
+        });
+    });
+    describe('describe', () => {
+        it('should provide a description', () => {
+            expect(customObjectCommands.describe).toBeDefined();
+        });
+    });
+    describe('builder', () => {
+        beforeEach(() => {
+            commandSpy.mockClear();
+            demandCommandSpy.mockClear();
+        });
+        const subcommands = [
+            create,
+            createSchema,
+            updateSchema,
+            listSchemas,
+            deleteSchema,
+            fetchSchema,
+            fetchAllSchemas,
+        ];
+        it('should demand the command takes one positional argument', () => {
+            customObjectCommands.builder(yargs);
+            expect(demandCommandSpy).toHaveBeenCalledTimes(1);
+            expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
+        });
+        it('should add the correct number of sub commands', () => {
+            customObjectCommands.builder(yargs);
+            expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
+        });
+        it.each(subcommands)('should attach the %s subcommand', module => {
+            customObjectCommands.builder(yargs);
+            expect(commandSpy).toHaveBeenCalledWith(module);
+        });
+    });
+});

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

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

package/commands/__tests__/doctor.test.js

@@ -0,0 +1,132 @@
+import { EXIT_CODES } from '../../lib/enums/exitCodes.js';
+import yargs from 'yargs';
+import doctorCommand from '../doctor.js';
+import { trackCommandUsage } from '../../lib/usageTracking.js';
+import { Doctor } from '../../lib/doctor/Doctor.js';
+import { uiLogger } from '../../lib/ui/logger.js';
+import fs from 'fs';
+import { getCwd as __getCwd } from '@hubspot/local-dev-lib/path';
+vi.mock('../../lib/doctor/Doctor');
+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;
+const mockedFs = vi.mocked(fs);
+const getCwd = __getCwd;
+const optionSpy = vi
+    .spyOn(yargs, 'option')
+    .mockReturnValue(yargs);
+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(doctorCommand.command).toEqual('doctor');
+        });
+    });
+    describe('describe', () => {
+        it('should have a description', () => {
+            expect(doctorCommand.describe).toEqual('Retrieve diagnostic information about your local HubSpot configurations.');
+        });
+    });
+    describe('builder', () => {
+        it('should apply the correct options', () => {
+            doctorCommand.builder(yargs);
+            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 doctorCommand.handler({});
+            expect(trackCommandUsage).toHaveBeenCalledTimes(1);
+            expect(trackCommandUsage).toHaveBeenCalledWith('doctor', undefined, accountId);
+        });
+        it('should log the diagnosis if it is defined', async () => {
+            await doctorCommand.handler({});
+            expect(uiLogger.log).toHaveBeenCalledTimes(1);
+            expect(uiLogger.log).toHaveBeenCalledWith(diagnosis);
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.SUCCESS);
+        });
+        it('should log an error if the diagnosis is undefined', async () => {
+            DoctorMock.mockImplementationOnce(() => {
+                return {
+                    diagnose: vi.fn().mockResolvedValue(undefined),
+                    accountId,
+                };
+            });
+            await doctorCommand.handler({});
+            expect(uiLogger.error).toHaveBeenCalledTimes(1);
+            expect(uiLogger.error).toHaveBeenCalledWith('Error generating diagnosis');
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(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 doctorCommand.handler({
+                outputDir: '/foo',
+            });
+            expect(uiLogger.log).not.toHaveBeenCalled();
+            expect(uiLogger.error).not.toHaveBeenCalled();
+            expect(mockedFs.writeFileSync).toHaveBeenCalledTimes(1);
+            expect(mockedFs.writeFileSync).toHaveBeenCalledWith(expectedOutputFile, expect.stringContaining(diagnosis));
+            expect(uiLogger.success).toHaveBeenCalledTimes(1);
+            expect(uiLogger.success).toHaveBeenCalledWith(expect.stringMatching(/Output written to /));
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(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 doctorCommand.handler({
+                outputDir: './foo',
+            });
+            expect(uiLogger.log).not.toHaveBeenCalled();
+            expect(uiLogger.error).not.toHaveBeenCalled();
+            expect(mockedFs.writeFileSync).toHaveBeenCalledTimes(1);
+            expect(mockedFs.writeFileSync).toHaveBeenCalledWith(expectedOutputFile, expect.stringContaining(diagnosis));
+            expect(uiLogger.success).toHaveBeenCalledTimes(1);
+            expect(uiLogger.success).toHaveBeenCalledWith(expect.stringMatching(/Output written to /));
+            expect(processExitSpy).toHaveBeenCalledTimes(1);
+            expect(processExitSpy).toHaveBeenCalledWith(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 doctorCommand.handler({
+                outputDir: '/foo',
+            });
+            expect(uiLogger.error).toHaveBeenCalledTimes(1);
+            expect(uiLogger.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(EXIT_CODES.ERROR);
+        });
+    });
+});

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

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