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

[NAYB-159] feat: 점원과 라이더는 배달 상세를 조회할 수 있다. #123

Merged
merged 4 commits into from
Sep 21, 2023

Conversation

hseong3243
Copy link
Collaborator

@hseong3243 hseong3243 commented Sep 21, 2023

⛏ 작업 사항

  • 점원, 라이더가 사용하기 위한 배달 상세 조회를 추가하였습니다.

📝 작업 요약

  • 관련 메서드 및 테스트를 추가하였습니다.

💡 관련 이슈

@github-actions
Copy link

Test Results

383 tests  +3   383 ✔️ +3   11s ⏱️ ±0s
188 suites +2       0 💤 ±0 
188 files   +2       0 ±0 

Results for commit 7dc2ebe. ± Comparison against base commit 3c2e596.

This pull request removes 3 and adds 6 tests. Note that renamed tests count towards both.
com.prgrms.nabmart.domain.delivery.service.DeliveryServiceTest$FindDeliveryTest ‑ 예외: 로그인 User와 일치하지 않는 Order의 User
com.prgrms.nabmart.domain.delivery.service.DeliveryServiceTest$FindDeliveryTest ‑ 예외: 존재하지 않는 Delivery
com.prgrms.nabmart.domain.delivery.service.DeliveryServiceTest$FindDeliveryTest ‑ 예외: 존재하지 않는 User
com.prgrms.nabmart.domain.delivery.controller.DeliveryControllerTest$FindDeliveryByOrderTest ‑ 성공
com.prgrms.nabmart.domain.delivery.service.DeliveryServiceTest$FindDeliveryByOrderTest ‑ 성공
com.prgrms.nabmart.domain.delivery.service.DeliveryServiceTest$FindDeliveryByOrderTest ‑ 예외: 로그인 User와 일치하지 않는 Order의 User
com.prgrms.nabmart.domain.delivery.service.DeliveryServiceTest$FindDeliveryByOrderTest ‑ 예외: 존재하지 않는 Delivery
com.prgrms.nabmart.domain.delivery.service.DeliveryServiceTest$FindDeliveryByOrderTest ‑ 예외: 존재하지 않는 User
com.prgrms.nabmart.domain.delivery.service.DeliveryServiceTest$FindDeliveryTest ‑ 예외: 존재하지 않는 배달

@hseong3243 hseong3243 merged commit c539360 into develop Sep 21, 2023
3 checks passed
@hseong3243 hseong3243 deleted the feature/NAYB-159 branch September 21, 2023 14:51
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.

2 participants