Compono-Team / I-backend

4 stars 0 forks source link

feat : 타임라인 #60

Closed jifrozen0110 closed 3 months ago

jifrozen0110 commented 4 months ago

작업 내용

주의사항

통일성을 위해 제가 임의로 몇 부분 수정하였는데 다른 방식으로 통일하길 원하시면 말씀해주세요!

  1. ErrorCode.java > 에러마다 코드 넘버를 400 > 4001 401 > 4100 404 > 4400 위와 같이 시작하도록 설정하였습니다. 4040으로 시작하면 설정 범위가 좁아서 위와 같은 결정을 내렸는데 가독성을 위해 40401이런식으로 5개 코드 사용하는것도 고려해볼 수 있을 것 같습니다.

  2. Fixtures 사용 (https://velog.io/@langoustine/Test-Fixture) test > common > fixtures 폴더를 보면 test에 필요한 생성자부분을 모아서 생성해두었습니다.

  3. test용 db 분리 ((https://medium.com/@chanhyeonglee/mock-%EA%B0%9D%EC%B2%B4-%EB%82%A8%EC%9A%A9%EC%9D%80-%ED%85%8C%EC%8A%A4%ED%8A%B8-%EC%BD%94%EB%93%9C%EB%A5%BC-%EB%A7%9D%EC%B9%9C%EB%8B%A4-f38129e5d40a)) Mock 객체로 테스트 하는것이 아닌 테스트용 실체를 만들어 테스트하는 방식으로 구현하였습니다.

  4. Schedule User 연관관계 분리

저번에 JPA연관관계를 사용할지 말지에 대해 이야기 나눠본적있었는데 결론이 연관관계를 사용하여 개발의 편의성을 가져오자라고 결론났던것 같습니다. 다시 번복드려 제안드려 죄송한데 Schedule의 경우 Schedule을 불러올때 User가 같이 오는것이 아니기 때문에 연관관계를 깨고 그냥 Long userId로 가지고 있는것이 어떨까요??

이전 pr에서 제안드렸던 부분인데 Schedule에서 user는 jpa 연관관계가 아닌 userId로 들고있는것이 어떨지 제안드렸습니다.

Closes #55

sonarcloud[bot] commented 4 months ago

Quality Gate Passed Quality Gate passed

Issues
15 New issues
2 Accepted issues

Measures
0 Security Hotspots
95.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud