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

test: 알림 도메인 테스트 작성 및 문서화 추가 #125

Merged
merged 3 commits into from
Sep 25, 2023

Conversation

hseong3243
Copy link
Collaborator

⛏ 작업 사항

  • 알림 컨트롤러, 서비스 테스트를 작성하였습니다.
  • 기존에 추가한 api 중 restDocs 문서 미반영분과 알림 컨트롤러 api 문서화를 추가하였습니다.

📝 작업 요약

  • NotificationController/Service 테스트 작성
  • 결제 완료된 주문 목록 조회(직원), 배달 생성, 배달 상세 조회(라이더, 직원), 알림 연결 api 문서화 추가

💡 관련 이슈

  • 없음

`배달 생성`,  `배달 상세 조회(라이더, 직원)`, `알림 연결` api 문서를 추가함
@github-actions
Copy link

github-actions bot commented Sep 25, 2023

Test Results

391 tests  +3   390 ✔️ +3   13s ⏱️ +2s
193 suites +3       1 💤 ±0 
193 files   +3       0 ±0 

Results for commit a5fb1a8. ± Comparison against base commit 34bdd98.

♻️ This comment has been updated with latest results.

@hseong3243 hseong3243 merged commit a0740f0 into develop Sep 25, 2023
3 checks passed
@hseong3243 hseong3243 deleted the feature/notification-docs branch September 25, 2023 10:55
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.

3 participants