[SSF-102] Add endpoints to update user info

apps/backend/src/users/dtos/update-user-info.dto.ts

@@ -0,0 +1,30 @@
+import {
+  IsString,
+  IsPhoneNumber,
+  IsOptional,
+  IsNotEmpty,
+  MaxLength,
+} from 'class-validator';
+
+export class updateUserInfoDto {
+  @IsOptional()
+  @IsString()
+  @IsNotEmpty()
+  @MaxLength(255)
+  firstName?: string;
+
+  @IsOptional()
+  @IsString()
+  @IsNotEmpty()
+  @MaxLength(255)
+  lastName?: string;
+
+  @IsOptional()
+  @IsString()
+  @IsNotEmpty()
+  @IsPhoneNumber('US', {
+    message:
+      'phone must be a valid phone number (make sure all the digits are correct)',
+  })
+  phone?: string;
+}

apps/backend/src/users/users.controller.spec.ts

@@ -1,12 +1,13 @@
-import { BadRequestException } from '@nestjs/common';
 import { UsersController } from './users.controller';
 import { UsersService } from './users.service';
 import { User } from './users.entity';
 import { Role } from './types';
 import { userSchemaDto } from './dtos/userSchema.dto';
-
 import { Test, TestingModule } from '@nestjs/testing';
 import { mock } from 'jest-mock-extended';
+import { updateUserInfoDto } from './dtos/update-user-info.dto';
+import { Pantry } from '../pantries/pantries.entity';
+import { BadRequestException } from '@nestjs/common';
 import { AuthenticatedRequest } from '../auth/authenticated-request';
 
 const mockUserService = mock<UsersService>();
@@ -79,24 +80,42 @@ describe('UsersController', () => {
     });
   });
 
-  describe('PUT /:id/role', () => {
-    it('should update user role with valid role', async () => {
-      const updatedUser = { ...mockUser1, role: Role.ADMIN };
+  describe('PATCH /:id', () => {
+    it('should update user info with valid information', async () => {
+      const updatedUser = {
+        ...mockUser1,
+        firstName: 'UpdatedFirstName',
+        lastName: 'UpdatedLastName',
+        phone: '777-777-7777',
+      };
       mockUserService.update.mockResolvedValue(updatedUser as User);
 
-      const result = await controller.updateRole(1, Role.ADMIN);
+      const updateUserSchema: updateUserInfoDto = {
+        firstName: 'UpdatedFirstName',
+        lastName: 'UpdatedLastName',
+        phone: '777-777-7777',
+      };
+      const result = await controller.updateInfo(1, updateUserSchema);
 
       expect(result).toEqual(updatedUser);
-      expect(mockUserService.update).toHaveBeenCalledWith(1, {
-        role: Role.ADMIN,
-      });
+      expect(mockUserService.update).toHaveBeenCalledWith(1, updateUserSchema);
     });
 
-    it('should throw BadRequestException for invalid role', async () => {
-      await expect(controller.updateRole(1, 'invalid_role')).rejects.toThrow(
-        BadRequestException,
+    it('should throw BadRequestException when DTO is empty', async () => {
+      mockUserService.update.mockRejectedValue(
+        new BadRequestException(
+          'At least one field must be provided to update',
+        ),
+      );
+
+      const updateUserSchema: updateUserInfoDto = {};
+
+      await expect(controller.updateInfo(1, updateUserSchema)).rejects.toThrow(
+        new BadRequestException(
+          'At least one field must be provided to update',
+        ),
       );
-      expect(mockUserService.update).not.toHaveBeenCalled();
+      expect(mockUserService.update).toHaveBeenCalledWith(1, updateUserSchema);
     });
   });
 

apps/backend/src/users/users.controller.ts

@@ -4,16 +4,15 @@ import {
   Get,
   Param,
   ParseIntPipe,
-  Put,
   Post,
-  BadRequestException,
   Body,
+  Patch,
   Req,
 } from '@nestjs/common';
 import { UsersService } from './users.service';
 import { User } from './users.entity';
-import { Role } from './types';
 import { userSchemaDto } from './dtos/userSchema.dto';
+import { updateUserInfoDto } from './dtos/update-user-info.dto';
 import { AuthenticatedRequest } from '../auth/authenticated-request';
 import { JwtAuthGuard } from '../auth/jwt-auth.guard';
 import { UseGuards } from '@nestjs/common';
@@ -38,15 +37,12 @@ export class UsersController {
     return this.usersService.remove(userId);
   }
 
-  @Put('/:id/role')
-  async updateRole(
+  @Patch('/:id')
+  async updateInfo(
     @Param('id', ParseIntPipe) id: number,
-    @Body('role') role: string,
+    @Body() dto: updateUserInfoDto,
   ): Promise<User> {
-    if (!Object.values(Role).includes(role as Role)) {
-      throw new BadRequestException('Invalid role');
-    }
-    return this.usersService.update(id, { role: role as Role });
+    return this.usersService.update(id, dto);
   }
 
   @Post('/')

apps/backend/src/users/users.entity.ts

@@ -23,13 +23,25 @@ export class User {
   })
   role!: Role;
 
-  @Column()
+  @Column({
+    type: 'varchar',
+    name: 'first_name',
+    length: 255,
+  })
   firstName!: string;
 
-  @Column()
+  @Column({
+    type: 'varchar',
+    name: 'last_name',
+    length: 255,
+  })
   lastName!: string;
 
-  @Column()
+  @Column({
+    type: 'varchar',
+    name: 'email',
+    length: 255,
+  })
   email!: string;
 
   @Column({