[BE] test: 테스트 환경에 영향받는 비동기 로직 작동 검증 테스트 제거

[hanueleee]

Issue

• close [BE] fix: 테스트 환경에 영향받는 비동기 테스트 삭제 #787

✨ 구현한 기능

ReviewDeleteEventListenerTest의
리뷰_삭제가_정상적으로_커밋되었지만_이미지가_존재하지_않으면_이미지_삭제_로직이_작동하지않는다() 테스트가
테스트 환경에 따라 성공 실패 번갈아 발생

• 로컬에서 성공
• 깃허브액션에서 성공
• 젠킨스 빌드시 성공 실패 번갈아 발생

실패하는 이유 추정 1

• reviewDeleteEventListener의 deleteReviewImageInS3는 비동기로 동작
• 테스트 실행 환경(스레드 개수?)에 따라 전 테스트에서 사용된 이벤트가 다음 테스트에서 잡히는 경우 생김
• ""인 이미지를 가지는 event를 사용하는 테스트에서 자꾸 "test3" 이미지(전 테스트에서 사용되는 이미지)를 가지는 event가 잡힘 -> ""여서 작동하지 않아야 하는 delete 메소드가 "test3"을 읽고 작동 -> 실패
• 해결 시도: 테스트 이후 event 비워주기 events.clear()
  => 근데 젠킨스에서 가끔 실패하는 것으로 보아 아닌듯 ..

실패하는 이유 추정 2

• events.clear로 event를 비워주더라도 이미 앞 테스트에서 uploader.delete는 비동기적으로 작동되었기 때문에 delete 작동 여부 검증에 걸리는듯

정확한 이유는 젠킨스 브랜치 매핑 따로 해서 로그 하나하나 찍어봐야 할 것 같은데
당장 내일 운영 배포해야하기 때문에.. 우선 테스트 삭제 결정

📢 논의하고 싶은 내용

• X

🎸 기타

• 비동기를 통합테스트하는게 맞을까?
  • 그냥 1) service에서 event 발행하는지 2) eventlistener의 작동(uploader.delete)을 따로따로 테스트하는게 나을까?

⏰ 일정

• 추정 시간 : 0.1
• 걸린 시간 : 0.1

[github-actions]

Test Results

272 tests   272 ✔️  20s ⏱️
136 suites      0 💤
136 files        0 ❌

Results for commit 82db9f9.