[4기 - 박세연] SpringBoot Part3 2차 Weekly Mission 제출합니다.

[parksey]

먼저 좋게 봐주셔서 감사합니다!

개인 프로젝트 등으로 늦게 확인하게 되었고, 그에 따라 피드백 반영이 늦어졌습니다. 죄송합니다 😢

✅ 피드백 반영사항

전반리뷰

패캐지 구성이

root
 - api
 - web
 - voucher
 - customer
 - util
 - ui

로 되어있었습니다.

당시에는 voucher, customer, ui(writter & reader)에 대한 도메인으로 분류한 후 api나 web같은 경우 한 곳에서 보게 되면 더 편하지 않을까 라는 생각으로 저렇게 구성했었는데,

말씀을 듣고,

1. 도메인 별로 분류한 것도 아니고
2. layer로 나눈 것도 아닌 이도 저도 아니게 되고
3. 시간이 지나니 오히려 불편했다.
   는 단점들이 보였습니다.

따라서 api와 web같은 경우 클라이언트에서의 요청을 처리하기 때문에 controller내부로 옮겼고, 세부적으로 api, web으로 분류하여 구성했습니다.

###Test코드 리뷰

기존 console과 내부 In-memory상황과 JDBC를 사용했을 대의 상황에 대해 테스트를 진행했는데, 부족한 부분과 테스트 코드가 동작이 안되는 부분이 있었습니다.

따라서 리뷰주신 부분 보강하였습니다.

[Service test]

서비스 부분만 테스트를 진행하기 위해 mockito를 기반으로 서비스 이외의 의존성을 mock객체로 변환하여 테스트를 진행

[DTO & Domain]

데이터가 송수신 및 저장을 위해 값이 잘 매핑되는지 확인