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

[중간점검] 03/21 스터디 전 중간점검 PR #1

Open

sung-silver wants to merge 5 commits into develop from test/#1

Member

sung-silver commented Mar 21, 2024

서비스 영역에서의 로직 중 도메인 로직인 부분 리팩터링
패키지 구조 변경 (common/AgeEnum -> domain/enums/AgeEnum
도메인 영역 테스트코드 작성
궁금한점: 단순 이넘에서의 반환 값을 나이대 별로 검증하였는데 이런 로직은 테스트 하나로 묶어도 괜찮을까요?

sung-silver added 4 commits

March 21, 2024 22:32


          [FEAT] 서비스 단에서 도메인 로직 분리

be318b2


          [STYLE] enum 위치 도메인 아래로 변경

6e2ec20


          [DOMAIN] AgeEnumTest 작성

ad8387c


          [DOMAIN] Result 엔티티에서 getTestDateToString 함수 테스트 작성

sung-silver requested a review from a team

March 21, 2024 14:00

sung-silver self-assigned this


          [REFACTOR] saveMember 리팩터링, repository에서 명시하지 않아도 되는 메서드 삭제

86e90bf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet