#60 PlaceReadServiceTest 비동기 로직 대기하도록 수정

[gihhyeon]

📌 PR 제목

PlaceReadServiceTest 비동기 로직 대기하도록 수정

✅ 작업 내용

• PlaceReadServiceTest 비동기 테스트 로직 수정
  • @Async로 인한 조회수 비동기 업데이트를 테스트하기 위해 Awaitility 사용
• await().untilAsserted()를 사용해, DB 업데이트가 완료될 때까지 대기한 후 assertThat을 검증하도록 수정

🎯 관련 이슈

• close [bug] PlaceReadServiceTest 오류 #60

💬 기타 공유하고 싶은 내용

• 이전엔 괜찮다가 갑자기 테스트 코드가 깨졌는데 이전에 왜 됐는지가 의문.....

  • 저번에 조회수 관련 로직을 비동기 처리로 고친 후 관련된 테스트 코드를 수정 안 했었던 상황. 배포할 때 테스트코드가 깨지지 않다가 11/3(월요일)에 깨지는 현상 발견
  • 깨졌어야 되는 테스트 코드가 왜 뒤늦게 깨졌을까요....

• 이것과 관련해서 동시성, 스레드풀, 비동기에 대해서 공부를 더 해보면 좋을까요??

[jcw1031]

awaitility라는 라이브러리는 처음 봤는데 나름 유용한 것 같네?

근데 만약 조회수 로직에 이슈가 있어서 조회수가 정상적으로 업데이트가 안 되면 최대 10초까지 대기하게 될 것 같아서 더 좋은 방법은 없을까 싶긴 하다

조회수 증가 로직을 mocking해서 단순하게 n번 호출되었는지를 검증하는 방법도 있긴 한데, 테스트하려는 게 멀티 스레드 환경에서 조회수가 정상적으로 잘 업데이트 되는지를 검증하는 거라면 mocking하는 게 좋은 방법은 아닌 것 같기도 하고..

[jcw1031]

지금 방식도 크게 문제는 없을 것 같긴 해~

동시성 테스트 작성한 것도 좋은 것 같구