feat: implement updateProfile method in UserService and corresponding tests

Author: tomast1337

server/src/user/user.controller.ts

@@ -1,11 +1,13 @@
-import { Controller, Inject, Get, Patch, Body, Query } from '@nestjs/common';
-import { ApiTags, ApiBearerAuth, ApiOperation } from '@nestjs/swagger';
-import { GetRequestToken, validateUser } from '@server/GetRequestUser';
+import { Body, Controller, Get, Inject, Patch, Query } from '@nestjs/common';
+import { ApiBearerAuth, ApiOperation, ApiTags } from '@nestjs/swagger';
+
 import { PageQueryDTO } from '@shared/validation/common/dto/PageQuery.dto';
 import { GetUser } from '@shared/validation/user/dto/GetUser.dto';
 import { UpdateUsernameDto } from '@shared/validation/user/dto/UpdateUsername.dto';
 import { UpdateUserProfileDto } from '@shared/validation/user/dto/UpdateUserProfile.dto';
 
+import { GetRequestToken, validateUser } from '@server/GetRequestUser';
+
 import { UserDocument } from './entity/user.entity';
 import { UserService } from './user.service';
 

server/src/user/user.service.spec.ts

@@ -8,6 +8,7 @@ import { Model } from 'mongoose';
 
 import { User, UserDocument } from './entity/user.entity';
 import { UserService } from './user.service';
+import { UpdateUserProfileDto } from '@shared/validation/user/dto/UpdateUserProfile.dto';
 
 const mockUserModel = {
   create: jest.fn(),
@@ -18,6 +19,7 @@ const mockUserModel = {
   exec: jest.fn(),
   select: jest.fn(),
   countDocuments: jest.fn(),
+  findOneAndUpdate: jest.fn(),
 };
 
 describe('UserService', () => {
@@ -332,6 +334,7 @@ describe('UserService', () => {
         username: 'testuser',
         save: jest.fn().mockReturnThis(),
       } as unknown as UserDocument;
+
       const body = { username: 'newuser' };
 
       jest.spyOn(service, 'usernameExists').mockResolvedValue(false);
@@ -358,4 +361,72 @@ describe('UserService', () => {
       );
     });
   });
+
+  describe('updateProfile', () => {
+    it('should update the user profile successfully', async () => {
+      const user = {
+        _id: 'userId',
+        description: 'old description',
+        socialLinks: {},
+        username: 'oldUsername',
+      } as unknown as UserDocument;
+
+      const body: UpdateUserProfileDto = {
+        description: 'new description',
+        socialLinks: { github: 'https://github.com/newuser' },
+        username: 'newUsername',
+      };
+
+      const updatedUser = {
+        ...user,
+        ...body,
+      };
+
+      jest
+        .spyOn(userModel, 'findOneAndUpdate')
+        .mockResolvedValue(updatedUser as any);
+
+      const result = await service.updateProfile(user, body);
+
+      expect(result).toEqual(updatedUser);
+
+      expect(userModel.findOneAndUpdate).toHaveBeenCalledWith(
+        { _id: user._id },
+        user,
+        { new: true },
+      );
+    });
+
+    it('should update only provided fields', async () => {
+      const user = {
+        _id: 'userId',
+        description: 'old description',
+        socialLinks: {},
+        username: 'oldUsername',
+      } as unknown as UserDocument;
+
+      const body: UpdateUserProfileDto = {
+        description: 'new description',
+      };
+
+      const updatedUser = {
+        ...user,
+        description: 'new description',
+      };
+
+      jest
+        .spyOn(userModel, 'findOneAndUpdate')
+        .mockResolvedValue(updatedUser as any);
+
+      const result = await service.updateProfile(user, body);
+
+      expect(result).toEqual(updatedUser);
+
+      expect(userModel.findOneAndUpdate).toHaveBeenCalledWith(
+        { _id: user._id },
+        user,
+        { new: true },
+      );
+    });
+  });
 });

server/src/user/user.service.ts

@@ -151,6 +151,14 @@ export class UserService {
   }
 
   public async updateProfile(user: UserDocument, body: UpdateUserProfileDto) {
-    throw new Error('Method not implemented.');
+    const { description, socialLinks, username } = body;
+
+    if (description) user.description = description;
+    if (socialLinks) user.socialLinks = socialLinks;
+    if (username) user.username = username;
+
+    return await this.userModel.findOneAndUpdate({ _id: user._id }, user, {
+      new: true,
+    });
   }
 }