Merge pull request #123 from ASAP-Lettering/ASAP-423

ASAP-423 사용자 회원 탈퇴 사유 저장 컬럼 추가

Author: tlarbals824

Application-Module/src/main/kotlin/com/asap/application/user/port/in/DeleteUserUsecase.kt

@@ -5,5 +5,6 @@ interface DeleteUserUsecase {
 
     data class Command(
         val userId: String,
+        val reason: String = ""
     )
 }

Application-Module/src/main/kotlin/com/asap/application/user/service/UserCommandService.kt

@@ -69,7 +69,7 @@ class UserCommandService(
         userManagementPort
             .getUserNotNull(DomainId(command.userId))
             .apply {
-                this.delete()
+                this.delete(command.reason)
                 userManagementPort.save(this)
             }.also {
                 userAuthManagementPort.getNotNull(it.id).apply {

Bootstrap-Module/src/main/kotlin/com/asap/bootstrap/web/user/api/UserApi.kt

@@ -68,6 +68,7 @@ interface UserApi {
     )
     fun deleteUser(
         @AccessUser userId: String,
+        @RequestBody(required = false) request: UnregisterUserRequest?,
     )
 
     @Operation(summary = "내 정보 조회")

Bootstrap-Module/src/main/kotlin/com/asap/bootstrap/web/user/controller/UserController.kt

@@ -40,10 +40,11 @@ class UserController(
         )
     }
 
-    override fun deleteUser(userId: String) {
+    override fun deleteUser(userId: String, request: UnregisterUserRequest?) {
         deleteUserUsecase.delete(
             DeleteUserUsecase.Command(
                 userId = userId,
+                reason = request?.reason.orEmpty(),
             ),
         )
     }

Bootstrap-Module/src/main/kotlin/com/asap/bootstrap/web/user/dto/UnregisterUserRequest.kt

@@ -0,0 +1,6 @@
+package com.asap.bootstrap.web.user.dto
+
+data class UnregisterUserRequest(
+    val reason: String? = null
+) {
+}

Bootstrap-Module/src/test/kotlin/com/asap/bootstrap/integration/user/UserApiIntegrationTest.kt

@@ -5,9 +5,12 @@ import com.asap.application.user.port.out.UserManagementPort
 import com.asap.bootstrap.IntegrationSupporter
 import com.asap.bootstrap.web.user.dto.LogoutRequest
 import com.asap.bootstrap.web.user.dto.RegisterUserRequest
+import com.asap.bootstrap.web.user.dto.UnregisterUserRequest
 import com.asap.bootstrap.web.user.dto.UpdateBirthdayRequest
 import com.asap.domain.common.DomainId
 import io.kotest.matchers.comparables.shouldBeGreaterThan
+import io.kotest.matchers.shouldBe
+import org.junit.jupiter.api.DisplayName
 import org.junit.jupiter.api.Nested
 import org.junit.jupiter.api.Test
 import org.springframework.http.MediaType
@@ -189,23 +192,51 @@ class UserApiIntegrationTest(
         }
     }
 
-    @Test
-    fun deleteUser() {
-        // given
-        val userId = userMockManager.settingUser()
-        userMockManager.settingUserAuth(userId = userId)
-        val accessToken = jwtMockManager.generateAccessToken(userId)
+    @Nested
+    @DisplayName("deleteUser")
+    inner class DeleteUser{
+        @Test
+        fun deleteUser() {
+            // given
+            val userId = userMockManager.settingUser()
+            userMockManager.settingUserAuth(userId = userId)
+            val accessToken = jwtMockManager.generateAccessToken(userId)
 
-        // when
-        val response =
-            mockMvc.delete("/api/v1/users") {
-                contentType = MediaType.APPLICATION_JSON
-                header("Authorization", "Bearer $accessToken")
+            // when
+            val response =
+                mockMvc.delete("/api/v1/users") {
+                    contentType = MediaType.APPLICATION_JSON
+                    header("Authorization", "Bearer $accessToken")
+                }
+
+            // then
+            response.andExpect {
+                status { isOk() }
             }
+        }
 
-        // then
-        response.andExpect {
-            status { isOk() }
+        @Test
+        fun deleteUser_with_reason(){
+            // given
+            val userId = userMockManager.settingUser()
+            userMockManager.settingUserAuth(userId = userId)
+            val accessToken = jwtMockManager.generateAccessToken(userId)
+            val request = UnregisterUserRequest("reason")
+
+            // when
+            val response =
+                mockMvc.delete("/api/v1/users") {
+                    contentType = MediaType.APPLICATION_JSON
+                    content = objectMapper.writeValueAsString(request)
+                    header("Authorization", "Bearer $accessToken")
+                }
+
+            // then
+            response.andExpect {
+                status { isOk() }
+            }
+            val user = userManagementPort.findById(DomainId(userId))
+            user!!.unregisterReason shouldBe request.reason
         }
     }
 

Domain-Module/src/main/kotlin/com/asap/domain/user/entity/User.kt

@@ -17,6 +17,7 @@ class User(
     var onboardingAt: LocalDateTime?,
     createdAt: LocalDateTime,
     updatedAt: LocalDateTime,
+    var unregisterReason: String? = null,
 ) : Aggregate<User>(id, createdAt, updatedAt) {
     companion object {
         fun create(
@@ -34,9 +35,10 @@ class User(
             }
     }
 
-    fun delete() {
+    fun delete(reason: String) {
         this.profileImage = "UNKNOWN"
         this.birthday = null
+        this.unregisterReason = reason
 
         registerEvent(UserEvent.UserDeletedEvent(this))
         updateTime()

Infrastructure-Module/Persistence/src/main/kotlin/com/asap/persistence/jpa/user/UserMapper.kt

@@ -28,6 +28,7 @@ object UserMapper {
             onboardingAt = user.onboardingAt,
             createdAt = user.createdAt,
             updatedAt = user.updatedAt,
+            unregisterReason = user.unregisterReason,
         )
 
     fun toUser(userEntity: UserEntity): User =
@@ -46,6 +47,7 @@ object UserMapper {
             onboardingAt = userEntity.onboardingAt,
             createdAt = userEntity.createdAt,
             updatedAt = userEntity.updatedAt,
+            unregisterReason = userEntity.unregisterReason,
         )
 
     fun toUserAuthEntity(userAuth: UserAuth): UserAuthEntity =

Infrastructure-Module/Persistence/src/main/kotlin/com/asap/persistence/jpa/user/entity/UserEntity.kt

@@ -17,6 +17,7 @@ class UserEntity(
     onboardingAt: LocalDateTime?,
     createdAt: LocalDateTime,
     updatedAt: LocalDateTime,
+    unregisterReason: String? = null,
 ) : AggregateRoot<UserEntity>(id, createdAt, updatedAt) {
     @Column(nullable = false)
     val username: String = username
@@ -35,4 +36,9 @@ class UserEntity(
     val birthday: LocalDate? = birthday
 
     val onboardingAt: LocalDateTime? = onboardingAt
+
+    @Column(
+        columnDefinition = "varchar(1000)",
+    )
+    val unregisterReason: String? = unregisterReason
 }

Infrastructure-Module/Persistence/src/main/resources/db/V1_19__alter_user_table_add_unregister_reason.sql

@@ -0,0 +1 @@
+alter table user add column unregister_reason varchar(1000);