feat(modules/auth.controller): fix profile method

Rigtch · Dec 19, 2023 · a24a128 · a24a128
1 parent adcc16c
commit a24a128
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 13 deletions.
diff --git a/src/modules/auth/auth.controller.spec.ts b/src/modules/auth/auth.controller.spec.ts
@@ -164,8 +164,17 @@ describe('AuthController', () => {
   })
 
   test('should return profile', async () => {
-    vi.spyOn(authService, 'profile').mockResolvedValue(profileMock)
-
-    expect(await authController.profile('123')).toEqual(profileMock)
+    const accessToken = '123'
+
+    const profileSpy = vi
+      .spyOn(authService, 'profile')
+      .mockResolvedValue(profileMock)
+    const findOneByProfileIdSpy = vi
+      .spyOn(usersRepository, 'findOneByProfileId')
+      .mockResolvedValue(userMock)
+
+    expect(await authController.profile(accessToken)).toEqual(userMock)
+    expect(profileSpy).toHaveBeenCalledWith(accessToken)
+    expect(findOneByProfileIdSpy).toHaveBeenCalledWith(profileMock.id)
   })
 })
diff --git a/src/modules/auth/auth.controller.ts b/src/modules/auth/auth.controller.ts
@@ -76,16 +76,16 @@ export class AuthController {
       spotifyProfile.id
     )
 
-    if (!foundUser && refreshToken) {
-      const profile = await this.profilesService.create(spotifyProfile)
+    if (refreshToken) {
+      if (!foundUser) {
+        const profile = await this.profilesService.create(spotifyProfile)
 
-      await this.usersRepository.createUser({
-        profile,
-        refreshToken,
-      })
-    }
+        await this.usersRepository.createUser({
+          profile,
+          refreshToken,
+        })
+      }
 
-    if (refreshToken && accessToken) {
       return {
         url: `${this.configService.get(
           CLIENT_CALLBACK_URL
@@ -119,7 +119,9 @@ export class AuthController {
   @ApiOkResponse({
     description: "User's profile has been succesfully found",
   })
-  profile(@Token() accessToken: string) {
-    return this.authService.profile(accessToken)
+  async profile(@Token() accessToken: string) {
+    const { id } = await this.authService.profile(accessToken)
+
+    return this.usersRepository.findOneByProfileId(id)
   }
 }