test: 문의방 검색 기준선 테스트 추가#585
Conversation
- 채팅 리팩토링을 작은 PR 단위로 진행하기 위해 admin 문의방 검색 정책을 먼저 회귀 테스트로 고정 - 관리자가 문의방 멤버가 아니어도 사용자 문의방을 검색할 수 있어야 하는 기존 접근 정책을 명시 - 후속 QueryRepository 분리 과정에서 admin direct 조회 조건이 누락되는 회귀를 막기 위한 안전망을 추가
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
🧪 JaCoCo Coverage Report (Changed Files)
|
There was a problem hiding this comment.
Pull request overview
채팅 리팩토링(#579/#580) 진행 중 회귀를 막기 위해, 관리자가 문의방 멤버가 아니어도 사용자 문의방을 검색할 수 있어야 한다는 기존 정책을 ChatApiTest 통합 테스트로 기준선 고정하는 PR입니다.
Changes:
ChatApiTest에 “관리자(멤버십 없음)도 사용자 문의방 검색 가능” 회귀 테스트 추가- 문의방 생성 → 사용자 메시지 남김(‘사용자 응답 방’ 조건 충족) → 다른 admin으로 검색 성공을 검증
- 문의방 멤버가
SYSTEM_ADMIN + 사용자2명으로 유지되는지 함께 검증
🔍 개요
🚀 주요 변경 내용
ChatApiTest에 admin 문의방 검색 회귀 테스트를 추가했습니다.SYSTEM_ADMIN + 일반 사용자2명으로 유지되는지도 함께 확인합니다.💬 참고 사항
CI=true ./gradlew test --tests 'gg.agit.konect.integration.domain.chat.ChatApiTest$SearchChats.searchChatsIncludesInquiryRoomForAdminWithoutMembership' --rerun-tasks✅ Checklist (완료 조건)