Skip to content

[URH-69] 테스트 커버리지 신규 #52

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

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

foresec
Copy link
Collaborator

@foresec foresec commented Aug 22, 2024

👾 Pull Request

  • 상태: 신규

1️⃣ Spec

  • npm run coverage로 Jest의 테스트 커버리지를 확인할 수 있습니다.
  • mock파일 제외
  • coverageThreshold에 개발 기간 내 잠정적으로 4가지 수치를 전부 70%로 설정, 리팩토링 기간에 90%로 변경
  • 추후 커버리지 수치 검증 자동화를 위해 다양한 커버리지 리포터 설정 유지

2️⃣ 변경 사항

  • gitignore에 coverage폴더 추가

3️⃣ 예시 코드

테스트 커버리지의 결과는

  1. 콘솔창에서 요약된 형태로 확인하거나
  2. 생성된 coverage/lcov-report안의 파일을 확인해서 세부적인 사항을 확인 가능함

4️⃣ 관련 문서 (선택 사항)

@foresec foresec self-assigned this Aug 22, 2024
@foresec foresec changed the title ✅ test: 테스트 커버리지 관련 설정 [URH-69] 테스트 커버리지 관련 설정 Aug 22, 2024
@foresec foresec changed the title [URH-69] 테스트 커버리지 관련 설정 [URH-69] 테스트 커버리지 관련 설정 Aug 22, 2024
@foresec foresec changed the title [URH-69] 테스트 커버리지 관련 설정 [URH-69] 테스트 커버리지 신규 Aug 22, 2024
Copy link
Member

@bicochan bicochan left a comment

Choose a reason for hiding this comment

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

소현 님 최고!👍🏻🔥

@foresec foresec requested review from suhyeoonn and Choozii August 23, 2024 04:51
Copy link
Member

@jeongbaebang jeongbaebang left a comment

Choose a reason for hiding this comment

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

👍👍

@foresec foresec merged commit 0a649ca into master Aug 23, 2024
7 checks passed
@foresec foresec deleted the URH-69/test-coverage-setting branch August 23, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants