Test: RunningRecord addRunningRecord 메서드에서 goal 모드인 경우 성공, 미달 테스트 케이스 추가

[Jaewon-pro]

🔗 이슈 연결

• 177

🚀 구현한 API

• X

💡 반영할 내용 및 변경 사항 요약

• Running 기록을 저장할 때 RunningAchievementMode가 GOAL이라면, 목표 성취(goal_achievement)를 올바르게 저장하는지 테스트하는 로직을 추가합니다.

  1. 목표값과 측정값 비교 후, 성공 / 실패 판별 로직 검증
     • 목표 > 실제 측정치, 목표 성취 테이블에는 실패 정보를 저장
     • 목표 < 실제 측정치, 목표 성취 테이블에는 성공 정보를 저장
     • 목표 == 실제 측정치, 목표 성취 테이블에는 성공 정보를 저장
  2. 보낸 입력값에 따라 반환하는 title을 검증
     • 거리를 기준으로 한다면 title에 km가 포함되어야 합니다.
     • 시간을 기준으로 한다면 title에 분이 포함되어야 합니다.

       🔍 리뷰 요청/참고 사항

• 민령님이 남겨주신 버그는 서비스 로직에는 없는 것으로 보입니다.
• 컨트롤러나 리포지터리 계층에서도 값을 받아서 그대로 넘겨주므로 처음에 request에 goalTime과 goalDistance를 둘다 적어서 생긴 문제 같아요

[Jaewon-pro]

서비스 단에서 타입에 따라 값을 체크하는 로직을 추가 하거나, custom validation을 사용해서 값을 체크하는 방식을 추가하면 좋을 것 같아요!

지금은 서비스 계층이 좀 복잡한 것 같아서, 2번째 방법인 request 객체 생성 시 goalTime과 goalDistance가 같이 있는지 검사하는 로직을 넣도록 하면 좋을 것 같아요!