base entity에 id가 있으면 안 좋을 것 같다는 의견
- base entity는 여러개여도 된다.
- 모든 엔터티에 무조건 있어야 하는 느낌
- 만약 id가 long이 아니게 된다면?
- id가 없는 테이블도 있을 수 있다.
- Id를 userId 등으로 바꿔서 쓸 수도 있다.
- 결국에 변화할 가능성이 너무 많아서 추천하지 않는다.
코드리뷰의 스타일
- 스타일도 코드리뷰를 하는 이유는 가독성 때문
- 개행도 결국의 제대로 개행을 해놔야 로직 이해가 잘되는 것처럼 사소한 코드리뷰도 중요하다고 생각한다.
- 코드 리뷰의 반은 가독성 때문이다.
테스트 코드의 스타일
- 단위 테스트, 통합 테스트
- 컨트롤러를 mockmvc로 서비스를 모킹 하는 단위 테스트와, 의존성을 이용하는 통합테스트가 필요함
- 레포지토리 단위 테스트, 서비스 단위 테스트, 컨트롤러 단위 테스트, 통합 테스트
통합 테스트는?
- view → … → repository 까지 거쳐서 결과가 제대로 나오나 확인 (모든 레이어를 거쳐서 결과가 잘 나오는지 확인하는 테스트)
서비스에서 단위테스트? 통합테스트?