Skip to content

Conversation

@doma17
Copy link
Contributor

@doma17 doma17 commented Dec 3, 2025

#️⃣ 연관된 이슈

#76

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • refactor: EventService 분할 및 Event 패키지 내부 정리
    • EventService를 EventQueryService와 EventCommandService로 분리했습니다.
    • EventStockProducerService, StockCheckJob에서 불필요한 주석, 내용을 삭제했습니다.
    • TicketingErrorCode에 PAGE_ILLEGAL_ARGUMENT 에러를 추가했습니다.
  • test: 테스트 코드 수정 및 추가
  • 관리 되지 않았던 테스트 코드들을 현재 비즈니스 로직에 맞게 수정했습니다. 또한, 분할 된 EventService에 맞게 EventServiceTest를 EventQueryServiceTest와 EventCommandServiceTest로 분할했습니다.

스크린샷 (선택)

image

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

develop 브랜치를 merge해와서 진행했는데, 지금 변경사항을 follow-up 못해서 테스트와 로직 간의 이상이 있다면 알려주세요.

doma17 and others added 10 commits October 16, 2025 17:49
feat: 이벤트 날짜/시간 응답 포맷을 한글로 변경
fix: 이벤트 응답 DTO에 Campus 값 누락 문제
fix: 이벤트가 시작되면 validationEvent가 항상 false인 문제
feat: 유저 티켓팅 정보 검증 조건에 name이 null이거나 길이가 2 이하일 경우 false 추가
feat: 티켓팅 엑셀 내보내기에서 상품 수령을 완료한 유저만 포함되도록 조건 추가
- EventService를 EventQueryService와 EventCommandService로 분리했습니다.
- EventStockProducerService, StockCheckJob에서 불필요한 주석, 내용을 삭제했습니다.
- TicketingErrorCode에 PAGE_ILLEGAL_ARGUMENT 에러를 추가했습니다.
관리 되지 않았던 테스트 코드들을 현재 비즈니스 로직에 맞게 수정했습니다. 또한, 분할 된 EventService에 맞게 EventServiceTest를 EventQueryServiceTest와 EventCommandServiceTest로 분할했습니다.
@doma17 doma17 self-assigned this Dec 3, 2025
@doma17 doma17 added the enhancement New feature or request label Dec 3, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 3, 2025

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.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch test/test-refactor

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.

@doma17 doma17 changed the title Test/test refactor test: 테스트 코드 수정 및 코드 구조 변경 Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants