Conversation
- 공지 목록, 상세 조회, 읽음 이력, 생성/수정/삭제 정책을 코드 기준으로 정리 - notice 작업 시 대학별 공지 격리와 read history 중복 방지, unread count 연계를 함께 확인하도록 기준화 - 문서 claim 중 목록 조회가 읽음 이력을 만들지 않는 정책을 통합 테스트로 고정
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 14 minutes and 11 seconds. ⌛ 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 (2)
✨ 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
공지(Notice) 도메인의 정책(목록/상세 조회, 읽음 이력, CRUD, unread count)을 코드 기준으로 문서화하고, “공지 목록 조회는 읽음 이력을 생성하지 않는다”는 정책을 통합 테스트로 고정하는 PR입니다.
Changes:
공지 목록 조회가CouncilNoticeReadHistory를 생성하지 않는지 검증하는 통합 테스트 추가- read history count 검증 로직을 테스트 helper 메서드로 공통화
- 공지 도메인 정책/동작/테스트 기준을
AGENTS.md로 문서화
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/test/java/gg/agit/konect/integration/domain/notice/NoticeApiTest.java | 목록 조회 시 read history 미생성 정책을 통합 테스트로 고정하고, read history count 조회 로직을 helper로 정리 |
| src/main/java/gg/agit/konect/domain/notice/AGENTS.md | 공지 도메인의 핵심 정책/동작/회귀 테스트 포인트를 코드 기준으로 정리한 가이드 문서 추가 |
🔍 개요
🚀 주요 변경 내용
src/main/java/gg/agit/konect/domain/notice/AGENTS.md를 추가해 공지 도메인의 주요 상태, 기능 동작, 테스트 기준을 문서화했습니다.CouncilNoticeReadHistory를 생성하지 않는지 검증하는 통합 테스트를 추가했습니다.💬 참고 사항
CI=true ./gradlew test --tests 'gg.agit.konect.integration.domain.notice.*'git diff --check, push 전 hook의checkstyleMain및compileJava./gradlew checkstyleTest는 기존 club/notification/chat 테스트의 line length 위반으로 실패했으며, 이번 변경 파일에서 새로 발생한 위반은 없습니다.✅ Checklist (완료 조건)