일단 머지하고 수정할 부분 다음 PR에 적용하는게 나을거같아요
멘토님 PR 피드백
- 한줄로 적으면 가독성이 떨어질 수 있다.
- Service 인터페이스 분리
- naming
- 이름 길이 규칙, 평균점수 소수점 → 내림
- validation 정규표현식
- @NotBlank
- @Min(0)
- https://hyeran-story.tistory.com/81 - Spring Validation Annotation 총정리
- softDelete setter말고 각각 엔티티 내부에서 변경
- baseEntity에서 함수 이름을 수정하면 되지않을까요?
- SoftDelete하면 조회 안되게
- 테스트에서 size(0)이 되게
- 이렇게 되어있는줄 알았는데 아닌가봐요..?
- 인증 인가가 필요한 로직인 경우 어떻게 테스트 할 수 있는지
- Color Code validation
컨벤션 정할 부분
- Exception
- Test
- 멘토님 : 서비스 테스트를 SpringBootTest로 하신 이유?
- mock 객체를 활용한 테스트보다는 직접 SpringBootTest로 확인해보고 싶었다~
- 통합 테스트 필수! + 필요에 따라 유닛 테스트는 자유로
- Controller
- /api/v1/users 공통?
- Service
- Impl 클래스 사용 X → 인터페이스 사용
- Entity
- @GeneratedValue(strategy = GenerationType.AUTO)? Identity?
- 생성자 주입 할때 @RequiredArgsConstructor or 생성자
- 다 롬복 사용
수정 사항
BaseEntity setIsDeleted() → delete() 메소드명 수정
EntityExceptionSuppliers 예외 메소드생성 전략 정하기 - auto or identity ⇒ auto로
프로젝트 이름 backend → surf
ApiResponse 빼기
프론트 얘기할 것
매번 토큰을 넣어서 요청을 달라