@hubspot/cli 8.3.0 → 8.4.0-beta.0

package/lib/middleware/__tests__/configMiddleware.test.js

@@ -1,118 +0,0 @@
-import { uiLogger } from '../../ui/logger.js';
-import * as cliConfig from '@hubspot/local-dev-lib/config';
-import * as validation from '../../validation.js';
-import { EXIT_CODES } from '../../enums/exitCodes.js';
-import { injectAccountIdMiddleware, validateConfigMiddleware, validateAccountOptions, } from '../configMiddleware.js';
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('../../validation');
-const validateAccountSpy = vi.spyOn(validation, 'validateAccount');
-const getConfigAccountIfExistsSpy = vi.spyOn(cliConfig, 'getConfigAccountIfExists');
-const globalConfigFileExistsSpy = vi.spyOn(cliConfig, 'globalConfigFileExists');
-const configFileExistsSpy = vi.spyOn(cliConfig, 'configFileExists');
-const getConfigFilePathSpy = vi.spyOn(cliConfig, 'getConfigFilePath');
-const validateConfigSpy = vi.spyOn(cliConfig, 'validateConfig');
-const processExitSpy = vi.spyOn(process, 'exit');
-describe('lib/middleware/configMiddleware', () => {
-    beforeEach(() => {
-        processExitSpy.mockImplementation(code => {
-            throw new Error(`Process.exit called with code ${code}`);
-        });
-        getConfigFilePathSpy.mockReturnValue('/path/to/config');
-    });
-    describe('injectAccountIdMiddleware()', () => {
-        it('should use HUBSPOT_ACCOUNT_ID from environment when useEnv is true', async () => {
-            const originalEnv = process.env;
-            process.env = {
-                ...originalEnv,
-                HUBSPOT_ACCOUNT_ID: '123',
-            };
-            const argv = {
-                _: ['some-command'],
-                useEnv: true,
-                $0: 'hs',
-            };
-            await injectAccountIdMiddleware(argv);
-            expect(argv.userProvidedAccount).toBeUndefined();
-            expect(argv.derivedAccountId).toBe(123);
-            expect(getConfigAccountIfExistsSpy).not.toHaveBeenCalled();
-            process.env = originalEnv;
-        });
-        it('should use getAccountId when useEnv is false', async () => {
-            getConfigAccountIfExistsSpy.mockReturnValue({
-                accountId: 456,
-            });
-            const argv = {
-                _: ['some-command'],
-                account: 'test-account',
-                useEnv: false,
-                $0: 'hs',
-            };
-            await injectAccountIdMiddleware(argv);
-            expect(argv.userProvidedAccount).toBe('test-account');
-            expect(argv.derivedAccountId).toBe(456);
-            expect(getConfigAccountIfExistsSpy).toHaveBeenCalledWith('test-account');
-        });
-    });
-    describe('validateConfigMiddleware()', () => {
-        it('should allow using --config flag', async () => {
-            configFileExistsSpy.mockReturnValue(true);
-            validateConfigSpy.mockReturnValue({ isValid: true, errors: [] });
-            const argv = {
-                _: ['some-command'],
-                config: 'custom-config.json',
-                $0: 'hs',
-            };
-            await validateConfigMiddleware(argv);
-            // Should not throw or exit - this is now allowed
-            expect(processExitSpy).not.toHaveBeenCalled();
-            expect(uiLogger.error).not.toHaveBeenCalled();
-        });
-        it('should validate config for non-init commands', async () => {
-            globalConfigFileExistsSpy.mockReturnValue(true);
-            validateConfigSpy.mockReturnValue({ isValid: true, errors: [] });
-            const argv = {
-                _: ['some-command'],
-                $0: 'hs',
-            };
-            await validateConfigMiddleware(argv);
-            expect(validateConfigSpy).toHaveBeenCalled();
-        });
-        it('should skip validation for init command', async () => {
-            globalConfigFileExistsSpy.mockReturnValue(false);
-            const argv = {
-                _: ['init'],
-                $0: 'hs',
-            };
-            await validateConfigMiddleware(argv);
-            expect(validateConfigSpy).not.toHaveBeenCalled();
-        });
-    });
-    describe('validateAccountOptions()', () => {
-        it('should validate account for non-skipped commands', async () => {
-            validateAccountSpy.mockResolvedValue(true);
-            const argv = {
-                _: ['some-command'],
-                $0: 'hs',
-            };
-            await validateAccountOptions(argv);
-            expect(validateAccountSpy).toHaveBeenCalledWith(argv);
-        });
-        it('should skip validation for init command', async () => {
-            const argv = {
-                _: ['init'],
-                $0: 'hs',
-            };
-            await validateAccountOptions(argv);
-            expect(validateAccountSpy).not.toHaveBeenCalled();
-        });
-        it('should exit with error when account validation fails', async () => {
-            validateAccountSpy.mockResolvedValue(false);
-            const argv = {
-                _: ['some-command'],
-                $0: 'hs',
-            };
-            await expect(validateAccountOptions(argv)).rejects.toThrow();
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-        });
-    });
-});

package/lib/middleware/__tests__/gitMiddleware.test.d.ts

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

package/lib/middleware/__tests__/gitMiddleware.test.js

@@ -1,43 +0,0 @@
-import * as config from '@hubspot/local-dev-lib/config';
-import * as gitUI from '../../ui/git.js';
-import { checkAndWarnGitInclusionMiddleware } from '../gitMiddleware.js';
-vi.mock('@hubspot/local-dev-lib/config');
-vi.mock('../../ui/git');
-const getConfigFilePathSpy = vi.spyOn(config, 'getConfigFilePath');
-const checkAndWarnGitInclusionSpy = vi.spyOn(gitUI, 'checkAndWarnGitInclusion');
-describe('lib/middleware/gitMiddleware', () => {
-    describe('checkAndWarnGitInclusionMiddleware()', () => {
-        it('should call checkAndWarnGitInclusion when command is provided and config path exists', () => {
-            const mockConfigPath = '/path/to/config.js';
-            getConfigFilePathSpy.mockReturnValue(mockConfigPath);
-            const argv = {
-                _: ['some-command'],
-                $0: 'hs',
-            };
-            checkAndWarnGitInclusionMiddleware(argv);
-            expect(getConfigFilePathSpy).toHaveBeenCalledTimes(1);
-            expect(checkAndWarnGitInclusionSpy).toHaveBeenCalledWith(mockConfigPath);
-        });
-        it('should not call checkAndWarnGitInclusion when no command is provided', () => {
-            const argv = {
-                _: [],
-                $0: 'hs',
-            };
-            checkAndWarnGitInclusionMiddleware(argv);
-            expect(getConfigFilePathSpy).not.toHaveBeenCalled();
-            expect(checkAndWarnGitInclusionSpy).not.toHaveBeenCalled();
-        });
-        it('should not call checkAndWarnGitInclusion when config path does not exist', () => {
-            getConfigFilePathSpy.mockImplementation(() => {
-                throw new Error('Config path does not exist');
-            });
-            const argv = {
-                _: ['some-command'],
-                $0: 'hs',
-            };
-            checkAndWarnGitInclusionMiddleware(argv);
-            expect(getConfigFilePathSpy).toHaveBeenCalledTimes(1);
-            expect(checkAndWarnGitInclusionSpy).not.toHaveBeenCalled();
-        });
-    });
-});

package/lib/middleware/__tests__/requestMiddleware.test.d.ts

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

package/lib/middleware/__tests__/requestMiddleware.test.js

@@ -1,15 +0,0 @@
-import { addUserAgentHeader } from '@hubspot/local-dev-lib/http';
-import { setRequestHeaders } from '../requestMiddleware.js';
-import { pkg } from '../../jsonLoader.js';
-vi.mock('@hubspot/local-dev-lib/http', () => ({
-    addUserAgentHeader: vi.fn(),
-}));
-describe('lib/middleware/requestMiddleware', () => {
-    describe('setRequestHeaders()', () => {
-        it('should call addUserAgentHeader with correct parameters', () => {
-            setRequestHeaders();
-            expect(addUserAgentHeader).toHaveBeenCalledTimes(1);
-            expect(addUserAgentHeader).toHaveBeenCalledWith('HubSpot CLI', pkg.version);
-        });
-    });
-});

package/lib/middleware/__tests__/usageTrackingMiddleware.test.d.ts

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

package/lib/middleware/__tests__/usageTrackingMiddleware.test.js

@@ -1,44 +0,0 @@
-import { handleDisableUsageTracking, isUsageTrackingDisableFlagSet, } from '../usageTrackingMiddleware.js';
-describe('lib/middleware/usageTrackingMiddleware', () => {
-    beforeEach(() => {
-        delete process.env.DISABLE_USAGE_TRACKING;
-    });
-    afterEach(() => {
-        delete process.env.DISABLE_USAGE_TRACKING;
-    });
-    describe('handleDisableUsageTracking()', () => {
-        it('should set environment variable when flag is true', () => {
-            const argv = { disableUsageTracking: true };
-            handleDisableUsageTracking(argv);
-            expect(process.env.DISABLE_USAGE_TRACKING).toBe('true');
-        });
-        it('should not set environment variable when flag is false', () => {
-            const argv = { disableUsageTracking: false };
-            handleDisableUsageTracking(argv);
-            expect(process.env.DISABLE_USAGE_TRACKING).toBeUndefined();
-        });
-        it('should not set environment variable when flag is undefined', () => {
-            const argv = {};
-            handleDisableUsageTracking(argv);
-            expect(process.env.DISABLE_USAGE_TRACKING).toBeUndefined();
-        });
-    });
-    describe('isUsageTrackingDisableFlagSet()', () => {
-        it('should return true when environment variable is set to "true"', () => {
-            process.env.DISABLE_USAGE_TRACKING = 'true';
-            expect(isUsageTrackingDisableFlagSet()).toBe(true);
-        });
-        it('should return false when environment variable is not set', () => {
-            delete process.env.DISABLE_USAGE_TRACKING;
-            expect(isUsageTrackingDisableFlagSet()).toBe(false);
-        });
-        it('should return false when environment variable is set to other values', () => {
-            process.env.DISABLE_USAGE_TRACKING = 'false';
-            expect(isUsageTrackingDisableFlagSet()).toBe(false);
-        });
-        it('should return false when environment variable is empty string', () => {
-            process.env.DISABLE_USAGE_TRACKING = '';
-            expect(isUsageTrackingDisableFlagSet()).toBe(false);
-        });
-    });
-});

package/lib/middleware/__tests__/yargsChecksMiddleware.test.d.ts

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

package/lib/middleware/__tests__/yargsChecksMiddleware.test.js

@@ -1,39 +0,0 @@
-import { EXIT_CODES } from '../../enums/exitCodes.js';
-import * as projectsConfig from '../../projects/config.js';
-import { performChecks } from '../yargsChecksMiddleware.js';
-import { uiLogger } from '../../ui/logger.js';
-import { commands } from '../../../lang/en.js';
-vi.mock('../../projects/config');
-const getIsInProjectSpy = vi.spyOn(projectsConfig, 'getIsInProject');
-const processExitSpy = vi.spyOn(process, 'exit');
-describe('lib/middleware/yargsChecksMiddleware', () => {
-    beforeEach(() => {
-        processExitSpy.mockImplementation(code => {
-            throw new Error(`Process.exit called with code ${code}`);
-        });
-    });
-    describe('performChecks()', () => {
-        it('should exit with error when running upload/watch inside a project', () => {
-            const argv = {
-                _: ['upload'],
-                src: './project',
-                $0: 'hs',
-            };
-            getIsInProjectSpy.mockReturnValue(true);
-            expect(() => performChecks(argv)).toThrow();
-            expect(processExitSpy).toHaveBeenCalledWith(EXIT_CODES.ERROR);
-            expect(uiLogger.error).toHaveBeenCalledWith(commands.generalErrors.srcIsProject(argv.src, argv._.toString()));
-        });
-        it('should return true when no checks fail', () => {
-            const argv = {
-                _: ['upload'],
-                src: './not-a-project',
-                $0: 'hs',
-            };
-            getIsInProjectSpy.mockReturnValue(false);
-            expect(performChecks(argv)).toBe(true);
-            expect(processExitSpy).not.toHaveBeenCalled();
-            expect(uiLogger.error).not.toHaveBeenCalled();
-        });
-    });
-});

package/lib/projects/__tests__/AppDevModeInterface.test.d.ts

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