Skip to content

test: 행사 서비스 조회 시나리오 추가#545

Open
dh2906 wants to merge 1 commit intostack/event-homefrom
stack/event-service-tests
Open

test: 행사 서비스 조회 시나리오 추가#545
dh2906 wants to merge 1 commit intostack/event-homefrom
stack/event-service-tests

Conversation

@dh2906
Copy link
Copy Markdown
Contributor

@dh2906 dh2906 commented Apr 14, 2026

🔍 개요

  • 행사 조회 서비스 로직 검증을 별도 stacked PR로 분리합니다.

🚀 주요 변경 내용

  • EventServiceTest를 추가합니다.
  • 부스맵, 미니 이벤트, 콘텐츠, 홈 응답의 매핑과 페이징 규칙을 검증합니다.
  • 서비스 레벨에서 조회 계약이 유지되는지 고정합니다.

💬 참고 사항

  • base PR: stack/event-home
  • 통합 테스트는 다음 PR에서 따로 쌓입니다.
  • pre-push hook 기준 checkstyleMain, compileJava를 통과했습니다.

✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

- stacked PR 크기를 유지하기 위해 event service 단위 검증을 통합 테스트와 분리한다
- 부스맵, 미니 이벤트, 콘텐츠 조회 매핑과 페이징 규칙을 서비스 레벨에서 먼저 고정한다
- 레포 전역 compile 이슈와 무관하게 event 도메인 조회 로직의 기대 응답을 리뷰할 수 있게 만든다
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 14, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2d588c18-6c40-4d2d-8252-8ff6b4fe9161

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch stack/event-service-tests

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

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant