Skip to content

Comments

[feat/#432] 모임 멤버 조회 API 마지막 운동일 필드 추가 및 테스트 작성#468

Merged
Dimo-2562 merged 6 commits intodevelopfrom
feat/#432
Feb 21, 2026
Merged

[feat/#432] 모임 멤버 조회 API 마지막 운동일 필드 추가 및 테스트 작성#468
Dimo-2562 merged 6 commits intodevelopfrom
feat/#432

Conversation

@Dimo-2562
Copy link
Contributor

❤️ 기능 설명

모임 멤버 조회 API (GET /api/parties/{partyId}/members) 응답에 멤버별 마지막 운동일 추가

  • MemberDetail DTO에 lastExerciseDate(LocalDate) 필드 추가
  • N+1 방지를 위해 멤버 ID 리스트로 한 번에 마지막 운동일 조회 (IN + GROUP BY)
  • 서비스 레이어 추상화 레벨 통일을 위해 getLastExerciseDateMap() 메서드 분리

swagger 테스트 성공 결과 스크린샷 첨부
image


연결된 issue

연결된 issue를 자동으로 닫기 위해 아래 {이슈넘버}를 입력해주세요.

close #432



🩷 Approve 하기 전 확인해주세요!

  • 통합 테스트 진행을 위해 테스트 컨테이너를 사용하였습니다.
  • 통합 테스트 코드 작성 시 IntegrationTest 클래스를 상속받아주시면 감사하겠습니다.
  • 리포지토리 단위 테스트를 할 정도의 복잡한 쿼리가 없어 서비스 단위 테스트만 진행하였습니다.

✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 테스트 결과 사진을 넣었는가?
  • 이슈넘버를 적었는가?

Copy link
Contributor

@kanghana1 kanghana1 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~! 시간이 된다면 테스트 설정이나 방식을 노션에 문서화해줘도 좋을 거 같습니당

public class IntegrationTestConfig {

private static final MySQLContainer<?> mysql =
new MySQLContainer<>("mysql:8.0.36");
Copy link
Contributor

Choose a reason for hiding this comment

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

버전들은 상수로 관리해줘도 좋을 거 같습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

버전을 딱 여기서만 관리해서 가독성이 이게 좋다고 판단했던거 같아요. 추후에 코드가 길어지고 버전 정보를 재사용하게 된다면 상수 관리 고려해보겠습니다! 감사합니다.

@Dimo-2562 Dimo-2562 merged commit 51e40e9 into develop Feb 21, 2026
1 check passed
@Dimo-2562 Dimo-2562 deleted the feat/#432 branch February 21, 2026 12:50
@Dimo-2562
Copy link
Contributor Author

고생하셨습니다~! 시간이 된다면 테스트 설정이나 방식을 노션에 문서화해줘도 좋을 거 같습니당

이거 진행하겠습니다!

@Dimo-2562 Dimo-2562 self-assigned this Feb 25, 2026
@Dimo-2562 Dimo-2562 added the 🛠️ FEAT This issue or pull request already exists label Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠️ FEAT This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 모임에서 멤버 정보 조회시 마지막 운동 정보도 조회하도록 변경

2 participants