Fix time-issue : 시간 문제 해결

[wjdxor]

26 매칭 생성시 자정이 넘어가는 시간대에 생기는 문제

• 문제해결방법
  • LocalTime 비교 방식에서 LocalDateTime으로 변경
• 해당 방법 사용 이유
  • matchingService에서 종료시간을 현재시간과 비교해서 판단했기에 단순히 테스트만 변경한다고 해결안됨.

[wjdxor]

입력이 다음날로 안된다는 것을 가정하고 테스트에 조건을 주는 방법으로 수정하였습니다.

• 현재 시간과 비교해야 하는 테스트이기 때문에 임의 입력이 어려웠습니다. -> 현재시간에서 minus했을때 시간이 전날로 넘어간다면 오늘 날짜 0시0분으로 지정해주었습니다. -> 현재시간에서 plus했을대 시간이 다음날로 넘어간다면 오늘 날짜의 23시 59분으로 지정해주었습니다.

[this-is-spear]

@wjdxor 이 문제 자체가 서비스 안의 now() 메서드에 의해 결정이 되는 상황이니 이런 방법은 어떨까요?

1. now() 메서드를 Mocking
2. Clock 클래스를 이용한 가짜 객체 생성

[wjdxor]

LocalTime.now(Clock clock)을 호출할때 Clock객체를 mocking할 것이다.

1. Clock 클래스를 만들어 Clock을 고정해주는 역할을 한다.
2. MatchingService에서 LocalTime.now()대신 Clock을 주입한다.
3. Test에서 clock을 통해 시간을 설정한다.

[참고] https://perfectacle.github.io/2018/03/30/mock-time-with-time-machine/

[sonarcloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

92.9% Coverage
0.0% Duplication