Cafegory / Cafegory_Backend_REST_API

Apache License 2.0
0 stars 1 forks source link

[BUILD SUCCESS] [BUILD FAIL] 테스트 코드 리팩토링 2차 #110

Closed donghyun0304 closed 3 months ago

donghyun0304 commented 3 months ago

반영 브랜치

main

변경 사항

이전 테스트 리팩토링에서 속도 개선은 했으나 단점이 많은 리팩토링이였음. 이번 리팩토링에서는 속도를 포기하고 리팩토링 내성을 높히는 쪽으로 리팩토링 함. 현재 테스트는 3~4초 정도 걸리므로 아직 속도면에서 괜찮다고 판단. 이후에 E2E 테스트를 도입했을때 속도 문제가 발생한다면, 그때 컨텍스트를 줄이는 등의 방식으로 리팩토링 하기로 결정

  1. Mock Repository 제거 후 @SpringBootTest로 변경
  2. 존재하는 모든 테스트 검토 후, 가치 없는 테스트 제거 또는 리팩토링
    • 구현 세부사항을 검증하는 코드 제거 또는 리팩토링
    • 비즈니스를 표현하도록 @DisplayName과 테스트명 변경
    • 모든 테스트에서 arrange 단계에서 TestFactory 객체와 SaveHelper 객체를 이용해서 테스트 픽스쳐 생성
  3. 테스트 픽스쳐 리팩토링
    • 기존에는 EntityManager를 의존하고 있었으나, 추상화인 Repository 의존하도록 변경

close #109