Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: 스터디/투두 생성 시에 잘못된 값으로 입력이 넘어와도 정상적으로 생성하는 버그를 수정한다 #245

Merged
merged 5 commits into from
Aug 8, 2023

Conversation

yujamint
Copy link
Member

@yujamint yujamint commented Aug 7, 2023

😋 작업한 내용

Request Dto에 입력값에 대한 유효성 검증 어노테이션을 추가했습니다.
Study와 Round 엔티티에 유효성 검사 로직을 추가했습니다.

🙏 PR Point

크게 2가지 변화가 있습니다. File Changes가 상당히 많기 때문에 커밋을 구분해서 보시는 게 편할 것 같습니다. 어려운 코드는 딱히 없으니 너무 경계하지 않으셔도 될 것 같습니다.

  1. Study 생성 시에 Request 객체의 startAt필드가 과거의 날짜를 입력받을 수 없다는 검증 로직을 추가했습니다. 이를 위해 startAt 필드를 LocalDate 타입으로 받고자 하였고, 그 과정에서 직렬화/역직렬화를 커스텀하는 JacksonConfig 클래스를 구현했습니다. 그러다 보니 Cucumber의 Given 절이 바뀌게 되었고 ... 상당히 많은 file changes가 생겼네요.
  1. 그 외로는 QA에서 포착된 스터디 제목, 투두 내용을 빈 칸으로 입력해도 정상 생성되는 버그를 잡기 위한 검증을 추가했습니다.

추가로 Jackson으로 LocalDate 타입 바인딩하기 위해 참고했던 레퍼런스 첨부하겠습니다.

👍 관련 이슈

@yujamint yujamint added backend🤎 백엔드 fix 기능 코드 버그 수정 jasmine👙 민트 labels Aug 7, 2023
@yujamint yujamint self-assigned this Aug 7, 2023
@github-actions
Copy link

github-actions bot commented Aug 7, 2023

Test Results

19 files  19 suites   11s ⏱️
64 tests 64 ✔️ 0 💤 0
65 runs  65 ✔️ 0 💤 0

Results for commit 2d7ca06.

♻️ This comment has been updated with latest results.

@yujamint yujamint merged commit 34572cb into BE/develop Aug 8, 2023
2 checks passed
@yujamint yujamint deleted the BE/fix/237 branch August 8, 2023 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend🤎 백엔드 fix 기능 코드 버그 수정 jasmine👙 민트
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants