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] 점주 주문 리스트 조회 #94

Merged
merged 27 commits into from
Aug 18, 2024
Merged

[feat] 점주 주문 리스트 조회 #94

merged 27 commits into from
Aug 18, 2024

Conversation

kimhyun5u
Copy link
Member

@kimhyun5u kimhyun5u commented Aug 16, 2024 •
edited
Loading

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

Issue Link - #85

  • 점주 주문 리스트 조회

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

  • 구현: RetrieveOrderListService, RetrieveOrderListCommand, OrderDTO
  • 수정: OrderApiController

💡 이런 고민을 했어요.

  • 추후에 페이지네이션 필터를 생각하면 DAO 로 설계했어야되나?
  • 관계가 많아지니까 테스트 더미 만드는게 어렵다. 모두 Mock 으로 사용해야하나?

✅ 셀프 체크리스트

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

kimhyun5u added 11 commits August 16, 2024 16:12
@kimhyun5u
[chore] .gitkeep 제거
f3eba0f
@kimhyun5u
[test] OrderRepositoryTest 구현
43e039f 
- OrderRepositoryTest.testFindAllByOwner: 점주가 관리하고 있는 모든 점포의 주문 조회
- OrderRepositoryTest.testFindAllByOwnerFailWithUnauthorized: 점주가 관리하고 있는 모든 점포의 주문 조회 실패
- OrderRepositoryTest.testFindByStore: 특정 매장 주문 조회
- OrderRepositoryTest.testFindByStoreFailWithUnauthorized: 특정 매장 주문 조회 실패 - 권한 없음
@kimhyun5u
[feat] OrderRepository 구현
049b831 
- OrderRepository.findAllByOwner: 점주가 관리하는 모든 점포의 주문 조회
- OrderRepository.findByStore: 특정 점포의 주문 조회
@kimhyun5u
[feat] RetrieveOrderListService 구현
0696119 
- RetrieveOrderListService.retrieveOrderListOfVendorStores: 점주가 관리하는 모든 점포의 주문 조회
- RetrieveOrderListService.retrieveOrderListOfStore: 특정 점포의 주문 조회
@kimhyun5u
[test] RetrieveOrderListServiceTest 구현
3fedac7 
- RetrieveOrderListServiceTest.testRetrieveOrderList: 점주 주문 리스트 조회 테스트 - 성공
- RetrieveOrderListServiceTest.testRetrieveOrderListOfStore: 점주 특정 매장 주문 리스트 조회 테스트 - 성공
- RetrieveOrderListServiceTest.testRetrieveOrderListOfStoreFailByNotFoundStore: 점주 특정 매장 주문 리스트 조회 테스트 - 실패(매장이 존재하지 않음)
- RetrieveOrderListServiceTest.testRetrieveOrderListOfStoreFailByNotEqualsOwner: 점주 특정 매장 주문 리스트 조회 테스트 - 실패(매장의 주인이 아님)
@kimhyun5u
[feat] RetrieveOrderListCommand 구현
233d752
@kimhyun5u
[test] OrderApiControllerTest 구현
d655a6e 
- OrderApiControllerTest.testRetrieveOrderList: 점주 주문 리스트 조회 테스트 - 성공
- OrderApiControllerTest.testRetrieveOrderListByStore: 점주 매장 주문 리스트 조회 테스트 - 성공
- OrderApiControllerTest.testRetrieveOrderListByStoreFail:점주 매장 주문 리스트 조회 테스트 - 실패(매장이 존재하지 않음)
- OrderApiControllerTest.testRetrieveOrderListByStoreFailWithUnauthorized: 점주 매장 주문 리스트 조회 테스트 - 실패(매장의 주인이 아님)
@kimhyun5u
[feat] OrderApiController 구현
b07e940 
- OrderApiController.retrieveOrderList: 점주가 관리하는 모든 점포의 주문 조회 요청 처리
- OrderApiController.retrieveOrderListByStore: 특점 점포의 주문 조회 요청 처리
@kimhyun5u
[feat] RetrieveOrderListResponse 구현
1a75f86 
- 주문 리스트 조회 응답
@kimhyun5u
[feat] OrderExceptionHandler 구현
e469672 
- 주문 예외 핸들러 구현
- OrderExceptionHandler.handleNotEqualsOwnerException: NotEqualsOwnerException 처리
- OrderExceptionHandler.handleNotFoundStoreException: NotFoundStoreException 처리
@kimhyun5u
[chore] OrderApiControllerTest 수정
80bf9b0 
- 사용하지 않는 코드 제거
@kimhyun5u kimhyun5u added the ✨ Feature 기능 개발 label Aug 16, 2024
@kimhyun5u kimhyun5u added this to the 프로토타입 만들기 milestone Aug 16, 2024
@kimhyun5u kimhyun5u self-assigned this Aug 16, 2024
@kimhyun5u kimhyun5u linked an issue Aug 16, 2024 that may be closed by this pull request
[기능] 점주 주문 리스트 조회 #85
Closed
kimhyun5u added 13 commits August 17, 2024 16:28
@kimhyun5u
[merge] remote-tracking branch 'origin/main' into feature/85_kimhyun5…
edb8b57 
…u_점주-주문-리스트-조회
@kimhyun5u
[merge] origin/main into feature/85_kimhyun5u_점주-주문-리스트-조회
98b6fd0
@kimhyun5u
[feat] OrderDTO 구현
e1216ed 
- 주문 조회 시 반환할 DTO 구현
- OrderDTO.RequesterDTO, OrderDTO.OrderItemDTO, OrderDTO.StoreDTO 를 내부에 두어 필요한 내용만 노출 시킬 수 있도록 함
@kimhyun5u
[feat] OrderItem 수정
38b80b1 
- DTO 변환 시 내부 내용 조회가 필요해 `@Getter` 추가
@kimhyun5u
[feat] Order 수정
41a7a8a 
- DTO 변환 시 내부 내용 조회가 필요해 getter 추가
@kimhyun5u
[test] OrderApiControllerTest 수정
6dc5942 
- OrderDTO 생성 후 데이터 검증 테스트 추가
@kimhyun5u
[feat] VendorFixture 추가
2aa42bb 
- VendorFixture.createTestVendor: 간단히 TestVendor 를 생성하는 메소드
@kimhyun5u
[feat] TestOrderDTO 구현
01e81ba 
- 테스트에 사용할 TestOrderDTO 구현
@kimhyun5u
[feat] StoreFixture 구현
20161ab 
- 반복되는 TestStore 생성 코드를 추상화
@kimhyun5u
[feat] StoreAddress 수정
3c3032f 
- DTO 생성을 위해 StoreAddress 필드를 조회할 수 있도록 변경
@kimhyun5u
[feat] OrderDTO 생성으로 인한 수정
ebcc576 
- 반환값을 OrderDTO 로 수정
@kimhyun5u
[merge] remote-tracking branch 'origin/main' into feature/85_kimhyun5…
f72d9b7 
…u_점주-주문-리스트-조회
@kimhyun5u
[feat] OrderDTO 수정
6be0374 
- requestID(customer_id) 와 StoreID 추가
@kimhyun5u kimhyun5u marked this pull request as ready for review August 18, 2024 05:38
@kimhyun5u kimhyun5u requested a review from Dr-KoKo August 18, 2024 05:38
@kimhyun5u kimhyun5u requested review from june-777, Hyeon-Uk and Dr-KoKo and removed request for Dr-KoKo August 18, 2024 05:38
june-777
june-777 approved these changes Aug 18, 2024
View reviewed changes
Copy link
Member

@june-777 june-777 left a comment

Choose a reason for hiding this comment

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

확인 완료입니다 🙂

@kimhyun5u
[merge] remote-tracking branch 'origin/main' into feature/85_kimhyun5…
25f085a 
…u_점주-주문-리스트-조회

# Conflicts:
#	src/main/java/camp/woowak/lab/order/domain/vo/OrderItem.java
@kimhyun5u
[fix] OrderDTO 수정
a6a0975 
-  OrderItem price 타입 변경으로 인한 변경
@kimhyun5u
[fix] OrderRepositoryTest 수정
94cd2e6 
-  OrderItem price 타입 변경으로 인한 변경
@kimhyun5u kimhyun5u merged commit 3d4fd5c into main Aug 18, 2024
1 check passed
@kimhyun5u kimhyun5u deleted the feature/85_kimhyun5u_점주-주문-리스트-조회 branch August 18, 2024 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

@june-777 june-777 june-777 approved these changes

@Hyeon-Uk Hyeon-Uk Awaiting requested review from Hyeon-Uk

Assignees

@kimhyun5u kimhyun5u

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

Successfully merging this pull request may close these issues.

[기능] 점주 주문 리스트 조회
3 participants
@kimhyun5u @june-777 @Dr-KoKo