[BE] refactor: request DTO Validation 추가(#462)

[xxeol2]

📌 관련 이슈

• closed: [BE] Request DTO Validation을 추가한다. #462

✨ PR 세부 내용

[github-actions]

Unit Test Results

  70 files    70 suites   12s ⏱️
195 tests 195 ✔️ 0 💤 0 ❌
198 runs  198 ✔️ 0 💤 0 ❌

Results for commit c9de21b.

♻️ This comment has been updated with latest results.

[seokjin8678]

충돌나는거 수정했고, 예외 메시지 일괄적이게 변경했습니다!

[seokjin8678]

고생하셨습니다~
추가적으로 적용해야할 부분에 대해 커멘트 밑에 남기겠습니다!

[seokjin8678]

검증 어노테이션이 많아 가독성이 낮아져 컨벤션을 정해야 할 것 같네요!
제 생각에는 해당 방식이 좋아보이는데 어떠신가요?

public record FestivalCreateRequest(
    @NotBlank(message = "name 은 공백일 수 없습니다.")
    String name,

    @NotNull(message = "startDate 는 null 일 수 없습니다.")
    @DateTimeFormat(iso = ISO.DATE)
    LocalDate startDate,

    @NotNull(message = "endDate 는 null 일 수 없습니다.")
    @DateTimeFormat(iso = ISO.DATE)
    LocalDate endDate,

    String thumbnail) {
}

[BGuga]

동의보감입니다.

[xxeol2]

진행시켜!

[seokjin8678]

일부 Contorller에 @Valid 어노테이션이 빠져 있네요!

public class StudentController {
    ...
    @PostMapping("/send-verification")
    public ResponseEntity<Void> sendEmail(@Member Long memberId,
                                          @RequestBody StudentSendMailRequest request) {
        studentService.sendVerificationMail(memberId, request);
        return ResponseEntity.ok()
            .build();
    }
    ...
}

[seokjin8678]

지금 GlobalExceptionHandler 클래스에서 MethodArgumentNotValidException 핸들링을 수행할 때 어떠한 필드에 값이 유효하지 않은지 알려주지 않고 "잘못된 요청입니다." 메시지만 반환해주고 있네요!
예외에서 getFieldErrors() 메서드를 통해 어떠한 검증이 실패했는지 알려주는게 좋을 것 같아요!

@RestControllerAdvice
public class GlobalExceptionHandler extends ResponseEntityExceptionHandler { 
    ...
    @Override
    protected ResponseEntity<Object> handleMethodArgumentNotValid(MethodArgumentNotValidException e, HttpHeaders headers,
                                                                  HttpStatusCode status, WebRequest request) {
        return ResponseEntity.status(HttpStatus.BAD_REQUEST)
            .body(ErrorResponse.from(ErrorCode.INVALID_REQUEST_ARGUMENT));
    }
    ...
}

[BGuga]

일부 Contorller에 @Valid 어노테이션이 빠져 있네요!

public class StudentController {
    ...
    @PostMapping("/send-verification")
    public ResponseEntity<Void> sendEmail(@Member Long memberId,
                                          @RequestBody StudentSendMailRequest request) {
        studentService.sendVerificationMail(memberId, request);
        return ResponseEntity.ok()
            .build();
    }
    ...
}

[BGuga]

죄송합니다 에임 이슈로 잠시 닫았었네요 😅

[carsago]

꿋이에요

[seokjin8678]

고생하셨습니다!

[BGuga]

수고하셨습니다!