feat : 답변 수정 삭제 api (#61)

gffd94 · web-flow · commit 01394fae8850 · 2025-07-17T09:27:00.000+09:00
* 답변 수정/삭제 구현

* 답변 수정 삭제 테스트 작성
diff --git a/src/main/java/com/oronaminc/join/answer/api/AnswerController.java b/src/main/java/com/oronaminc/join/answer/api/AnswerController.java
@@ -3,6 +3,8 @@
 import com.oronaminc.join.answer.dto.AnswerGetResponse;
 import com.oronaminc.join.answer.service.AnswerService;
 import com.oronaminc.join.member.security.MemberDetails;
+import io.swagger.v3.oas.annotations.Operation;
+import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -20,6 +22,15 @@ public class AnswerController {
 
     private final AnswerService answerService;
 
+    @Operation(
+        summary = "답변 조회",
+        description = "답변 보기 클릭 시 질문에 대한 답변을 조회",
+        responses = {
+            @ApiResponse(responseCode = "200", description = "답변 조회 성공"),
+            @ApiResponse(responseCode = "400", description = "답변이 없는데 답변보기 버튼이 활성화 되어 잘못된 조회 접근")
+
+        }
+    )
     @GetMapping("/rooms/{roomId}/questions/{questionId}/answers")
     @ResponseStatus(HttpStatus.OK)
     public ResponseEntity<AnswerGetResponse> getAnswer(
diff --git a/src/main/java/com/oronaminc/join/answer/domain/Answer.java b/src/main/java/com/oronaminc/join/answer/domain/Answer.java
@@ -1,6 +1,5 @@
 package com.oronaminc.join.answer.domain;
 
-import com.oronaminc.join.answer.dto.AnswerCreateRequest;
 import com.oronaminc.join.global.entity.BaseEntity;
 import com.oronaminc.join.member.domain.Member;
 import com.oronaminc.join.question.domain.Question;
@@ -60,6 +59,10 @@ public static Answer create(Question question, Member member, String content) {
             .build();
     }
 
+    public void updataContent(String content) {
+        this.content = content;
+    }
+
     public Long incrementEmojiCount() {
         return ++this.emojiCount;
     }
diff --git a/src/main/java/com/oronaminc/join/answer/dto/AnswerCreateRequest.java b/src/main/java/com/oronaminc/join/answer/dto/AnswerCreateRequest.java
diff --git a/src/main/java/com/oronaminc/join/answer/dto/AnswerCreateResponse.java b/src/main/java/com/oronaminc/join/answer/dto/AnswerCreateResponse.java
@@ -2,6 +2,8 @@
 
 import com.oronaminc.join.global.dto.WriterDto;
 import io.swagger.v3.oas.annotations.media.Schema;
+import jakarta.validation.constraints.NotBlank;
+import jakarta.validation.constraints.Size;
 import java.time.LocalDateTime;
 import lombok.Builder;
 
@@ -16,6 +18,8 @@ public record AnswerCreateResponse(
     @Schema(description = "답변 ID", example = "11")
     Long answerId,
     @Schema(description = "답변 내용", example = "답변입니다.")
+    @NotBlank(message = "답변 내용을 입력해주시기 바랍니다.")
+    @Size(max = 300, message = "답변 내용은 최대 300자까지 입력할 수 있습니다.")
     String content,
     @Schema(description = "답변 내용에 대한 공감 수", example = "23")
     int emojiCount,
diff --git a/src/main/java/com/oronaminc/join/answer/dto/AnswerDeleteResponse.java b/src/main/java/com/oronaminc/join/answer/dto/AnswerDeleteResponse.java
@@ -0,0 +1,12 @@
+package com.oronaminc.join.answer.dto;
+
+import io.swagger.v3.oas.annotations.media.Schema;
+
+@Schema(description = "답변 삭제 응답 DTO")
+public record AnswerDeleteResponse(
+    Long answerId,
+    @Schema(description = "삭제 이벤트", example = "DELETE")
+    String event
+) {
+
+}
diff --git a/src/main/java/com/oronaminc/join/answer/dto/AnswerRequest.java b/src/main/java/com/oronaminc/join/answer/dto/AnswerRequest.java
@@ -0,0 +1,16 @@
+package com.oronaminc.join.answer.dto;
+
+import io.swagger.v3.oas.annotations.media.Schema;
+import jakarta.validation.constraints.NotBlank;
+import jakarta.validation.constraints.Size;
+
+@Schema(description = "답변 생성/수정 요청 DTO")
+public record AnswerRequest(
+    //TODO: 빈값 or " " (space) 처리
+    @NotBlank(message = "답변 내용을 입력해주시기 바랍니다.")
+    @Size(max = 300, message = "답변 내용은 최대 300자까지 입력할 수 있습니다.")
+    @Schema(description = "답변 내용", example = "답변입니다.")
+    String content
+) {
+
+}
diff --git a/src/main/java/com/oronaminc/join/answer/dto/AnswerUpdateResponse.java b/src/main/java/com/oronaminc/join/answer/dto/AnswerUpdateResponse.java
@@ -0,0 +1,17 @@
+package com.oronaminc.join.answer.dto;
+
+import io.swagger.v3.oas.annotations.media.Schema;
+import lombok.Builder;
+
+@Builder
+@Schema(description = "답변 수정 응답 DTO")
+public record AnswerUpdateResponse(
+    Long answerId,
+    @Schema(description = "수정 이벤트", example = "UPDATE")
+    String event,
+    @Schema(description = "수정된 내용", example = "수정된 답변입니다.")
+    String content
+
+) {
+
+}
diff --git a/src/main/java/com/oronaminc/join/answer/mapper/AnswerMapper.java b/src/main/java/com/oronaminc/join/answer/mapper/AnswerMapper.java
@@ -1,9 +1,10 @@
 package com.oronaminc.join.answer.mapper;
 
 import com.oronaminc.join.answer.domain.Answer;
-import com.oronaminc.join.answer.dto.AnswerCreateRequest;
+import com.oronaminc.join.answer.dto.AnswerRequest;
 import com.oronaminc.join.answer.dto.AnswerCreateResponse;
 import com.oronaminc.join.answer.dto.AnswerGetResponse;
+import com.oronaminc.join.answer.dto.AnswerUpdateResponse;
 import com.oronaminc.join.global.dto.WriterDto;
 import com.oronaminc.join.member.domain.Member;
 import com.oronaminc.join.question.domain.Question;
@@ -43,7 +44,16 @@ public static AnswerGetResponse toAnswerGetResponse(Answer answer, Long emojiCou
             .build();
     }
 
-    public static Answer toEntity(Question question, Member member, AnswerCreateRequest request) {
+    public static Answer toEntity(Question question, Member member, AnswerRequest request) {
         return Answer.create(question, member, request.content());
     }
+
+    public static AnswerUpdateResponse toAnswerUpdateResponse(Answer answer) {
+        return AnswerUpdateResponse.builder()
+            .answerId(answer.getId())
+            .event("UPDATE")
+            .content(answer.getContent())
+            .build();
+    }
+
 }
diff --git a/src/main/java/com/oronaminc/join/answer/service/AnswerService.java b/src/main/java/com/oronaminc/join/answer/service/AnswerService.java
@@ -4,12 +4,11 @@
 
 import com.oronaminc.join.answer.dao.AnswerRepository;
 import com.oronaminc.join.answer.domain.Answer;
-import com.oronaminc.join.answer.dto.AnswerCreateRequest;
+import com.oronaminc.join.answer.dto.AnswerRequest;
 import com.oronaminc.join.answer.dto.AnswerGetResponse;
 import com.oronaminc.join.answer.mapper.AnswerMapper;
 import com.oronaminc.join.emoji.domain.TargetType;
 import com.oronaminc.join.emoji.service.EmojiReader;
-import com.oronaminc.join.global.exception.ErrorCode;
 import com.oronaminc.join.global.exception.ErrorException;
 import com.oronaminc.join.member.domain.Member;
 import com.oronaminc.join.member.service.MemberReader;
@@ -38,7 +37,7 @@ public class AnswerService {
 
     @Transactional
     public Answer create(Long roomId, Long memberId, Long questionId,
-        AnswerCreateRequest requestDto) {
+        AnswerRequest request) {
 
         Member member = memberReader.getById(memberId);
         Room room = roomReader.getById(roomId);
@@ -50,7 +49,7 @@ public Answer create(Long roomId, Long memberId, Long questionId,
             throw new ErrorException(BADREQUEST_DUPLICATION_ANSWER);
         }
 
-        Answer answer = AnswerMapper.toEntity(question, member, requestDto);
+        Answer answer = AnswerMapper.toEntity(question, member, request);
 
         answerRepository.save(answer);
 
@@ -71,12 +70,28 @@ public AnswerGetResponse getAnswer(Long roomId, Long questionId, Long memberId)
         return AnswerMapper.toAnswerGetResponse(answer, emojiCount, isEmojied);
     }
 
+    @Transactional
+    public Answer update(Long answerId, AnswerRequest request) {
+        Answer answer = answerReader.getById(answerId);
+
+        answer.updataContent(request.content());
+
+        return answer;
+    }
+
+    @Transactional
+    public void delete(Long answerId) {
+        Answer answer = answerReader.getById(answerId);
+        answerRepository.delete(answer);
+    }
+
+    @Transactional
     public void deleteByQuestion(Long questionId) {
         answerRepository.deleteByQuestionId(questionId);
     }
 
+    @Transactional
     public void deleteByQuestionList(List<Question> questions) {
         answerRepository.deleteByQuestionIn(questions);
     }
-
 }
diff --git a/src/main/java/com/oronaminc/join/answer/util/PermissionValidator.java b/src/main/java/com/oronaminc/join/answer/util/PermissionValidator.java
@@ -1,29 +1,61 @@
 package com.oronaminc.join.answer.util;
 
-import static com.oronaminc.join.global.exception.ErrorCode.NOT_FOUND_PARTICIPANT;
+import static com.oronaminc.join.global.exception.ErrorCode.UNAUTHORIZED_DELETE_ANSWER;
+import static com.oronaminc.join.global.exception.ErrorCode.UNAUTHORIZED_EDIT_ANSWER;
 import static com.oronaminc.join.global.exception.ErrorCode.UNAUTHORIZED_ROLE_ANSWER;
 
+import com.oronaminc.join.answer.domain.Answer;
+import com.oronaminc.join.answer.service.AnswerReader;
 import com.oronaminc.join.global.exception.ErrorException;
-import com.oronaminc.join.participant.dao.ParticipantRepository;
 import com.oronaminc.join.participant.domain.Participant;
 import com.oronaminc.join.participant.domain.ParticipantType;
+import com.oronaminc.join.participant.service.ParticipantReader;
+import com.oronaminc.join.question.domain.Question;
+import com.oronaminc.join.room.domain.Room;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Component;
 
 @Component
 @RequiredArgsConstructor
 public class PermissionValidator {
 
-    private final ParticipantRepository participantRepository;
+    private final ParticipantReader participantReader;
+    private final AnswerReader answerReader;
 
     public void validateAnswerPermission(Long roomId, Long memberId) {
-        Participant participant = participantRepository.findByRoomIdAndMemberId(roomId,memberId)
-            .orElseThrow(() -> new ErrorException(NOT_FOUND_PARTICIPANT));
+        // TODO: 생성시 조건 ( null 이면 안됨, " " 안됨, 팀원이나 발표자, 질문 작성자 본인만 생성가능 )
+        Participant participant = participantReader.getByRoomIdAndMemberId(roomId, memberId);
         ParticipantType type = participant.getParticipantType();
 
-        if(type == ParticipantType.GUEST){
+        if (type == ParticipantType.GUEST) {
             throw new ErrorException(UNAUTHORIZED_ROLE_ANSWER);
         }
     }
 
+    public void validateAnswerUpdatePermission(Long answerId, Long memberId) {
+        Answer answer = answerReader.getById(answerId);
+
+        if (!answer.getMember().getId().equals(memberId)) {
+            throw new ErrorException(UNAUTHORIZED_EDIT_ANSWER);
+        }
+    }
+
+    public void validateAnswerDeletePermission(Long answerId, Long memberId) {
+        Answer answer = answerReader.getById(answerId);
+
+        Room room = answer.getQuestion().getRoom();
+        Question question = answer.getQuestion();
+
+        Participant participant = participantReader.getByRoomIdAndMemberId(room.getId(), memberId);
+        ParticipantType type = participant.getParticipantType();
+
+        boolean isQuestionWriter = question.getMember().getId().equals(memberId);
+
+        if (!(isQuestionWriter || type == ParticipantType.TEAM
+            || type == ParticipantType.PRESENTER)) {
+            throw new ErrorException(UNAUTHORIZED_DELETE_ANSWER);
+        }
+
+    }
+
 }
diff --git a/src/main/java/com/oronaminc/join/global/exception/ErrorCode.java b/src/main/java/com/oronaminc/join/global/exception/ErrorCode.java
@@ -44,6 +44,8 @@ public enum ErrorCode {
     NOT_FOUND_EXIST_ANSWER("ANSWER-002", "해당 질문에 대한 답변이 존재하지 않습니다.", NOT_FOUND),
     NOT_FOUND_ANSWER("ANSWER-003", "답변이 존재하지 않습니다.", NOT_FOUND),
     BADREQUEST_DUPLICATION_ANSWER("ANSWER-004", "이미 답변한 질문입니다.", BAD_REQUEST),
+    UNAUTHORIZED_EDIT_ANSWER("ANSWER-005", "작성자가 아니면 해당 댓글을 수정할 수 없습니다.", UNAUTHORIZED),
+    UNAUTHORIZED_DELETE_ANSWER("ANSWER-006", "작성자 혹은 팀원, 발표자가 아니면 해당 댓글을 삭제할 수 없습니다.", UNAUTHORIZED),
 
 
     ACCESS_DENIED_SESSION("SESSION-1201", "접근 권한이 없습니다.", FORBIDDEN),
diff --git a/src/main/java/com/oronaminc/join/websocket/api/AnswerWebsocketController.java b/src/main/java/com/oronaminc/join/websocket/api/AnswerWebsocketController.java
@@ -1,15 +1,17 @@
 package com.oronaminc.join.websocket.api;
 
+import static com.oronaminc.join.global.exception.ErrorCode.UNAUTHORIZED_MEMBER;
+
 import com.oronaminc.join.answer.domain.Answer;
-import com.oronaminc.join.answer.dto.AnswerCreateRequest;
 import com.oronaminc.join.answer.dto.AnswerCreateResponse;
+import com.oronaminc.join.answer.dto.AnswerDeleteResponse;
+import com.oronaminc.join.answer.dto.AnswerRequest;
+import com.oronaminc.join.answer.dto.AnswerUpdateResponse;
 import com.oronaminc.join.answer.mapper.AnswerMapper;
 import com.oronaminc.join.answer.service.AnswerService;
 import com.oronaminc.join.answer.util.PermissionValidator;
-import com.oronaminc.join.member.dao.MemberRepository;
-import com.oronaminc.join.participant.dao.ParticipantRepository;
-import com.oronaminc.join.question.dao.QuestionRepository;
-import com.oronaminc.join.room.dao.RoomRepository;
+import com.oronaminc.join.global.exception.ErrorException;
+import jakarta.validation.Valid;
 import java.security.Principal;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
@@ -32,10 +34,10 @@ public class AnswerWebsocketController {
     public AnswerCreateResponse create(
         @DestinationVariable Long roomId,
         @DestinationVariable Long questionId,
-        @Payload AnswerCreateRequest request,
+        @Payload @Valid AnswerRequest request,
         Principal principal
     ) {
-        Long memberId = Long.valueOf(principal.getName());
+        Long memberId = getMemberId(principal);
 
         permissionValidator.validateAnswerPermission(roomId, memberId);
 
@@ -46,4 +48,45 @@ public AnswerCreateResponse create(
         return AnswerMapper.toAnswerCreateResponse(answer);
     }
 
+    @MessageMapping("/answers/{answerId}/update")
+    @SendTo("/topic/rooms/{roomId}/answers")
+    public AnswerUpdateResponse update(
+        @DestinationVariable Long answerId,
+        @Payload @Valid AnswerRequest request,
+        Principal principal
+    ) {
+
+        Long memberId = getMemberId(principal);
+
+        permissionValidator.validateAnswerUpdatePermission(answerId, memberId);
+
+        Answer answer = answerService.update(answerId, request);
+
+        return AnswerMapper.toAnswerUpdateResponse(answer);
+    }
+
+    @MessageMapping("/answers/{answerId}/delete")
+    @SendTo("/topic/rooms/{roomId}/answers")
+    public AnswerDeleteResponse delete(
+        @DestinationVariable Long roomId,
+        @DestinationVariable Long questionId,
+        @DestinationVariable Long answerId,
+        Principal principal
+    ) {
+        Long memberId = getMemberId(principal);
+
+        permissionValidator.validateAnswerDeletePermission(answerId, memberId);
+
+        answerService.delete(answerId);
+
+        return new AnswerDeleteResponse(answerId, "DELETE");
+    }
+
+    private Long getMemberId(Principal principal) {
+        if (principal == null) {
+            throw new ErrorException(UNAUTHORIZED_MEMBER);
+        }
+        return Long.valueOf(principal.getName());
+    }
+
 }
diff --git a/src/test/java/com/oronaminc/join/answer/api/PermissionValidTests.java b/src/test/java/com/oronaminc/join/answer/api/PermissionValidTests.java
diff --git a/src/test/java/com/oronaminc/join/answer/service/AnswerServiceTests.java b/src/test/java/com/oronaminc/join/answer/service/AnswerServiceTests.java
