이력서 도메인 리팩토링

[strangehoon]

📕 이력서 도메인 리팩토링

📗 작업 내용

구현 내용 및 작업 했던 내역

• swagger 어노테이션 에러 반환값 코드 수정
• service단 for문을 스트림으로 리팩토링
• repository단 테스트 코드 리팩토링
• service단 테스트 코드 리팩토링

📘 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

• 어제까지 인턴 마감이었던게 있는데 오늘부터 포폴 열람하실거 같아서 여기까지 머지하고 controller 테스트 코드는 오늘이나 내일까지 올리겠습니다!!
• 현재 swagger보시면 예외일때 응답값이 이상한데 controller단 swagger 어노테이션에서 예외일때는 @Schema(implementation = ErrorResponse.class) 로 해줘야할거 같습니다.

[char-yb]

유효하지 않은 회원에 대해서 NotAuthorizedException을 NotFoundExcetion으로 하셨더라고요. 404는 보통 프론트엔드에서 찾을 수 없는 페이지일때 나타내는 에러 코드인데 401이 더 적합해보이지 않을까요??

[strangehoon]

인가 관련된 로직들은 컨트롤러 앞단 필터쪽에 따로 존재해서 따로 인가 관련된 에러 메세지가 나간다고 생각했습니다. service 레이어에서 user 객체를 찾는것은 인가를 위한 로직이 아니라 실제 user 객체를 활용하는 로직이라 판단해서 db에 리소스가 있는지 확인하는 NotFoundException을 사용하는게 좋다고 생각했습니다!! 404를 db에 리소스가 없을때 사용해도 괜찮지 않을까요??

[char-yb]

넵 듣고보니 그렇기도 하네요 확인했습니다~!