-
Notifications
You must be signed in to change notification settings - Fork 7
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
[BE/TEST] 각종 테스트 #376
[BE/TEST] 각종 테스트 #376
Conversation
- Swagger 요청은 로깅 안되게 수정
- 회원 탈퇴 데이터 검증
resultActions.andExpect(status().isOk()); | ||
assertAll(() -> assertFalse(memberRepository.existsById(memberId))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
회원 탈퇴시 회원 이외에도 제거되는 데이터들도 검증에 포함되어 있으면 좋을 것 같습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그런데 굳이 이벤트까지 써가면서 결합도를 낮췄는데 테스트에서 결합도를 높일 필요가 있나 싶어요
Controller 테스트에서 이벤트가 잘 publish되는지 테스트도 하고 있어서 굳이 다른 데이터가 삭제 되었는지 확인 할필요 없지않나
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트의 목적이 해당 기능이 의도하는 데로 확인하는 거 아닌가요?
결합도가 올라간다고 통합 테스트시 해야갈 검증을 안 하는 거는 옳지 않다고 생각합니다.
📌 관련 이슈
closes #359
🔑 주요 변경사항