@hubspot/cli 7.6.0-beta.5 → 7.6.0-beta.6

package/commands/__tests__/auth.test.js

@@ -1,29 +1,24 @@
-"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"));
+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_1.default);
+    .mockReturnValue(yargs);
 describe('commands/auth', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(auth_1.default.command).toEqual('auth');
+            expect(authCommand.command).toEqual('auth');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(auth_1.default.describe).toBeDefined();
+            expect(authCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
         it('should support the correct options', () => {
-            auth_1.default.builder(yargs_1.default);
+            authCommand.builder(yargs);
             expect(optionsSpy).toHaveBeenCalledTimes(1);
             expect(optionsSpy).toHaveBeenCalledWith({
                 'auth-type': expect.objectContaining({
@@ -34,10 +29,10 @@ describe('commands/auth', () => {
                 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);
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargs);
+            expect(addTestingOptions).toHaveBeenCalledTimes(1);
+            expect(addTestingOptions).toHaveBeenCalledWith(yargs);
         });
     });
 });

package/commands/__tests__/cms.test.js

@@ -1,47 +1,42 @@
-"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"));
+import yargs from 'yargs';
+import lighthouseScore from '../cms/lighthouseScore.js';
+import convertFields from '../cms/convertFields.js';
+import getReactModule from '../cms/getReactModule.js';
+import cmsCommand from '../cms.js';
 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);
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
 const demandCommandSpy = vi
-    .spyOn(yargs_1.default, 'demandCommand')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
 describe('commands/cms', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(cms_1.default.command).toEqual('cms');
+            expect(cmsCommand.command).toEqual('cms');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(cms_1.default.describe).toBeDefined();
+            expect(cmsCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
-        const subcommands = [lighthouseScore_1.default, convertFields_1.default, getReactModule_1.default];
+        const subcommands = [lighthouseScore, convertFields, getReactModule];
         it('should demand the command takes one positional argument', () => {
-            cms_1.default.builder(yargs_1.default);
+            cmsCommand.builder(yargs);
             expect(demandCommandSpy).toHaveBeenCalledTimes(1);
             expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
         });
         it('should add the correct number of sub commands', () => {
-            cms_1.default.builder(yargs_1.default);
+            cmsCommand.builder(yargs);
             expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
         });
         it.each(subcommands)('should attach the %s subcommand', module => {
-            cms_1.default.builder(yargs_1.default);
+            cmsCommand.builder(yargs);
             expect(module).toBeDefined();
             expect(commandSpy).toHaveBeenCalledWith(module);
         });

package/commands/__tests__/config.test.js

@@ -1,29 +1,24 @@
-"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"));
+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_1.default, 'command')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
 const demandCommandSpy = vi
-    .spyOn(yargs_1.default, 'demandCommand')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
 describe('commands/config', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(config_1.default.command).toEqual('config');
+            expect(configCommands.command).toEqual('config');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(config_1.default.describe).toBeDefined();
+            expect(configCommands.describe).toBeDefined();
         });
     });
     describe('builder', () => {
@@ -31,18 +26,18 @@ describe('commands/config', () => {
             commandSpy.mockClear();
             demandCommandSpy.mockClear();
         });
-        const subcommands = [set_1.default, migrate_1.default];
+        const subcommands = [set, migrate];
         it('should demand the command takes one positional argument', () => {
-            config_1.default.builder(yargs_1.default);
+            configCommands.builder(yargs);
             expect(demandCommandSpy).toHaveBeenCalledTimes(1);
             expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
         });
         it('should add the correct number of sub commands', () => {
-            config_1.default.builder(yargs_1.default);
+            configCommands.builder(yargs);
             expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
         });
         it.each(subcommands)('should attach the %s subcommand', module => {
-            config_1.default.builder(yargs_1.default);
+            configCommands.builder(yargs);
             expect(commandSpy).toHaveBeenCalledWith(module);
         });
     });

package/commands/__tests__/create.test.js

@@ -1,37 +1,32 @@
-"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"));
+import yargs from 'yargs';
+import createCommand from '../create.js';
 const positionalSpy = vi
-    .spyOn(yargs_1.default, 'positional')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'positional')
+    .mockReturnValue(yargs);
 const optionSpy = vi
-    .spyOn(yargs_1.default, 'option')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'option')
+    .mockReturnValue(yargs);
 describe('commands/create', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(create_1.default.command).toEqual('create <type> [name] [dest]');
+            expect(createCommand.command).toEqual('create <type> [name] [dest]');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(create_1.default.describe).toBeDefined();
+            expect(createCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
         it('should support the correct positional arguments', () => {
-            create_1.default.builder(yargs_1.default);
+            createCommand.builder(yargs);
             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);
+            createCommand.builder(yargs);
             expect(optionSpy).toHaveBeenCalledWith('internal', expect.objectContaining({ type: 'boolean', hidden: true }));
         });
     });

package/commands/__tests__/customObject.test.js

@@ -1,25 +1,20 @@
-"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"));
+import yargs from 'yargs';
+import create from '../customObject/create.js';
+import schema from '../customObject/schema.js';
+import customObjectCommands from '../customObject.js';
 vi.mock('../customObject/create');
 vi.mock('../customObject/schema');
 vi.mock('../../lib/commonOpts');
 const commandSpy = vi
-    .spyOn(yargs_1.default, 'command')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'command')
+    .mockReturnValue(yargs);
 const demandCommandSpy = vi
-    .spyOn(yargs_1.default, 'demandCommand')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'demandCommand')
+    .mockReturnValue(yargs);
 describe('commands/customObject', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(customObject_1.default.command).toEqual([
+            expect(customObjectCommands.command).toEqual([
                 'custom-object',
                 'custom-objects',
                 'co',
@@ -28,7 +23,7 @@ describe('commands/customObject', () => {
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(customObject_1.default.describe).toBeDefined();
+            expect(customObjectCommands.describe).toBeDefined();
         });
     });
     describe('builder', () => {
@@ -36,18 +31,18 @@ describe('commands/customObject', () => {
             commandSpy.mockClear();
             demandCommandSpy.mockClear();
         });
-        const subcommands = [create_1.default, schema_1.default];
+        const subcommands = [create, schema];
         it('should demand the command takes one positional argument', () => {
-            customObject_1.default.builder(yargs_1.default);
+            customObjectCommands.builder(yargs);
             expect(demandCommandSpy).toHaveBeenCalledTimes(1);
             expect(demandCommandSpy).toHaveBeenCalledWith(1, '');
         });
         it('should add the correct number of sub commands', () => {
-            customObject_1.default.builder(yargs_1.default);
+            customObjectCommands.builder(yargs);
             expect(commandSpy).toHaveBeenCalledTimes(subcommands.length);
         });
         it.each(subcommands)('should attach the %s subcommand', module => {
-            customObject_1.default.builder(yargs_1.default);
+            customObjectCommands.builder(yargs);
             expect(commandSpy).toHaveBeenCalledWith(module);
         });
     });

package/commands/__tests__/doctor.test.js

@@ -1,28 +1,23 @@
-"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");
+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 { logger } from '@hubspot/local-dev-lib/logger';
+import fs from 'fs';
+import { getCwd as __getCwd } from '@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 DoctorMock = Doctor;
+const mockedFs = vi.mocked(fs);
+const getCwd = __getCwd;
 const optionSpy = vi
-    .spyOn(yargs_1.default, 'option')
-    .mockReturnValue(yargs_1.default);
+    .spyOn(yargs, 'option')
+    .mockReturnValue(yargs);
 const date = new Date('2022-02-22');
 vi.useFakeTimers().setSystemTime(date);
 describe('doctor', () => {
@@ -35,17 +30,17 @@ describe('doctor', () => {
     });
     describe('command', () => {
         it('should have the proper command name', () => {
-            expect(doctor_1.default.command).toEqual('doctor');
+            expect(doctorCommand.command).toEqual('doctor');
         });
     });
     describe('describe', () => {
         it('should have a description', () => {
-            expect(doctor_1.default.describe).toEqual('Retrieve diagnostic information about your local HubSpot configurations.');
+            expect(doctorCommand.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);
+            doctorCommand.builder(yargs);
             expect(optionSpy).toHaveBeenCalledWith('output-dir', {
                 describe: 'Directory to save a detailed diagnosis JSON file in',
                 type: 'string',
@@ -64,16 +59,16 @@ describe('doctor', () => {
             });
         });
         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);
+            await doctorCommand.handler({});
+            expect(trackCommandUsage).toHaveBeenCalledTimes(1);
+            expect(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);
+            await doctorCommand.handler({});
+            expect(logger.log).toHaveBeenCalledTimes(1);
+            expect(logger.log).toHaveBeenCalledWith(diagnosis);
             expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.SUCCESS);
+            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.SUCCESS);
         });
         it('should log an error if the diagnosis is undefined', async () => {
             DoctorMock.mockImplementationOnce(() => {
@@ -82,58 +77,58 @@ describe('doctor', () => {
                     accountId,
                 };
             });
-            await doctor_1.default.handler({});
-            expect(logger_1.logger.error).toHaveBeenCalledTimes(1);
-            expect(logger_1.logger.error).toHaveBeenCalledWith('Error generating diagnosis');
+            await doctorCommand.handler({});
+            expect(logger.error).toHaveBeenCalledTimes(1);
+            expect(logger.error).toHaveBeenCalledWith('Error generating diagnosis');
             expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.ERROR);
+            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 doctor_1.default.handler({
+            await doctorCommand.handler({
                 outputDir: '/foo',
             });
-            expect(logger_1.logger.log).not.toHaveBeenCalled();
-            expect(logger_1.logger.error).not.toHaveBeenCalled();
+            expect(logger.log).not.toHaveBeenCalled();
+            expect(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(logger.success).toHaveBeenCalledTimes(1);
+            expect(logger.success).toHaveBeenCalledWith(expect.stringMatching(/Output written to /));
             expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.SUCCESS);
+            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 doctor_1.default.handler({
+            await doctorCommand.handler({
                 outputDir: './foo',
             });
-            expect(logger_1.logger.log).not.toHaveBeenCalled();
-            expect(logger_1.logger.error).not.toHaveBeenCalled();
+            expect(logger.log).not.toHaveBeenCalled();
+            expect(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(logger.success).toHaveBeenCalledTimes(1);
+            expect(logger.success).toHaveBeenCalledWith(expect.stringMatching(/Output written to /));
             expect(processExitSpy).toHaveBeenCalledTimes(1);
-            expect(processExitSpy).toHaveBeenCalledWith(exitCodes_1.EXIT_CODES.SUCCESS);
+            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 doctor_1.default.handler({
+            await doctorCommand.handler({
                 outputDir: '/foo',
             });
-            expect(logger_1.logger.error).toHaveBeenCalledTimes(1);
-            expect(logger_1.logger.error).toHaveBeenCalledWith(expect.stringMatching(/Unable to write output to/));
+            expect(logger.error).toHaveBeenCalledTimes(1);
+            expect(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);
+            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
         });
     });
 });

package/commands/__tests__/feedback.test.js

@@ -1,62 +1,24 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
-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 = __importStar(require("../../lib/commonOpts"));
-const feedback_1 = __importDefault(require("../feedback"));
+import yargs from 'yargs';
+import * as commonOpts from '../../lib/commonOpts.js';
+import feedbackCommand from '../feedback.js';
 vi.mock('yargs');
 vi.mock('../../lib/commonOpts');
 describe('commands/feedback', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(feedback_1.default.command).toEqual('feedback');
+            expect(feedbackCommand.command).toEqual('feedback');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(feedback_1.default.describe).toBeDefined();
+            expect(feedbackCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
         it('should support the correct options', () => {
-            feedback_1.default.builder(yargs_1.default);
+            feedbackCommand.builder(yargs);
             expect(commonOpts.addGlobalOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts.addGlobalOptions).toHaveBeenCalledWith(yargs_1.default);
+            expect(commonOpts.addGlobalOptions).toHaveBeenCalledWith(yargs);
         });
     });
 });

package/commands/__tests__/fetch.test.js

@@ -1,56 +1,57 @@
-"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");
+import yargs from 'yargs';
+import { addAccountOptions, addConfigOptions, addOverwriteOptions, addCmsPublishModeOptions, addUseEnvironmentOptions, } from '../../lib/commonOpts.js';
 vi.mock('../../lib/commonOpts');
 // Import this last so mocks apply
-const fetch_1 = __importDefault(require("../fetch"));
+import fetchCommand from '../fetch.js';
+const positionalSpy = vi
+    .spyOn(yargs, 'positional')
+    .mockReturnValue(yargs);
+const optionsSpy = vi
+    .spyOn(yargs, 'options')
+    .mockReturnValue(yargs);
 describe('commands/fetch', () => {
     describe('command', () => {
         it('should have the correct command structure', () => {
-            expect(fetch_1.default.command).toEqual('fetch <src> [dest]');
+            expect(fetchCommand.command).toEqual('fetch <src> [dest]');
         });
     });
     describe('describe', () => {
         it('should provide a description', () => {
-            expect(fetch_1.default.describe).toBeDefined();
+            expect(fetchCommand.describe).toBeDefined();
         });
     });
     describe('builder', () => {
         it('should support the correct positional arguments', () => {
-            fetch_1.default.builder(yargs_1.default);
-            expect(yargs_1.default.positional).toHaveBeenCalledTimes(2);
-            expect(yargs_1.default.positional).toHaveBeenCalledWith('src', expect.objectContaining({ type: 'string' }));
-            expect(yargs_1.default.positional).toHaveBeenCalledWith('dest', expect.objectContaining({ type: 'string' }));
+            fetchCommand.builder(yargs);
+            expect(positionalSpy).toHaveBeenCalledTimes(2);
+            expect(positionalSpy).toHaveBeenCalledWith('src', expect.objectContaining({ type: 'string' }));
+            expect(positionalSpy).toHaveBeenCalledWith('dest', expect.objectContaining({ type: 'string' }));
         });
         it('should support the correct options', () => {
-            fetch_1.default.builder(yargs_1.default);
-            expect(yargs_1.default.options).toHaveBeenCalledTimes(2);
-            expect(yargs_1.default.options).toHaveBeenCalledWith({
+            fetchCommand.builder(yargs);
+            expect(optionsSpy).toHaveBeenCalledTimes(2);
+            expect(optionsSpy).toHaveBeenCalledWith({
                 staging: expect.objectContaining({
                     type: 'boolean',
                     default: false,
                     hidden: true,
                 }),
             });
-            expect(yargs_1.default.options).toHaveBeenCalledWith({
+            expect(optionsSpy).toHaveBeenCalledWith({
                 assetVersion: expect.objectContaining({ type: 'number' }),
             });
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addConfigOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addAccountOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addOverwriteOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addOverwriteOptions).toHaveBeenCalledWith(yargs_1.default);
-            expect(commonOpts_1.addCmsPublishModeOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addCmsPublishModeOptions).toHaveBeenCalledWith(yargs_1.default, {
+            expect(addConfigOptions).toHaveBeenCalledTimes(1);
+            expect(addConfigOptions).toHaveBeenCalledWith(yargs);
+            expect(addAccountOptions).toHaveBeenCalledTimes(1);
+            expect(addAccountOptions).toHaveBeenCalledWith(yargs);
+            expect(addOverwriteOptions).toHaveBeenCalledTimes(1);
+            expect(addOverwriteOptions).toHaveBeenCalledWith(yargs);
+            expect(addCmsPublishModeOptions).toHaveBeenCalledTimes(1);
+            expect(addCmsPublishModeOptions).toHaveBeenCalledWith(yargs, {
                 read: true,
             });
-            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
-            expect(commonOpts_1.addUseEnvironmentOptions).toHaveBeenCalledWith(yargs_1.default);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledTimes(1);
+            expect(addUseEnvironmentOptions).toHaveBeenCalledWith(yargs);
         });
     });
 });