[LO-214] 서비스 테스트 @Transactional 제거하기

[hyuk0309]

🛠️ 작업 내용

• 서비스 테스트 @Transactional 제거하기

🗨️ 기타

• 서비스 테스트 데이터 셋업은 BaseServiceTest에서 Service 메서드를 이용합니다. 덕분에 @Transactional을 제거해도 문제가 되지 않았습니다. 문제가 되는 부분은 테스트 후 남아 있는 데이터였습니다.
• 남아 있는 데이터를 지우기위한 유틸용 메서드를 만들었습니다. (b2929203c6f4f92e91fe31679e69dc3089acef26)
• @Transactional을 지우니 확실히 프로덕션 환경이랑 테스트 환경이 비슷해진 것 같아 테스트에 대한 신뢰성이 올라갔어요. 하지만 트레이드 오프로 테스트 속도가 이전보다 조금 느려졌습니다.
• 알림쪽 서비스 테스트는 @transactional을 제거하니 실패하네요.. 이 부분은 추후에 알림 기능 손보면서 같이 손봐야할 것 같아요.

---

통합 테스트와 트랜잭션 관련 글이 있는데 저와 생각이 비슷한 것 같아 공유드립니다! (참조)

• The production code is using transactions with different scope.
• It is easy to forget about a flush and thus have false positive in test.
• On failure and when debugging it is hard to see what values were actually saved in db.
• It is much harder to write tests of production code that requires a transaction to be committed.
• The test code needs to be more tightly coupled to production code and we all know that it hinders refactoring.

😎 리뷰어

@ndy2 , @jk05018

[github-actions[bot]]

Unit Test Results

233 tests   233 :heavy_check_mark:  42s :stopwatch:   72 suites      0 :zzz:   72 files        0 :x:

Results for commit bfd1cf63.

:recycle: This comment has been updated with latest results.