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

[#5}서비스와 유닛테스트 작성하기 #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

seunghaen
Copy link
Collaborator

멘토링 이후 수정사항 반영

  • Entity -> ResDTO 메서드위치변경
  • 옵션 및 선택지 CRUD API 별도 분리
  • ConcurrentHashMap변경
  • 통합테스트 Order대신 로직중복
  • 유닛테스트 times 및 never검증

pay, order, ,choice, option

  • 서비스 및 유닛테스트 코드 완성하기

1. Entity -> ResDTO 메서드위치변경
2. 옵션 및 선택지 CRUD API 별도 분리
3. ConcurrentHashMap변경
4. 통합테스트 Order대신 로직중복
5. 유닛테스트 times 및 never검증
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.

1 participant