@hubspot/cli 8.1.0 → 8.1.1-experimental.0

package/api/__tests__/migrate.test.js

@@ -1,199 +0,0 @@
-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

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

package/commands/__tests__/account.test.js

@@ -1,69 +0,0 @@
-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

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

package/commands/__tests__/auth.test.js

@@ -1,43 +0,0 @@
-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

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

package/commands/__tests__/cms.test.js

@@ -1,87 +0,0 @@
-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

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

package/commands/__tests__/config.test.js

@@ -1,44 +0,0 @@
-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

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

package/commands/__tests__/customObject.test.js

@@ -1,68 +0,0 @@
-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

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

package/commands/__tests__/doctor.test.js

@@ -1,132 +0,0 @@
-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

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