fix: tests

Author: Dobrunia

src/integrations/github/__mocks__/service.ts

@@ -0,0 +1,5 @@
+export const deleteInstallationMock = jest.fn().mockResolvedValue(undefined);
+
+export const GitHubService = jest.fn().mockImplementation(() => ({
+  deleteInstallation: deleteInstallationMock,
+}));

src/integrations/github/routes.ts

@@ -522,15 +522,15 @@ export function createGitHubRouter(factories: ContextFactories): express.Router
      * just log query parameters and respond with 200 without signature validation.
      */
     if (req.method !== 'POST') {
+      // eslint-disable-next-line @typescript-eslint/camelcase, camelcase
       const { code, installation_id, setup_action, state, ...restQuery } = req.query as Record<string, unknown>;
 
+      // eslint-disable-next-line @typescript-eslint/camelcase, camelcase
       if (code || installation_id || state || setup_action) {
-        // eslint-disable-next-line @typescript-eslint/camelcase, camelcase
         log('info', `${WEBHOOK_LOG_PREFIX}Received non-POST request on /webhook with OAuth-like params`, {
-          // eslint-disable-next-line @typescript-eslint/camelcase, camelcase
           code,
-          installation_id,
-          setup_action,
+          installation_id, // eslint-disable-line @typescript-eslint/camelcase, camelcase
+          setup_action, // eslint-disable-line @typescript-eslint/camelcase, camelcase
           state,
           query: restQuery,
         });

src/resolvers/project.js

@@ -421,7 +421,7 @@ module.exports = {
          */
         const taskManager = project.taskManager;
 
-        if (taskManager && taskManager.type === 'github' && taskManager.config?.installationId) {
+        if (taskManager && taskManager.type === 'github' && taskManager.config && taskManager.config.installationId) {
           const githubService = new GitHubService();
 
           await githubService.deleteInstallation(taskManager.config.installationId);

test/integrations/github-routes.test.ts

@@ -480,7 +480,7 @@ describe('GitHub Routes - /integration/github/connect', () => {
 
       expect(response.status).toBe(302);
       expect(response.body).toContain('http://localhost:8080/');
-      expect(response.body).toContain('error=Missing+or+invalid+OAuth+code');
+      expect(response.body).toContain('apiError=Missing+or+invalid+OAuth+code');
     });
 
     it('should redirect with error when state is missing', async () => {
@@ -501,7 +501,7 @@ describe('GitHub Routes - /integration/github/connect', () => {
 
       expect(response.status).toBe(302);
       expect(response.body).toContain('http://localhost:8080/');
-      expect(response.body).toContain('error=Missing+or+invalid+state');
+      expect(response.body).toContain('apiError=Missing+or+invalid+state');
     });
 
     it('should redirect with error when state is invalid or expired', async () => {
@@ -525,7 +525,7 @@ describe('GitHub Routes - /integration/github/connect', () => {
 
       expect(response.status).toBe(302);
       expect(response.body).toContain('http://localhost:8080/');
-      expect(response.body).toContain('error=Invalid+or+expired+state');
+      expect(response.body).toContain('apiError=Invalid+or+expired+state');
       expect(mockGetState).toHaveBeenCalledWith(state);
     });
 

test/resolvers/project.test.ts

@@ -3,6 +3,11 @@ import { ObjectId } from 'mongodb';
 import { ProjectDBScheme, ProjectTaskManagerConfig } from '@hawk.so/types';
 import { ResolverContextWithUser } from '../../src/types/graphql';
 import { ApolloError, UserInputError } from 'apollo-server-express';
+
+jest.mock('../../src/integrations/github/service');
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const { deleteInstallationMock, GitHubService } = require('../../src/integrations/github/service');
+
 // @ts-expect-error - CommonJS module, TypeScript can't infer types properly
 import projectResolverModule from '../../src/resolvers/project';
 
@@ -141,6 +146,8 @@ describe('Project Resolver - Task Manager Mutations', () => {
       )) as { taskManager: ProjectTaskManagerConfig | null };
 
       expect(context.factories.projectsFactory.findById).toHaveBeenCalledWith(mockProject._id.toString());
+      expect(GitHubService).toHaveBeenCalledTimes(1);
+      expect(deleteInstallationMock).toHaveBeenCalledWith('123456');
       expect(mockProject.updateProject).toHaveBeenCalledWith({
         taskManager: null,
       });
@@ -217,6 +224,8 @@ describe('Project Resolver - Task Manager Mutations', () => {
         context
       )) as { taskManager: ProjectTaskManagerConfig | null };
 
+      expect(GitHubService).not.toHaveBeenCalled();
+      expect(deleteInstallationMock).not.toHaveBeenCalled();
       expect(mockProject.updateProject).toHaveBeenCalledWith({
         taskManager: null,
       });