[BE] 27.07 주가 지수 서비스 테스트 코드 작성 #237 #240
Merged
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
uuuo3o
approved these changes
Dec 2, 2024
| }); | ||
|
|
||
| it('주가 지수 차트 조회 API에서 정상적인 데이터를 조회한 경우, 형식에 맞춰 정상적으로 반환한다.', async () => { | ||
| (axios.get as jest.Mock).mockImplementation((url: string) => { |
There was a problem hiding this comment.
🟢 요 쪽 코드 한번만 설명해주실 수 있나요? axios를 써서 하려니까 잘 모르겠어서.. 저는 accessToken mock 하는거랑 동일하게 jest로 데이터를 넣었거든요
There was a problem hiding this comment.
저도 완벽히 이해하고 쓴 코드는 아니라서... 일단 설명 드리면 axios.get 메소드를 뒤에 있는 콜백 함수로 모킹하는 방식입니다. 근데 여기서 차트 데이터 가지고 오는 api도 get 메소드를 쓰고, 값 가지고 오는 api도 get 메소드를 쓰니까 뒤에 url로 구분해서 다른 값 리턴하도록 구현했어요!
There was a problem hiding this comment.
아~~~~!!!
둘 다 get 메서드로 가져오는데, url은 다르니까! 아 좋습니당!!! 감사합니다 ㅎㅎㅎ
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.
✅ 주요 작업