BUG:Fix error in group update

Author: navnitms-kv

src/authorization/repository/group.repository.ts

@@ -1,5 +1,4 @@
 import { Injectable } from '@nestjs/common';
-import { UpdateGroupInput } from 'src/schema/graphql.schema';
 import { DataSource, In } from 'typeorm';
 import Group from '../entity/group.entity';
 import UserGroup from '../entity/userGroup.entity';
@@ -15,11 +14,8 @@ export class GroupRepository extends BaseRepository<Group> {
     return this.findOneBy({ id });
   }
 
-  async updateGroupById(
-    id: string,
-    updateGroupInput: UpdateGroupInput,
-  ): Promise<boolean> {
-    const result = await this.update(id, updateGroupInput);
+  async updateGroupById(id: string, name?: string): Promise<boolean> {
+    const result = await this.update(id, { name });
 
     return result.affected === 1;
   }

src/authorization/service/group.service.ts

@@ -123,7 +123,10 @@ export class GroupService implements GroupServiceInterface {
    * @returns
    */
   async updateGroup(id: string, group: UpdateGroupInput): Promise<Group> {
-    const updatedGroup = await this.groupRepository.updateGroupById(id, group);
+    const updatedGroup = await this.groupRepository.updateGroupById(
+      id,
+      group?.name,
+    );
 
     if (!updatedGroup) {
       throw new GroupNotFoundException(id);

src/config/migration.config.ts

@@ -1,6 +1,7 @@
 import { DataSource } from 'typeorm';
 import { SnakeNamingStrategy } from 'typeorm-naming-strategies';
-
+import dotenv from 'dotenv';
+dotenv.config();
 /**
  * Uses env params to configure TypeORM database library
  */

test/authorization/repository/group.repository.test.ts

@@ -80,7 +80,7 @@ describe('test Group repository', () => {
 
       const result = await groupRepository.updateGroupById(
         VALID_GROUP_ID,
-        updateGroupInput,
+        updateGroupInput.name,
       );
 
       expect(updateMock).toBeCalledWith(VALID_GROUP_ID, {

test/authorization/service/group.service.test.ts

@@ -277,7 +277,7 @@ describe('test Group Service', () => {
 
       expect(updateGroupByIdMock).toBeCalledWith(
         'ae032b1b-cc3c-4e44-9197-276ca877a7f8',
-        input,
+        input.name,
       );
       expect(getGroupByIdMock).toBeCalledWith(
         'ae032b1b-cc3c-4e44-9197-276ca877a7f8',
@@ -366,7 +366,7 @@ describe('test Group Service', () => {
       } catch (error) {
         expect(updateGroupByIdMock).toBeCalledWith(
           'ae032b1b-cc3c-4e44-9197-276ca877a7f9',
-          input,
+          input.name,
         );
         expect(getGroupByIdMock).not.toBeCalled();