@hubspot/cli 7.10.0 → 7.11.0-experimental.0

package/lib/projects/__tests__/deploy.test.js

@@ -3,14 +3,16 @@ import { validateBuildIdForDeploy, logDeployErrors, handleProjectDeploy, } from
 import { uiLogger } from '../../ui/logger.js';
 import { commands } from '../../../lang/en.js';
 import { PROJECT_ERROR_TYPES } from '../../constants.js';
-import { deployProject } from '@hubspot/local-dev-lib/api/projects';
+import { deployProjectV1, deployProjectV2, } from '@hubspot/local-dev-lib/api/projects';
 import { pollDeployStatus } from '../pollProjectBuildAndDeploy.js';
 // Mock external dependencies
 vi.mock('../../ui/logger.js');
 vi.mock('@hubspot/local-dev-lib/api/projects');
+vi.mock('@hubspot/local-dev-lib/config');
 vi.mock('../pollProjectBuildAndDeploy.js');
 const mockUiLogger = vi.mocked(uiLogger);
-const mockDeployProject = vi.mocked(deployProject);
+const mockDeployProjectV1 = vi.mocked(deployProjectV1);
+const mockDeployProjectV2 = vi.mocked(deployProjectV2);
 const mockPollDeployStatus = vi.mocked(pollDeployStatus);
 describe('lib/projects/deploy', () => {
     beforeEach(() => {
@@ -122,12 +124,12 @@ describe('lib/projects/deploy', () => {
                 source: 'HUBSPOT_USER',
                 subdeployStatuses: [],
             };
-            mockDeployProject.mockResolvedValue({
+            mockDeployProjectV2.mockResolvedValue({
                 data: mockDeployResponseData,
             });
             mockPollDeployStatus.mockResolvedValue(mockDeployResult);
             const deploy = await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
-            expect(mockDeployProject).toHaveBeenCalledWith(targetAccountId, projectName, buildId, useV2Api, force);
+            expect(mockDeployProjectV2).toHaveBeenCalledWith(targetAccountId, projectName, buildId, force);
             expect(deploy).toEqual(mockDeployResult);
         });
         it('handles blocked deploy with warnings', async () => {
@@ -147,7 +149,7 @@ describe('lib/projects/deploy', () => {
                     },
                 ],
             };
-            mockDeployProject.mockResolvedValue({
+            mockDeployProjectV2.mockResolvedValue({
                 data: mockBlockedResponse,
             });
             await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
@@ -170,7 +172,7 @@ describe('lib/projects/deploy', () => {
                     },
                 ],
             };
-            mockDeployProject.mockResolvedValue({
+            mockDeployProjectV2.mockResolvedValue({
                 data: mockBlockedResponse,
             });
             await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
@@ -189,7 +191,7 @@ describe('lib/projects/deploy', () => {
                     },
                 ],
             };
-            mockDeployProject.mockResolvedValue({
+            mockDeployProjectV2.mockResolvedValue({
                 data: mockBlockedResponse,
             });
             await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
@@ -197,18 +199,18 @@ describe('lib/projects/deploy', () => {
             expect(mockUiLogger.log).toHaveBeenCalledWith(commands.project.deploy.errors.deployIssueComponentGeneric('component-1', 'module'));
         });
         it('handles general deploy failure', async () => {
-            mockDeployProject.mockResolvedValue({ data: null });
+            mockDeployProjectV2.mockResolvedValue({ data: null });
             const deploy = await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
             expect(mockUiLogger.error).toHaveBeenCalledWith(commands.project.deploy.errors.deploy);
             expect(deploy).toBeUndefined();
         });
         it('handles undefined deploy response', async () => {
-            mockDeployProject.mockResolvedValue({ data: undefined });
+            mockDeployProjectV2.mockResolvedValue({ data: undefined });
             const deploy = await handleProjectDeploy(targetAccountId, projectName, buildId, useV2Api, force);
             expect(mockUiLogger.error).toHaveBeenCalledWith(commands.project.deploy.errors.deploy);
             expect(deploy).toBeUndefined();
         });
-        it('passes correct parameters to deployProject', async () => {
+        it('passes correct parameters to deployProjectV1', async () => {
             const mockDeployResponseData = {
                 id: 'deploy-123',
                 buildResultType: 'DEPLOY_QUEUED',
@@ -216,14 +218,15 @@ describe('lib/projects/deploy', () => {
                     status: 'http://status-url',
                 },
             };
-            mockDeployProject.mockResolvedValue({
+            mockDeployProjectV1.mockResolvedValue({
                 data: mockDeployResponseData,
             });
             mockPollDeployStatus.mockResolvedValue({});
-            await handleProjectDeploy(targetAccountId, projectName, buildId, false, true);
-            expect(mockDeployProject).toHaveBeenCalledWith(targetAccountId, projectName, buildId, false, // useV2Api
+            await handleProjectDeploy(targetAccountId, projectName, buildId, false, // isV2Project
             true // force
             );
+            expect(mockDeployProjectV1).toHaveBeenCalledWith(targetAccountId, projectName, buildId, true // force
+            );
         });
     });
 });