[TEST] 열차 조회 서비스 테스트(TrainSearchApplicationService, TrainSearchService) #235
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Ogu1208
requested review from
EndlessMilkyway,
Friox,
Jimin730,
Yunsung-Jo and
chanwonlee
chanwonlee
approved these changes
Aug 7, 2025
Contributor
chanwonlee
left a comment
chanwonlee
left a comment
There was a problem hiding this comment.
열차 조회 서비스 테스트 코드 확인했습니다. 해당 부분이 검색 조건도 복잡하고 다양한 케이스가 있어서 테스트 작성하기 어려웠을 텐데 꼼꼼히 테스트 코드 작성해서 커버리지를 거의 100% 근접하게 달성한게 대단하신것 같습니다! 고생하셨습니다!
src/test/java/com/sudo/railo/train/application/TrainSearchServiceTest.java
Outdated
Show resolved
Hide resolved
Friox
approved these changes
Aug 7, 2025
Contributor
Friox
left a comment
Friox
left a comment
There was a problem hiding this comment.
열차 부분에서도 고려해야 할 케이스가 정말 많았을 텐데, 거의 100%에 가까운 커버리지를 달성하신 것이 정말 대단하다고 생각합니다!
고생하셨습니다! 👍👍
Jimin730
approved these changes
Aug 7, 2025
Contributor
Jimin730
left a comment
Jimin730
left a comment
There was a problem hiding this comment.
기능 변경된 부분과 테스트 코드 확인했습니다!
테스트 코드 내에 record 타입을 따로 만들어서 여러가지 시나리오를 한 번에 테스트 할 수 있군요...
저도 다음에 참고해서 적용해보고싶네요!!
고생하셨습니다 👍 👍
test: 입석 관련 테스트 메서드명 수정 (#201)
Ogu1208
force-pushed
the
test/201-train-search-service
branch
from
ffc9782 to
5f92551
Compare
EndlessMilkyway
approved these changes
Aug 7, 2025
Contributor
EndlessMilkyway
left a comment
EndlessMilkyway
left a comment
There was a problem hiding this comment.
테스트 코드 뿐만 아니라 비즈니스 영역에도 대대적인 수정이 이루어졌군요!
민아님께서 테스트 코드를 작성하신 부분이 이 프로젝트에서 가장 복잡한 부분이라고 생각됩니다...!
고생하셨습니다👍
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
관련 Issue (필수)
주요 변경 사항 (필수)
테스트
TrainSearchApplicationServiceTest: Application Service 통합 테스트TrainSearchServiceSeatStatusTest: 좌석 상태 로직 검증TrainSearchServiceOverlapReservationTest: 다구간 예약 겹침 로직 검증TrainSearchServiceTest: 기본 검색 기능 테스트TrainSearchValidationTest: DTO 입력 검증 테스트추가로, 좌석 상태 및 기존 예약의 구간 겹침 판단 검증을 위해 아래 Helper 클래스에 로직을 추가했습니다.
ReservationTestHelperTrainTestHelper: 현실적인 열차 구성(2+2, 2+1 좌석 배열로 직접 객차 개수, 행 개수 지정) 생성 메서드 추가기능 변경
TrainCarListResponse에trainScheduleId필드 추가 : 연관 데이터 추적리뷰어 참고 사항
추가 정보
없음
PR 작성 체크리스트 (필수)