refactor : /users delete API auth -> user로 옮김

* 사용자를 삭제하는 API는 용도가 user 모듈가 더 가까워 옮김
boostcampwm2023 · Feb 23, 2024 · 3ba12e9 · 3ba12e9
1 parent c935cb6
commit 3ba12e9
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 26 deletions.
diff --git a/server/nest/src/auth/auth.controller.ts b/server/nest/src/auth/auth.controller.ts
@@ -60,12 +60,4 @@ export class AuthController {
     );
     return { userId: user.user_id };
   }
-
-  @Delete()
-  @UseGuards(AuthGuard())
-  @HttpCode(HTTP_STATUS_CODE.SUCCESS)
-  async deleteUser(@ReqUser() user: User): Promise<{ userId: string }> {
-    this.logger.log(`DELETE /users - nickname=${user.nickname}`);
-    return await this.authService.deleteUser(user);
-  }
 }
diff --git a/server/nest/src/auth/auth.service.ts b/server/nest/src/auth/auth.service.ts
@@ -195,22 +195,4 @@ export class AuthService {
 
     return true;
   }
-
-  async deleteUser(user: User): Promise<{ userId: string }> {
-    try {
-      await this.userRepository.deleteUser(user.user_id);
-
-      return { userId: user.user_id };
-    } catch (err) {
-      if (err instanceof CatchyException) {
-        throw err;
-      }
-
-      throw new CatchyException(
-        'SERVICE_ERROR',
-        HTTP_STATUS_CODE.SERVER_ERROR,
-        ERROR_CODE.SERVICE_ERROR,
-      );
-    }
-  }
 }
diff --git a/server/nest/src/user/user.controller.ts b/server/nest/src/user/user.controller.ts
@@ -12,6 +12,7 @@ import {
   Logger,
   Put,
   ValidationPipe,
+  Delete,
 } from '@nestjs/common';
 import { UserService } from './user.service';
 import { HTTP_STATUS_CODE } from 'src/codes/httpStatusCode.enum';
@@ -48,6 +49,14 @@ export class UserController {
     };
   }
 
+  @Delete()
+  @UseGuards(AuthGuard())
+  @HttpCode(HTTP_STATUS_CODE.SUCCESS)
+  async deleteUser(@ReqUser() user: User): Promise<{ userId: string }> {
+    this.logger.log(`DELETE /users - nickname=${user.nickname}`);
+    return await this.userService.deleteUser(user);
+  }
+
   @Get('duplicate/:name')
   @HttpCode(HTTP_STATUS_CODE.NOT_DUPLICATED_NICKNAME)
   async checkDuplicateNickname(

diff --git a/server/nest/src/user/user.service.ts b/server/nest/src/user/user.service.ts
@@ -41,6 +41,24 @@ export class UserService {
     }
   }
 
+  async deleteUser(user: User): Promise<{ userId: string }> {
+    try {
+      await this.userRepository.deleteUser(user.user_id);
+
+      return { userId: user.user_id };
+    } catch (err) {
+      if (err instanceof CatchyException) {
+        throw err;
+      }
+
+      throw new CatchyException(
+        'SERVICE_ERROR',
+        HTTP_STATUS_CODE.SERVER_ERROR,
+        ERROR_CODE.SERVICE_ERROR,
+      );
+    }
+  }
+
   async getRecentPlayedMusicByUserId(
     userId: string,
     count: number,