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

37 정산 view api #39

Merged
merged 11 commits into from
Jul 24, 2024
Merged

37 정산 view api #39

merged 11 commits into from
Jul 24, 2024

Conversation

seongjunnoh
Copy link
Collaborator

📝 요약

메뉴의 정산 홈으로 들어왔을때 필요한 데이터들을 프론트단으로 보내주는 api 입니다
유저가 해당 스페이스 내에서 요청한 정산 중, 현재 진행중인 정산 리스트 정보와
유저가 해당 스페이스 내에서 요청받은 정산 중, 현재 진행중인 정산 리스트 정보를 response 값으로 보내줍니다.
서비스단의 코드를 junit5와 Mockito 를 이용하여 테스트하였고, 테스트가 잘 작동되는것을 확인했습니다.
다만, 레포지토리단의 코드는 아직 테스트를 해보지 않았습니다.
추후에 정산을 직접 생성하는 api를 개발한 후, 실제 db에 데이터가 잘 들어가는지 확인하거나, 레포지토리단의 테스트코드도 추가해보겠습니다.

이슈 번호 : #37

🔖 변경 사항

  • 정산 관련 컨트롤러, 서비스, 레포지토리 클래스 생성 (계층 구조 생성)
  • request로 넘어오는 spaceId에 해당하는 스페이스 엔티티를 찾기위해 SpaceUtils 클래스 정의
  • jwt로 확인할 수 있는 유저가 request로 넘어오는 스페이스에 속하는 유저인지를 확인하기 위해 UserSpaceUtils의 검증 로직을 컨트롤러에서 수행
    -> 이 검증 로직은 유저가 스페이스 내에서 수행하는 작업을 위한 모든 api 에서 필요할 것 같습니다.
  • 서비스 단 코드를 테스트하기 위한 PayServiceTest 클래스 생성

✅ 리뷰 요구사항

현재 정산 관련 서비스 단의 코드가 조금 지저분합니다.
추후에 리펙토링을 할 계획인데, 코드 리펙토링에 대한 의견있으면 리뷰 남겨주시면 감사하겠습니다

📸 확인 방법 (선택)



📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@seongjunnoh seongjunnoh linked an issue Jul 24, 2024 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@drbug2000 drbug2000 left a comment

Choose a reason for hiding this comment

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

OK

@seongjunnoh seongjunnoh merged commit e576e92 into develop Jul 24, 2024
3 checks passed
@hyunn522 hyunn522 deleted the 37-정산-view-api branch July 25, 2024 12:30
seongjunnoh added a commit that referenced this pull request Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

정산 view api
2 participants