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

[DEL] 필요 없는 order 관련 로직 및 필드 삭제 #120

Merged
merged 19 commits into from
Mar 3, 2024

Conversation

Jin409
Copy link
Contributor

@Jin409 Jin409 commented Mar 2, 2024

🌍 이슈 번호

📝 구현 내용

  • 응원톡 조회 시 order 필드 삭제
  • 게임 상세 조회 시 order 필드 삭제
    • order 필드 삭제에 따라서, 게임팀 순서대로 반환되는지 확인하는 테스트 추가
  • 게임 전체 조회 시 order 필드 삭제
  • 응원 횟수 조회 시 order 필드 삭제
  • 라인업 선수 조회 시 order 필드 삭제
    • order 필드 삭제에 따라서, 게임팀 순서대로 반환되는지 확인하는 테스트 추가

🍀 확인해야 할 부분

  • 인수 테스트의 경우, GameQueryAcceptanceTest 가 라인업 관련 로직을 포함하고 있어서 우선은 GameQueryServiceTest 에 라인업 테스트 로직도 포함해뒀어요. 그런데 query.application 에는 게임 조회 로직은 GameQueryService 에, 라인업 관련 로직은 LineupPlayerQueryService 에 포함되어 있어서 기준이 명확하지 않다는 생각이 드네요..! 어떻게 통일하는 게 좋을까요?
  • GameTeamServiceUtils 를 삭제할까요? 아니면 혹시 모르니.. 그대로 둘까요?

@Jin409 Jin409 changed the title Del/#119 order [DEL] 필요 없는 order 관련 로직 및 필드 삭제 Mar 2, 2024
@Jin409 Jin409 marked this pull request as ready for review March 2, 2024 15:43
@Jin409 Jin409 requested a review from ldk980130 March 2, 2024 15:43
@Jin409 Jin409 self-assigned this Mar 2, 2024
Copy link
Contributor

@ldk980130 ldk980130 left a comment

Choose a reason for hiding this comment

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

삭제 작업 수고하셨습니다!

GameTeamServiceUtils가 아직 남아있던데 확인해보니 더이상 사용되는 곳이 없더라구요. 이 클래스까지 지우고 나서 머지하면 될 것 같습니다!

@Jin409 Jin409 merged commit cd5a0e6 into main Mar 3, 2024
1 check passed
@Jin409 Jin409 deleted the del/#119-order branch March 3, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DEL] 필요 없는 order 관련 로직 및 필드 삭제
2 participants