Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 18, 2025

📄 요약(Summary)

member에 대해 command가 일어나는 서비스 일 때, department를 fetch하도록 수정

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • member에 대해 command가 일어나는 서비스 일 때, department를 fetch하도록 수정
  • 팀 현황 조회 시에 department를 fetch하도록 수정

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

  • 테스트 코드에 불필요한 stub이 존재할 시에 오류가 발생
    • Mockito는 불필요한 코드를 제거하고 코드베이스를 깨끗하게 유지하는 것을 권장한다.
    • 테스트 실행 중에 설정한 스텁이 실제로 사용되지않았을 경우에 오류가 발생한다.

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #590

@joowojr joowojr added the 🐛 bug 버그를 수정합니다 label Feb 18, 2025
@joowojr joowojr self-assigned this Feb 18, 2025
@joowojr joowojr added 🚑 hotfix and removed 🐛 bug 버그를 수정합니다 labels Feb 18, 2025
@joowojr joowojr merged commit f48faa3 into develop Feb 18, 2025
1 check passed
@joowojr joowojr deleted the CLAP-441 branch February 18, 2025 06:14
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.

CLAP-451 회원 entity 조회 시 departement 조회 쿼리가 실행되지 않도록 리팩토링

2 participants