[LO-215] 컨트롤러 테스트 @Transactional 제거 & 컨밴션에 맞게 리팩토링

[hyuk0309]

🛠️ 작업 내용

• 컨트롤러 테스트 @Transactional 제거 & 컨밴션에 맞게 리팩토링

🗨️ 기타

• Controller 테스트에도 @Transactional을 제거했습니다.
• Controller 테스트 네이밍과, when/then 영역을 구분하는 작업을 했습니다..
• 너무 많은 것을 테스트하는 것 같은 테스트는 여러개의 테스트로 분리했습니다.

---

• 리팩토링 하다보니까 저희 알림 기능도 진짜 아픈 손가락이네요 ㅎㅎ
• 서비스와 컨트롤러 테스트의 트랜잭션을 제거해 프로덕션 코드와 비슷한 환경이 됐습니다. 하지만 단점으로 테스트 시간이 오래 걸리네요. 이것도 한번 최적화에 대해 공부한 다음 적용해보면 재밌을 것 같아요. 😄

😎 리뷰어

@ndy2 , @jk05018

[github-actions[bot]]

Unit Test Results

237 tests   237 :heavy_check_mark:  1m 7s :stopwatch:   72 suites      0 :zzz:   72 files        0 :x:

Results for commit c5da8a8b.

[hyuk0309]

제 로컬에서는 50초 정도 걸립니다.. ㅎㅎ 20초 정도 늘어난 것 같아요.

근데 DB를 모킹해서 해결하는 방식보다는, 서비스나 컨트롤러 테스트에서 (infra 모듈을 모킹하면 어느정도 최적화가 가능할 것 같아요.) 그리고 DB도 항상 데이터를 비워주는게 아니라, 조회용 테스트, 컨맨드용 테스트로 나눠서 조회는 @beforeAll, @afterAll 커맨드는 @beforeEach, @afterEach를 사용해도 어느정도 최적화가 가능할 것 같아요!