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

[BE] feat(#610): 테스트 코드 시간 최적화 #628

Merged
merged 8 commits into from
Oct 6, 2024

Conversation

rndudals
Copy link
Contributor

@rndudals rndudals commented Sep 15, 2024

#️⃣ 연관된 이슈

ex) #610

Pull Request Type

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 코드 리팩토링
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 테스트 추가, 테스트 리팩토링
  • 코드에 영향을 주지 않는 변경사항
    • 오타 수정, 문서 수정, 변수명 변경, 주석 추가 및 수정

Pull Request Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 변경 사항에 대한 테스트를 모두 통과했나요?
  • 코드 정렬은 적용했나요?
  • 새로운 branch에 작업 후 dev로 PR을 요청했나요?

📝 작업 내용

변경 내용

  • 컨트롤러단의 @MockBean을 하나의 설정 클래스 안에 묶기
  • @DirtiesContext 제거

결과

  • 테스트 코드를 50초 → 30초(40%) 단축
  • Context Init Count도 23개 → 10개(57%) 단축

💬 리뷰 요구사항

리뷰 부탁드립니다~!

@rndudals rndudals self-assigned this Sep 15, 2024
@rndudals rndudals changed the base branch from main to dev September 15, 2024 11:36
Copy link

github-actions bot commented Oct 6, 2024

📝 테스트 커버리지 리포트

Overall Project 78.32% 🍏

There is no coverage information present for the Files changed

@rndudals rndudals changed the title [BE] feat(#610): 테스트 최적화 [BE] feat(#610): 테스트 코드 시간 최적화 Oct 6, 2024
@rndudals rndudals merged commit e846651 into dev Oct 6, 2024
3 checks passed
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.

1 participant