Fix: time issue

[wjdxor]

문제발생 (#26 )

LocalTime.now() 라이브러리 메소드를 사용하였더니 현재 시간에 따라 테스트 성공 유무가 바뀌는 문제 발생

서비스 안에 now()가 메서드에 의해 결정되는 상황을 피하기 위한 방법이 필요

해결방법

시간 설정 클래스를 만들어 Bean에 저장하여 사용

[sonarcloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
4 Code Smells

100.0% Coverage
0.0% Duplication

[wjdxor]

문제

• 매칭 종료시간이 현재 시간보다 뒤인 경우를 테스트할 때
  • 종료 시간을 현재시간 + 1시간으로 설정
  • 이 경우, 자정을 지나 종료시간이 다음날로 설정 되는 경우가 있다.
  • 이 때, 매칭 시작시간보다 종료시간이 빨라 매칭 생성이 안되는 에러가 발생

    해결

• 현재시간을 Mocking 한다.
  • 현재 시간을 설정파일로 만들어 서비스레이어에서 이 설정파일에 의존 해야한다.
• 매칭 생성할 때 날짜와 시간을 계산한다.
  • 기획 단계에서 날짜를 넘어가는 매칭은 없다고 생각하였기때문에 매칭 정보로 날짜(LocalDate), 시작시간(LocalTime), 종료시간(LocalTime) 을 저장한다. 날짜와 시간을 같이 계산하기 위해선 도메인 설계 수정이 필요해 보인다.