Fixed breaking dto

src/auth/auth.e2e.spec.ts

@@ -41,20 +41,22 @@ describe('Auth Controller Tests', () => {
   it('should be able to sign up using email', async () => {
     const response = await app.inject({
       method: 'POST',
-      url: '/auth/send-verification-email',
+      url: '/auth/email',
       payload: {
         email: '[email protected]',
       },
     });
 
     expect(response.statusCode).toEqual(201);
-    expect(response.json().email).toEqual('[email protected]');
+    expect(response.json()).toEqual({
+      status: 'success',
+    });
   });
 
   it('should send verification code to email on sign up', async () => {
     await app.inject({
       method: 'POST',
-      url: '/auth/send-verification-email',
+      url: '/auth/email',
       payload: {
         email: '[email protected]',
       },
@@ -101,7 +103,7 @@ describe('Auth Controller Tests', () => {
     // Sign up
     await app.inject({
       method: 'POST',
-      url: '/auth/send-verification-email',
+      url: '/auth/email',
       payload: {
         email: '[email protected]',
       },

src/auth/service/auth.service.ts

@@ -220,22 +220,17 @@ export class AuthService {
       throw new NotFoundException('User not found');
     }
 
-    const updatedUser = await this.prisma.user.update({
-      where: {
-        email,
-      },
-      data: {
-        isEmailVerified: true,
-      },
-      select: {
-        id: true,
-        email: true,
-        name: true,
-        profilePictureUrl: true,
-        authType: true,
-        isEmailVerified: true,
-      },
-    });
+    const updatedUser = plainToClass(
+      UserResponseDto,
+      await this.prisma.user.update({
+        where: {
+          email,
+        },
+        data: {
+          isEmailVerified: true,
+        },
+      }),
+    );
 
     await this.prisma.verificationCode.delete({
       where: {

src/user/dto/user-response.dto.ts

@@ -1,14 +1,32 @@
 import { AuthType } from '@prisma/client';
-import { Exclude } from 'class-transformer';
+import { Exclude, Expose } from 'class-transformer';
 
 @Exclude()
 export default class UserResponseDto {
+  @Expose()
+  id: string;
+
+  @Expose()
   email: string;
+
+  @Expose()
   name: string;
+
+  @Expose()
   location: string;
+
+  @Expose()
   onboarded: boolean;
+
+  @Expose()
   profilePictureUrl: string;
+
+  @Expose()
   authType: AuthType;
+
+  @Expose()
   isEmailVerified: boolean;
+
+  @Expose()
   headline: string;
 }

src/user/service/user.service.ts

@@ -30,7 +30,7 @@ export class UserService {
   ) {}
 
   async getSelf(user: User) {
-    return user;
+    return plainToClass(UserResponseDto, user);
   }
 
   async updateSelf(user: User, dto: UpdateUserDto) {

src/user/user.e2e.spec.ts

@@ -54,7 +54,7 @@ describe('User Controller Tests', () => {
     expect(prisma).toBeDefined();
   });
 
-  it('should not be able to update itself if onboarding is not finished', async () => {
+  it('should be able to update itself if onboarding is not finished', async () => {
     await prisma.user.update({
       where: {
         email: '[email protected]',
@@ -75,7 +75,14 @@ describe('User Controller Tests', () => {
       },
     });
 
-    expect(response.statusCode).toBe(401);
+    const updatedUser = await prisma.user.findUnique({
+      where: {
+        email: '[email protected]',
+      },
+    });
+    expect(updatedUser.name).toBe('Jane Doe');
+
+    expect(response.statusCode).toBe(200);
   });
 
   it('should be able to get the current user', async () => {