test : 그룸 멤버 전체 랭킹 API 테스트코드 추가 #61
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
* feat : 그룹멤버 Role추가 * refactor : 그룹멤버 Role 수정 * refactor : 그룹멤버 Role 수정 * refactor : 그룹멤버 Role 2차수정 * refactor : 그룹멤버 Role 3차수정 * refactor : 그룹멤버 Role 4차수정 * Merge pull request #48 * feat : 스터디 그룹 즐겨찾기 추가/취소 API 추가 * test : 스터디 그룹 즐겨찾기 API service, controller 테스트 작성 * refactor: User Role을 String enum으로 수정 (#54) * refactor : 그룹멤버 Role 최종수정 --------- Co-authored-by: rladmstn <[email protected]>
* feat : 스터디 그룹 목록 조회 API에서 즐겨찾기 카테고리 추가 * test : 스터디 그룹 목록 조회 API 테스트 코드 작성 스터디 그룹 목록에 bookmarked 카테고리 추가 후 controller, service 테스트 작성
rladmstn
requested changes
Aug 22, 2024
Comment on lines
395
to
396
| assertThat((result.get(1).getRank())).isEqualTo(2); |
There was a problem hiding this comment.
테스트에서 2등을 user, 1등을 user2로 둔 것 같은데 단순히 rank만 검증하지 말고 response에 대한 웬만한 필드는 검증하는게 좋다고 생각해요 GetRankingResponse로 들어갈 내용이 rank만 있을게 아닐테니까요!
There was a problem hiding this comment.
아아 사실 2위 1위 순서로 리스트에 넣은지라 중요한 함수인 정렬함수만 잘 돌아가면 된다고 생각해서 그렇게 했슴다.. 고칠게요!
There was a problem hiding this comment.
사실 저도 다 하는건 아닙니다만,, 그래도 어느정도 중요한 정보로 userNickname정도는? 해주면 좋지 않을까 싶네욥
|
테스트코드 뉴비절단기네 |
|
ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ테코에서 다 Re-request받는게 개 슬프네 진짜로...ㅠㅠㅠㅠㅠㅠㅠㅠㅠ |
|
컨플릭 해결은
|
rladmstn
approved these changes
Aug 22, 2024
Comment on lines
395
to
396
| assertThat((result.get(1).getRank())).isEqualTo(2); |
There was a problem hiding this comment.
사실 저도 다 하는건 아닙니다만,, 그래도 어느정도 중요한 정보로 userNickname정도는? 해주면 좋지 않을까 싶네욥
s-hwan
approved these changes
Aug 23, 2024
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.
📌 Related Issue
#52
🚀 Description
그룸 멤버 전체 랭킹 API 테스트코드를 만들어봤습니다
📢 Review Point
성공하는경우 , 그룹을 못찾아서 실패하는경우, 권한이 없어서 실패하는경우 세가지를 테스트 했습니다!
📚Etc (선택)