Skip to content

Conversation

@chanwonlee
Copy link
Contributor

관련 Issue (필수)

주요 변경 사항 (필수)

  • TrainSeatQueryService 테스트 코드 추가

리뷰어 참고 사항

image 해당 테스트로 TrainSeatQueryService 테스트 커버리지 100% 달성하였습니다!

추가 정보

없음

PR 작성 체크리스트 (필수)

  • 제목이 Issue와 동일함을 확인했습니다.
  • 리뷰어를 지정했습니다.
  • 프로젝트를 연결했습니다.

Copy link
Contributor

@Yunsung-Jo Yunsung-Jo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다! 👍

Copy link
Contributor

@Jimin730 Jimin730 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

예외 케이스까지 꼼꼼하게 작성해주셔서 복잡한 로직의 흐름을 확인하기 좋았던 것 같습니다 !
고생하셨습니다 👍 👍

Copy link
Contributor

@Friox Friox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

케이스를 꼼꼼히 체크해서 커버리지 100%를 달성하셨네요,
고생하셨습니다! 👍👍

Copy link
Contributor

@Ogu1208 Ogu1208 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

꼼꼼한 테스트 코드를 보니 한결 마음이 편해지네요!
도와주셔서 감사합니다 ㅠㅠ 🥲 저도 예약쪽은 reservationService에 위임할 걸 그랬나 싶네요!
고생하셨습니다~ 👍👍

@chanwonlee chanwonlee force-pushed the test/220-train-seat-query branch from a97ab62 to b04061d Compare August 7, 2025 02:16
@chanwonlee chanwonlee merged commit 81f3d5b into develop Aug 7, 2025
3 checks passed
@chanwonlee chanwonlee deleted the test/220-train-seat-query branch August 7, 2025 02:18
Copy link
Contributor

@EndlessMilkyway EndlessMilkyway left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

성공과 예외에 대한 모든 케이스를 꼼꼼하게 테스트해주셨군요! 고생많으셨습니다! 👍

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

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[TEST] TrainSeatQueryService 테스트

7 participants