Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BE] 27.07 급상승/급하락 상위 5개 종목 테스트 코드 작성 #237 #243

Merged
merged 3 commits into from
Dec 3, 2024

Conversation

uuuo3o
Copy link
Collaborator

@uuuo3o uuuo3o commented Dec 2, 2024

✅ 주요 작업

  • topfive 로직 테스트 코드 작성
    • 상/하위로 5개까지 받아오는지 확인
    • 종목명이 포함되어있는지 확인

💭 고민과 해결과정

  • 기존 코드였던 axios.get으로 하려고 하니 잘 되지 않아서 accessToken을 모킹하는 것과 동일하게 목데이터를 넣었다.

@uuuo3o uuuo3o added BE 백엔드 TEST 테스트 코드 작성 API API 구현 labels Dec 2, 2024
@uuuo3o uuuo3o requested review from jinddings and sieunie December 2, 2024 07:39
@uuuo3o uuuo3o self-assigned this Dec 2, 2024
@uuuo3o uuuo3o linked an issue Dec 2, 2024 that may be closed by this pull request
@sieunie
Copy link
Collaborator

sieunie commented Dec 2, 2024

🟡 PR 이름이 이상한 것 같아요!

Copy link
Collaborator

@sieunie sieunie 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
Collaborator

Choose a reason for hiding this comment

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

🟢 목데이터 정말 크네요.........ㅋㅋㅋㅋㅋㅋ

@uuuo3o
Copy link
Collaborator Author

uuuo3o commented Dec 2, 2024

🟡 PR 이름이 이상한 것 같아요!

ㅋㅋㅋㅋㅋ 죄송해요 ... 시은님 pr 복붙해서 썼는데 바꾸질 않았네요...ㅎㅎ

@uuuo3o uuuo3o changed the title [BE] 27.07 주가 지수 서비스 테스트 코드 작성 #237 [BE] 27.07 급상승/급하락 상위 5개 종목 테스트 코드 작성 #237 Dec 2, 2024
@uuuo3o uuuo3o merged commit b53a05c into back/main Dec 3, 2024
2 checks passed
@uuuo3o uuuo3o deleted the test/topfive-#237 branch December 5, 2024 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API API 구현 BE 백엔드 TEST 테스트 코드 작성
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] 27.07 백엔드 테스트 코드 작성
2 participants