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

Write test code of GoalService #76

Merged
merged 8 commits into from
Jan 17, 2024
Merged

Write test code of GoalService #76

merged 8 commits into from
Jan 17, 2024

Conversation

mkSpace
Copy link
Collaborator

@mkSpace mkSpace commented Jan 17, 2024

변경사항

  • BaseException을 외부 모듈로 두어 core, api에서 공동으로 사용하게 해서 Exception을 일원화 했습니다
  • user table 변경 이후 DDL을 변경하지 않아 오류가 발생한 부분 수정했습니다
  • core 모듈에서 원래 Repository 와 Service의 테스트 환경을 나누려고 했는데 api 단에서도 controller, applicationService 단으로 또 나눠야 해서 core 모듈은 비즈니스 로직도 별로 없다 생각하여 통합 환경에서 테스트하게 수정했습니다
  • ExceptionSupplier를 빈으로 등록해서 NotFoundException을 쉽게 던질 수 있게 편의성 코드를 추가했습니다
  • GoalService 테스트 코드를 작성했습니다
  • GoalService에 find, load의 용어가 혼동되어 있어 find로 통일했습니다. (다른 서비스는 테스트 코드를 작성하면서 반영할 예정입니다)

@mkSpace mkSpace requested a review from K-Diger January 17, 2024 08:52
@mkSpace mkSpace merged commit bd2be06 into develop Jan 17, 2024
1 check passed
@mkSpace mkSpace deleted the feature/service-test branch January 17, 2024 08:58
This pull request was closed.
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