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

[Feat/15] 회원 차단 API 구현 및 테스트 #17

Merged
merged 8 commits into from
Dec 6, 2024
Merged

Conversation

Eunjin3395
Copy link
Member

@Eunjin3395 Eunjin3395 commented Dec 6, 2024

🚀 개요

회원 차단 API 구현 및 테스트

⏳ 작업 상세 내용

  • 회원 차단 API 구현
  • MemberValidator 구현
  • 회원 및 차단 관련 Exception, ErrorCode 추가
  • 컨트롤러 테스트 기본 설정을 위한 ControllerTestSupport 추가
  • 컨트롤러 테스트 작성
  • 서비스 통합 테스트 작성
  • test application.yml에 환경변수 값 추가

📝 더 꼼꼼히 봐야할 부분

  • 기존에는 엔티티 생성해야할 때 서비스 코드에서 builder().build()를 하고 연관관계도 세팅해줘야 하면 set ~~ 하면서 엔티티 생성+매핑만 몇줄씩 차지했었는데 그 부분을 좀 수정했습니다. 엔티티마다 create 메소드를 만들고 그 안에서 builder로 엔티티 생성, 연관관계 매핑까지 하도록 바꿨습니다,, 이렇게 해도 괜찮을까요??

🔍 반드시 참고해야하는 변경 사항

  • 없을 경우 체크

@Eunjin3395 Eunjin3395 requested a review from rimi3226 December 6, 2024 09:53
@Eunjin3395 Eunjin3395 self-assigned this Dec 6, 2024
@Eunjin3395 Eunjin3395 linked an issue Dec 6, 2024 that may be closed by this pull request
3 tasks
Copy link
Contributor

@rimi3226 rimi3226 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@Eunjin3395 Eunjin3395 merged commit 9f4b6d8 into develop Dec 6, 2024
@Eunjin3395 Eunjin3395 deleted the feat/15 branch December 6, 2024 15:10
@hzee97 hzee97 self-requested a review December 9, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ [Feat] 회원 차단 API 구현 및 테스트
2 participants