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.

Sign up for GitHub

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

Jump to bottom

[feat] 구매자 매장상세조회 #106

Merged
merged 14 commits into from
Aug 19, 2024
Merged

[feat] 구매자 매장상세조회 #106

merged 14 commits into from
Aug 19, 2024

Conversation

june-777
Copy link
Member

@june-777 june-777 commented Aug 17, 2024 •
edited
Loading

💡 다음 이슈를 해결했어요.

Issue Link - #95


💡 이슈를 처리하면서 추가된 코드가 있어요.

StoreDisplayResponse

  • StoreDisplayResponse는 구매자-매장상세조회에 대한 응답 DTO입니다.
  • 매장 상세 조회에는 매장의 메뉴를 리스트로 갖고 있어야하므로, 내부 레코드로 MenuResponse를 구현했습니다.

💡 이런 고민을 했어요.

Response DTO 를 어떻게 구조화할까

  • 가게 상세 조회는 가게 메뉴 리스트도 함께 갖고 있어야 함.
  • MenuResponse 를 별도의 클래스 혹은 레코드로 생성하게되면, 다른 사람들도 MenuResponse를 사용하게 될 가능성이 많아지고, 점진적으로 Response DTO를 조립해서 사용하는 구조가 됨.
  • 이렇게 조립해서 DTO를 사용하면, DTO 이곳저곳에서 사용하게 되므로 변경하기 매우 어려워짐
  • 따라서 내부 레코드로 구현하여 진행함.
  • 이 부분에 대한 피드백을 해주시면 감사하겠습니다 :)

단순 조회 로직을 수행하는 서비스 계층을 통합테스트 해야할까

  • Mock + 단위 테스트만으로 충분하다고 판단!

✅ 셀프 체크리스트

  • 내 코드를 스스로 검토했습니다.
  • 필요한 테스트를 추가했습니다.
  • 모든 테스트를 통과합니다.
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • wiki를 수정했습니다.

june-777 added 11 commits August 17, 2024 18:50
@june-777
[feat] 구매자 매장 상세 조회 서비스
615e663
@june-777
[feat] 구매자 매장 상세 조회 Response 객체
b43df27 
- 구매자는 기본적인 매장 상세 정보를 확인할 수 있어야 함
@june-777
[feat] 가게주소, 가게카테고리에 대한 getter 추가
3bbe608 
호출자 쪽에서의 getter depth 를 줄이기 위함
@june-777
[feat] 가게 id로 메뉴 목록을 조회하는 기능
3c8fbba
@june-777
[feat] 가게 전시 응답에 가게 메뉴를 포함
16c6667
@june-777
[feat] getter 추가
1562c3b
@june-777
[feat] 가게 전시 API 엔드포인트 구현
579db33
@june-777
Merge remote-tracking branch 'origin/main' into feature/95_june-777_구…
05c54af 
…매자-매장상세조회

# Conflicts:
#	src/main/java/camp/woowak/lab/menu/domain/Menu.java
#	src/main/java/camp/woowak/lab/menu/repository/MenuRepository.java
@june-777
[refactor] 가게 전시 메서드 이름 변경
5d4a388
@june-777
[test] 가게 전시 서비스 계층 단위 테스트
408a387 
- responseDTO 을 적절히 매핑하여 반환하는지 테스트하기 위함
@june-777
[test] 가게 전시 API 컨트롤러 계층 단위 테스트
9700941 
- API response 명세를 검증 및 추적하기 위함
@june-777 june-777 marked this pull request as ready for review August 17, 2024 14:15
@june-777 june-777 added the ✨ Feature 기능 개발 label Aug 17, 2024
@june-777 june-777 added this to the 프로토타입 만들기 milestone Aug 17, 2024
@june-777 june-777 self-assigned this Aug 17, 2024
@june-777 june-777 linked an issue Aug 17, 2024 that may be closed by this pull request
[기능] 구매자는 매장의 상세 정보를 조회할 수 있다. #95
Closed
Dr-KoKo
Dr-KoKo approved these changes Aug 18, 2024
View reviewed changes
Copy link
Member

@Dr-KoKo Dr-KoKo left a comment

Choose a reason for hiding this comment

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

고도화하면서 도메인에서 조회를 위한 기능을 분리하는 것도 이야기해보면 좋겠습니다.

@june-777
[text] 가게 조회 API 단위 테스트 - 메뉴 리스트 검증하도록 수정 및 테스트
ebfc65c
@june-777
Merge remote-tracking branch 'origin/main' into feature/95_june-777_구…
49cad6a 
…매자-매장상세조회

# Conflicts:
#	src/main/java/camp/woowak/lab/menu/domain/Menu.java
#	src/main/java/camp/woowak/lab/web/api/store/StoreApiController.java
#	src/test/java/camp/woowak/lab/web/api/store/StoreApiControllerTest.java
@june-777
[merge] 머지 컨플릭트 해결
2185e1f
@june-777 june-777 merged commit 6cc9211 into main Aug 19, 2024
1 check passed
@june-777 june-777 deleted the feature/95_june-777_구매자-매장상세조회 branch August 19, 2024 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kimhyun5u kimhyun5u kimhyun5u left review comments

@Dr-KoKo Dr-KoKo Dr-KoKo approved these changes

@Hyeon-Uk Hyeon-Uk Hyeon-Uk approved these changes

Assignees

@june-777 june-777

Labels
✨ Feature 기능 개발
Projects
3K1K Project
Status: Done
Milestone
프로토타입 만들기
Development

Successfully merging this pull request may close these issues.

[기능] 구매자는 매장의 상세 정보를 조회할 수 있다.
4 participants
@june-777 @kimhyun5u @Hyeon-Uk @Dr-KoKo