refactor: 테스트 최적화 #867

20HyeonsuLee · 2024-09-03T07:42:40Z

🔥 연관 이슈

close 테스트 시간을 단축한다. #853

🚀 작업 내용

테스트코드 실행 시간 최적화를 진행했습니다.
MockMvc를 이용하여 테스트코드를 작성하였습니다.
@Transactional을 이용한 롤백테스트로 변경하였습니다.
- 롤백테스트로 변경되었기 때문에 beforeEach로 초기화해주던 테스트데이터를 beforeAll을 이용해서 테스트중 한번만 생성하도록 하였습니다. → beforeEach로 매번 생성하는 비용 감소
이벤트 발송을 검증할 때 다음과 같이 작성하도록 변경되었습니다.
- 이벤트는 기본적으로 after_commit시점에 발송됨.
- 트랜잭션 롤백테스트는 커밋하지 않음. 따라서 이벤트가 발송되지 않음.
- forceVerify함수에서 강제 커밋해서 이벤트를 발송시킨 후 검증함
- 강제 커밋했기 때문에 자동으로 롤백되지 않음. → clear하여 db를 초기화 한 후 setup을 실행하여 수동 롤백을 해줘야함.

forceVerify(() -> verify(coopEventListener).onDiningImageUploadRequest(any()));
        clear();
        setUp();

변경된 테스트를 참고해서 앞으로의 테스트를 작성해주시면 되겠습니다.

맥북 에어 M2 기준 속도

기존 속도

개선 속도

9/10(화) 주간공유때 전체적으로 한번 설명 드리겠습니다~

💬 리뷰 중점사항

테스트 작성된거 보면서 궁금한점 있으면 질문해주세요~

…SDLab/KOIN_API_V2 into task/853-test-optimization-other

github-actions · 2024-09-03T07:48:12Z

Unit Test Results

274 tests 273 ✔️ 1m 0s ⏱️
  30 suites     1 💤
  30 files     0 ❌

Results for commit ddcd062.

♻️ This comment has been updated with latest results.

…' into task/853-test-optimization-admin

…timization' into refactor/shop-test-fix # Conflicts: # src/test/java/in/koreatech/koin/AcceptanceTest.java

duehee

양이 완전 많네요.. 트랙장님 대박😮
고생 많으셨습니다! 간단한 코멘트 남겼으니 확인해주세요~

duehee · 2024-09-09T00:44:56Z

src/test/java/in/koreatech/koin/acceptance/ActivityApiTest.java

 class ActivityApiTest extends AcceptanceTest {

    @Autowired
    protected ActivityFixture activityFixture;

    @Test
-    @DisplayName("BCSD Lab 활동 내역을 조회한다.")
-    void getActivities() {
+    void BCSD_Lab_활동_내역을_조회한다() throws Exception {


A

테스트 컨벤션이 바뀌겠네요 👀
머지 되면 위키 수정하러 가야겠습니당

src/test/java/in/koreatech/koin/acceptance/AuthApiTest.java

src/test/java/in/koreatech/koin/acceptance/MemberApiTest.java

src/test/java/in/koreatech/koin/acceptance/TrackApiTest.java

Choi-JJunho

이렇게 많은양을.. 진짜 고생하셨습니다
몇가지 코멘트 남겼으니 확인 부탁드릴게요~

+) 추가로 PR에 속도가 얼마나 개선되었는지 결과를 알려주면 좋을 것 같아요!

PC 사양, 이전 속도, 개선 후 속도

src/test/java/in/koreatech/koin/AcceptanceTest.java

src/test/java/in/koreatech/koin/acceptance/ArticleApiTest.java

src/test/java/in/koreatech/koin/acceptance/AuthApiTest.java

src/test/java/in/koreatech/koin/acceptance/BusApiTest.java

src/test/java/in/koreatech/koin/acceptance/CoopShopTest.java

src/test/java/in/koreatech/koin/acceptance/DiningApiTest.java

src/test/java/in/koreatech/koin/acceptance/KeywordApiTest.java

src/test/java/in/koreatech/koin/admin/acceptance/AdminShopApiTest.java

src/test/java/in/koreatech/koin/admin/acceptance/AdminTrackApiTest.java

daheeParkk

코드 양이 많네요.. 고생하셨습니다..!

songsunkook

엄청나네요.. ㄷㄷ
고생하셨습니다!

songsunkook · 2024-09-11T06:32:33Z

src/main/java/in/koreatech/koin/admin/shop/service/AdminShopService.java

A

잘 몰라서 혹시 테스트 최적화와 관련있는 내용인가요?

songsunkook · 2024-09-11T06:36:30Z

src/test/java/in/koreatech/koin/AcceptanceTest.java

+    void initIncrement() {
+        dataInitializer.initIncrement();
+        dataInitializer.clearRedis();
+    }


A

resetAutoIncrement같은 네이밍은 어떤가요? 함수 용도를 유추하기 애매한 것 같아요

songsunkook · 2024-09-11T06:37:02Z

src/test/java/in/koreatech/koin/AcceptanceTest.java

+    protected void clear() {
        dataInitializer.clear();
    }


A

이것도 어떤 걸 clear하는지 명시하면 좋겠어요 (ex. dbClear)

songsunkook · 2024-09-11T06:39:04Z

src/test/java/in/koreatech/koin/AcceptanceTest.java

+    protected void forceVerify(Runnable runnable) {
+        TestTransaction.flagForCommit();
+        TestTransaction.end();
+        runnable.run();
+    }


A

이건 어떤 용도인가요?

songsunkook · 2024-09-11T06:43:31Z

src/test/java/in/koreatech/koin/acceptance/TrackApiTest.java

+    @BeforeAll
+    void setup() {
+        clear();
+    }


A

@BeforeAll로 clear()를 호출하는 기준이 있나요? 일부 클래스에는 없는 것 같아서 궁금합니다.

dradnats1012

왕많네요
고생하셨습니당

Choon0414

이 많은 양을 다 하셨다니.. 고생하셨습니다.

HyeonsuLee added 12 commits September 2, 2024 10:04

feat: DB초기화 방식 변경

08adaeb

테스트 수정중

873d16e

feat: BUS 테스트 변

3dddbd6

feat: 게시판 테스트 수

cd4a38b

feat: 게시판 테스트 수정

03e7d43

feat: dining테스트 변

c9020b7

feat: 사용하지 않는 임포트 제거

70bcaf0

feat: 키워드 알림 테스트 수

116044f

feat: 복덕방 테스트 수정

cc6d69d

feat: 멤버 테스트 수정

990340b

feat: pull develop

0968158

Merge branch 'feature/853-test-optimization' of https://github.com/BC…

6a0934d

…SDLab/KOIN_API_V2 into task/853-test-optimization-other

20HyeonsuLee self-assigned this Sep 3, 2024

github-actions bot requested review from duehee and johnny19991006 September 3, 2024 07:42

github-actions bot added the 기능 새로운 기능을 개발합니다. label Sep 3, 2024

krSeonghyeon and others added 13 commits September 3, 2024 19:02

feat: test optimization admin(shop, user 제외)

c15aa33

refactor: 중간 커밋 (Tx 논의중)

8bf2904

feat: 수동 커밋,롤백 테스트 작성

e836f23

feat: test optimization admin(shop 3개제외 완성)

091b2ff

refactor: 불필요 어노테이션 제거

4d3e770

refactor: 충돌해결 위치변경

4565eea

fix: git conflict 해결

482e68b

Merge remote-tracking branch 'origin/task/853-test-optimization-admin…

79c9d0d

…' into task/853-test-optimization-admin

fix: merge 진행

7cc253e

Merge remote-tracking branch 'origin/task/853-test-optimization-admin…

feea3b4

…' into task/853-test-optimization-admin

Merge remote-tracking branch 'refs/remotes/origin/feature/853-test-op…

943ebfa

…timization' into refactor/shop-test-fix # Conflicts: # src/test/java/in/koreatech/koin/AcceptanceTest.java

feat: BeforeAll설정

1786f38

refactor: 테스트 3개 불통 왜 안되지..

7977dac

20HyeonsuLee requested review from ImTotem, Backend-Changyeon, BaeJinho4028, daheeParkk, dradnats1012, Choon0414, kwoo28, Soundbar91, DHkimgit and kih1015 September 6, 2024 11:18

duehee approved these changes Sep 9, 2024

View reviewed changes

HyeonsuLee added 4 commits September 9, 2024 10:56

feat: 트랜잭션 중복선언 제거

2ad37cc

feat: BCSD_Lab → BCSDLab으로 통

ca77bb4

feat: DisplayName

153b529

feat: 반환값 jsonPath이용한 검증으로 변

00d6e3c

Choi-JJunho requested changes Sep 9, 2024

View reviewed changes

feat: 리뷰 반

1327e4f

krSeonghyeon requested review from krSeonghyeon and removed request for krSeonghyeon September 9, 2024 04:15

feat: random part 제거

ddcd062

daheeParkk approved these changes Sep 11, 2024

View reviewed changes

songsunkook approved these changes Sep 11, 2024

View reviewed changes

Choi-JJunho approved these changes Sep 11, 2024

View reviewed changes

dradnats1012 approved these changes Sep 11, 2024

View reviewed changes

Choon0414 approved these changes Sep 12, 2024

View reviewed changes

20HyeonsuLee merged commit 76bcbdf into develop Sep 12, 2024
4 checks passed

20HyeonsuLee deleted the feature/853-test-optimization branch September 12, 2024 14:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor: 테스트 최적화 #867

refactor: 테스트 최적화 #867

20HyeonsuLee commented Sep 3, 2024 •

edited

Loading

github-actions bot commented Sep 3, 2024 •

edited

Loading

duehee left a comment

duehee Sep 9, 2024

Choi-JJunho left a comment

daheeParkk left a comment

songsunkook left a comment

songsunkook Sep 11, 2024

songsunkook Sep 11, 2024

songsunkook Sep 11, 2024

songsunkook Sep 11, 2024

songsunkook Sep 11, 2024

dradnats1012 left a comment

Choon0414 left a comment

refactor: 테스트 최적화 #867

refactor: 테스트 최적화 #867

Conversation

20HyeonsuLee commented Sep 3, 2024 • edited Loading

🔥 연관 이슈

🚀 작업 내용

맥북 에어 M2 기준 속도

기존 속도

개선 속도

💬 리뷰 중점사항

github-actions bot commented Sep 3, 2024 • edited Loading

Unit Test Results

duehee left a comment

Choose a reason for hiding this comment

duehee Sep 9, 2024

Choose a reason for hiding this comment

A

Choi-JJunho left a comment

Choose a reason for hiding this comment

daheeParkk left a comment

Choose a reason for hiding this comment

songsunkook left a comment

Choose a reason for hiding this comment

songsunkook Sep 11, 2024

Choose a reason for hiding this comment

A

songsunkook Sep 11, 2024

Choose a reason for hiding this comment

A

songsunkook Sep 11, 2024

Choose a reason for hiding this comment

A

songsunkook Sep 11, 2024

Choose a reason for hiding this comment

A

songsunkook Sep 11, 2024

Choose a reason for hiding this comment

A

dradnats1012 left a comment

Choose a reason for hiding this comment

Choon0414 left a comment

Choose a reason for hiding this comment

20HyeonsuLee commented Sep 3, 2024 •

edited

Loading

github-actions bot commented Sep 3, 2024 •

edited

Loading