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

refactor: 독서록 조회 기능 Service Layer dto 테스트 코드 작성 #26

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

dani820
Copy link
Collaborator

@dani820 dani820 commented Sep 19, 2024

이 PR을 통해 해결하려는 문제

  • 독서록 단 건 조회 기능 Service Layer 응답 dto에 대한 테스트 코드를 작성하여 코드 정확성을 보장하고 애플리케이션 안정성을 유지한다.

추가 및 변경사항

  • 각 dto 클래스에 대한 테스트 코드 작성
  • MemberDetailsOutput 내 auto increment와 매치되는 memberNo 필드 삭제, memberId 는 uuid 로 세팅할 것을 감안하여 작성
    (따로 PR 올릴 예정 PR 요청 완료)

참고(옵션)

  • Member 관련 도메인 클래스와 dto 클래스 수정 예정

체크리스트

  • 리뷰어를 지정하였는가
  • 코드가 오류나 경고없이 빌드되는가
  • 추가 및 변경된 사항에 대해 충분히 테스트 하였는가

- MemberDetailsOutput 필드 수정 감안하여 작성 -> memberNo 삭제 예정
@dani820 dani820 self-assigned this Sep 19, 2024
Copy link
Collaborator

@f-lab-bradley f-lab-bradley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

단순해 보이는 코드 이지만 필요한 테스트 들이에요.
다만 요런건 AI 같은걸로 코드 주면서 테스트 코드 만들어줘 하면 만들어 주니 잘 활용 해 보세요

@dani820 dani820 merged commit 380dddb into develop Sep 20, 2024
1 check passed
@dani820 dani820 deleted the refactor/25 branch September 20, 2024 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants