[fix] 일정 미정으로 생성된 모임의 일정 투표 현황 재조회 시 집계 결과 누락 버그 해결

[eujjong18]

📌 관련 이슈

• close [fix] 일정 미정으로 생성된 모임의 일정 투표 현황 재조회 시 집계 결과 누락 버그 해결 #142

✨ 작업 내용

• '일정 투표 현황 조회' 서비스 로직 수정
  • 조건 입력을 통해 일정을 확정하는 경우
    '일정 투표 현황 조회 API'를 두번째로 호출하는 시점부터 투표 집계 결과 없이 최종 확정된 일정만 보여주는 문제 해결

📸 스크린샷

• 모임 내 서로 다른 참여자로 '일정 투표 현황 조회' API 호출 테스트

Summary by CodeRabbit

• 버그 수정

  • 일정이 확정된 모임이라도 일정 투표 기록이 있으면 투표 결과를 정상적으로 집계해 표시합니다.
  • 모임 생성 시 일정이 확정되고 투표 기록이 없는 경우에는 기존처럼 확정된 일정을 즉시 제공합니다.
  • 투표로 확정된 일정은 조회 시 불필요하게 재확정하지 않고 기존 집계 결과를 그대로 반환합니다.

• 테스트

  • 일정 확정 방식과 투표 기록 유무에 따른 조회 동작을 검증하는 테스트를 보강했습니다.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: dbca6700-7b22-4fb3-98ed-699dc18b6428

📥 Commits

Reviewing files that changed from the base of the PR and between 0b2dbdf and 3ed76dd.

📒 Files selected for processing (3)

• src/main/java/com/moive/MoiveBE/domain/meeting/repository/DateVoteRepository.java
• src/main/java/com/moive/MoiveBE/domain/vote/service/VoteService.java
• src/test/java/com/moive/MoiveBE/domain/vote/service/VoteServiceTest.java

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

일정 투표 기록 존재 여부를 확인하는 저장소 메서드를 추가했습니다. 투표 기록이 있으면 VoteService가 투표 집계를 수행하고, 기록이 없을 때만 확정 일정을 즉시 반환합니다. 관련 테스트를 갱신하고 재조회 시나리오를 추가했습니다.

Changes

일정 투표 현황 조회

Layer / File(s)	Summary
투표 기록 기반 조회 흐름 src/main/java/com/moive/MoiveBE/domain/meeting/repository/DateVoteRepository.java, src/main/java/com/moive/MoiveBE/domain/vote/service/VoteService.java	DateVoteRepository에 existsByMeetingId(Long meetingId)를 추가했습니다. 일정이 확정되어도 투표 기록이 있으면 투표 집계 흐름으로 진행합니다.
조회 경로 테스트 검증 src/test/java/com/moive/MoiveBE/domain/vote/service/VoteServiceTest.java	모임 생성 시 확정된 일정의 투표 생략 경로를 갱신했습니다. 투표 기록이 있는 모임의 재조회에서 집계 결과 반환과 confirmSchedule 미호출을 검증합니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix · Severity of issue fixed: Low

Merge Risk: ⚪ Minimal · up to 3ed76

The endpoint now preserves vote aggregates when a meeting was finalized through voting, while retaining the skipped-vote response for meetings created with a fixed date. No merge-blocking risk is identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 42.86% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 3 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목은 일정 미정 모임의 일정 투표 현황 재조회 시 집계 결과가 누락되는 버그를 수정한다는 주요 변경 사항을 명확하게 설명합니다.
Description check	✅ Passed	PR 설명은 관련 이슈, 작업 내용, 테스트 화면을 포함합니다. 템플릿의 필수 섹션을 충족하며 변경 목적과 수정 범위를 명확하게 설명합니다.
Linked Issues check	✅ Passed	[#142] 요구사항을 충족합니다. VoteService는 일정이 확정된 모임이라도 DateVoteRepository.existsByMeetingId(...)가 true이면 확정 일정 즉시 반환 경로를 사용하지 않고 투표 집계 흐름을 수행합니다. 따라서 조건 입력으로 일정이 확정된 모임의 재조회에서도 투표 결과를 유지합니다. `VoteServiceT…
Out of Scope Changes check	✅ Passed	변경 범위는 [#142]의 일정 투표 현황 조회 로직과 그 동작을 검증하는 테스트로 제한됩니다. DateVoteRepository의 존재 여부 조회 추가, VoteService의 분기 수정, 관련 테스트 보완은 모두 이 버그 수정에 직접 연결됩니다. 관련 없는 변경은 확인되지 않습니다.

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}