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/issue#160: 모임 조회 시 자신이 속한 모임만 조회하도록 수정 #165

Merged
merged 2 commits into from
Nov 11, 2024

Conversation

us4c0d3
Copy link
Contributor

@us4c0d3 us4c0d3 commented Nov 11, 2024

PR

✨ 작업 내용

  • 모임 조회를 할 때 자신이 속한 모임만 조회하도록 수정
  • 테스트 코드가 작동하도록 mocking 일부 수정

🔍 참고 사항

  • 참고해야 할 사항이 있다면 작성

⚠️ 의논 필요

  • 현재 테스트 코드를 보니 단위 테스트가 대부분 서비스 테스트이고 도메인 테스트가 없습니다
  • 이번 주 테스트 명세 작성 시 도메인 테스트 및 통합 테스트 위주로 작성하는 것이 좋을 것 같습니다.
  • 대부분 mocking을 이용한 것이므로 이 테스트가 진짜 유효한 테스트인지 생각할 필요가 있어보입니다

🐞현재 버그

  • 현재 버전에 버그가 있다면 작성

#️⃣ 연관 이슈(Git Close)

close #160


😊 리뷰 규칙을 지킵시다

코드 리뷰는 Pn룰에 따라 작성하기.
Reviewer가 피드백을 남길 때 Assignee에게 얼마나 해당 피드백에 대해 강조하고 싶은 지 표현하기 위한 규칙입니다.

  • P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
  • P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
  • P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@us4c0d3 us4c0d3 added 🐞 Bug 버그 발견 ✨ Feature 기능 개발 labels Nov 11, 2024
@us4c0d3 us4c0d3 self-assigned this Nov 11, 2024
Copy link

Test Results

74 tests   74 ✅  8s ⏱️
19 suites   0 💤
19 files     0 ❌

Results for commit 77c8733.

Copy link
Contributor

@Lucerna00 Lucerna00 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!
말씀처럼 서비스 외의 테스트를 추가해야겠네요

Copy link
Contributor

@dkswoals dkswoals left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!!
통합, 도메인 테스트 코드 작성 및
기존 단위 테스트 코드 검토 동의합니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Bug 버그 발견 ✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FIX: 모임 목록 조회 시 자신이 속한 모임 목록만 조회하도록 변경
3 participants