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

feat: findSumOfPointBySamePartAndGeneration 테스트 추가 #338

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

kseysh
Copy link
Member

@kseysh kseysh commented Jul 28, 2024

📝 PR Summary

SoptampPointRepository의 findSumOfPointBySamePartAndGeneration의 테스트를 추가하였습니다.
이를 통해 기존 findSumOfPointBySamePartAndGeneration에 문제가 있다는 것을 확인하였습니다.
현재는 soptampUser의 part값을 사용하지 않지만, 이를 이용해 soptampUser의 파트와 기수가 같은 soptampPoint의 point 합을 구하는 것을 파악하였고, 사용되지 않는 soptampUser의 part값이 혼란을 발생시킬 수 있다는 생각이 들었습니다.

따라서 아래와 같이 수정하고자 합니다.

soptamp 개선안 (5기에 soptamp가 사용된다면 변경)

=> soptampUser의 part를 활용한다, soptampUser의 generation 값을 삭제한다.
=> soptampUser의 닉네임에 파트를 추가하는 것이 아닌 soptampUser의 part값을 활용한다. (soptamp관련 정보는 매 기수마다 초기화한다.)
추가적으로, totalPoint를 삭제한다.(그 대신 soptampPoint.points 정보를 활용한다.) #333

🌴 Works

  • findSumOfPointBySamePartAndGeneration 테스트 추가

🌱 Related Issue

#332

@kseysh kseysh added the ✨ Feat 새로운 피쳐 생성 label Jul 28, 2024
@kseysh kseysh requested a review from ozzing July 28, 2024 12:18
@kseysh kseysh self-assigned this Jul 28, 2024
Copy link

height bot commented Jul 28, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

…o refactor/#332-improvement-soptamp-ranking-api
@kseysh
Copy link
Member Author

kseysh commented Jul 29, 2024

전반적으로 테스트 세팅을 변경해야할 것 같아요... 실제 DB를 사용하다보니 duplicate id 오류가 발생하는 부분이 있네요.
h2로 변경하는 방법 등 방법을 다시 고민해보고 머지하도록 하겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 피쳐 생성 size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant