2단계 - 경로 조회 기능 #10
Open
2단계 - 경로 조회 기능 #10
Conversation
This file contains 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
test: 동일 객체의 필드끼리 비교할 때 usingRecursiveComparison() 사용하도록 개선
gmelon
commented
Jun 15, 2023
| addTestData(); | ||
| } | ||
|
|
||
| private void addTestData() { |
There was a problem hiding this comment.
통합 테스트 이고, Section Station Controller에서 호출되는 PathService의 동작까지 함께 테스트하기 위해 직접 API 호출을 통해 데이터를 넣어주고 있는데, 코드가 지저분해지는걸까? 싶기도 하고 클래스로 분리해야되는건가 싶기도 하고 고민이 되더라구요.
addTestData() 메서드에 대해 어떻게 생각하시는지 궁금해요!!
gmelon
commented
Jun 15, 2023
| import java.util.List; | ||
| import java.util.stream.Stream; | ||
|
|
||
| public class FindShortestPathParam { |
There was a problem hiding this comment.
PathService와 PathIntegrationTest의 ParameterizedTest에서 공통으로 사용되는 methodSource를 분리해보았는데 패키지가 이 곳이 적합할지가 고민이 됩니다. 어떻게 보시나요?!
gmelon
commented
Jun 15, 2023
| } | ||
|
|
||
| @PostMapping | ||
| public ResponseEntity<StationResponse> createStation(@RequestBody StationRequest stationRequest) { | ||
| StationResponse station = stationService.saveStation(stationRequest); | ||
| pathService.addVertex(station.getId()); | ||
|
|
There was a problem hiding this comment.
Section, Station Controller에서 PathService의 로직을 호출하는데 이에 대한 검증을 PathController 통합 테스트에서만 진행해도 괜찮을까요?? 아니면 Section과 Station 통합 테스트에서도 어떻게든 검증을 하는게 좋을까요??
통합 테스트 이기 때문에 Path 통합 테스트에서 Section/Station API를 호출하고 제대로된 경로 정보를 반환하는지 확인하는게 더 적절한가 싶기도 하고 고민이 되네요🥲🥲
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.
안녕하세요.
1단계 피드백 주신 내용과 2단계 요구사항 반영했습니다!
구현하면서 들었던 고민들은 따로 코드에 코멘트로도 질문 남겨놓았습니다.
리뷰 해주시는 내용 잘 활용하겠습니다!!
감사합니다 :)
아 그리고 현재 PathIntegrationTest와 PathServiceTest에서 테스트 데이터로 넣어주고 있는 Line, Station, Section 정보들 개발하면서 보기 쉽도록 정리해 사용했었는데 혹시 몰라 같이 첨부드립니다!
