Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

단위 테스트(jest 테스트) 실행 시간 최적화 #834

Merged
merged 11 commits into from
Nov 9, 2023
Merged

Conversation

inyeong-kang
Copy link
Member

🔥 연관 이슈

close: #832

📝 작업 요약

단위 테스트(jest 테스트) 실행에 소요되는 시간을 줄였습니다.

⏰ 소요 시간

2시간

🔎 작업 상세 설명

  • MSW로 서버 통신을 모킹하는 과정에서 delay를 주고 있는데, 이 delay가 jest 테스트할 때도 반영되고 있어 테스트 시간이 길어졌습니다. 따라서 jest 테스트하는 경우에는 .env.testVOTOGETHER_MOCKING_DELAY 값이 delay 값이 되도록 환경변수를 추가했습니다. (이제 jest 테스트 시 delay 값은 0이 됩니다. 43초 -> 36초로, 약 16% 단축)
  • test 실행 명령어로 --watch를 추가하여 변경 사항이 있는 테스트 파일만 테스트하도록 하였습니다. (43초 -> 16초로, 약 62% 단축)
  • test coverage(실제 테스트에 반영되는 코드)를 테스트해보고, coverage를 높이기 위해 jest.config.tscollectCoverageFrom 옵션을 추가했습니다. (src/components, src/pages 의 코드는 테스트에 포함되지 않음. jest 테스트에 UI 관련 코드를 사용할 일이 거의 없기 때문.)

🌟 논의 사항

없음.

Copy link

github-actions bot commented Nov 3, 2023

⚡️ Lighthouse report!

Category Score
🟠 Performance 62
🟠 Accessibilty 89
🟢 SEO 100
🟠 PWA 89
Category Score
🟢 First Contentful Paint 0.7 s
🟠 Largest Contentful Paint 3.5 s
🔴 Total Blocking Time 1,970 ms
🟢 Cumulative Layout Shift 0.046
🟠 Speed Index 3.4 s

Copy link
Collaborator

@Gilpop8663 Gilpop8663 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 테스트 속도를 개선하고 싶다는 생각이 있었는데요. 제로가 먼저 제안해주셔서 너무 좋았어요

그리고 혼자서는 어떻게 해야할 지 생각이 안났는데요. 같이하니깐 좋았습니다 😀

Copy link
Collaborator

@chsua chsua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

해당 부분을 개선할 수 있는지 몰랐네요!
너무 좋아요.
고생하셨습니다!

@inyeong-kang inyeong-kang merged commit db4d7e2 into dev Nov 9, 2023
1 check passed
@inyeong-kang inyeong-kang deleted the feat/#832 branch November 9, 2023 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

단위 테스트(jest 테스트) 최적화
3 participants