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

Fix: jacoco test Report 불필요 클래스 제외 #203

Merged
merged 3 commits into from
Sep 4, 2024

Conversation

adorableco
Copy link
Contributor

@adorableco adorableco commented Sep 4, 2024

PR

✨ 작업 내용

  • 🛠️ Fix: jacoco 테스트 레포트 설정 변경 #200 이슈에 이어서 jacocoTestReport 에 표시될 필요가 없는 entity 패키지 외의 클래스들, queryDSL 을 위한 Q~ 클래스 를 제외했습니다.
  • 또한 앞으로 테스트 커버리지를 어느정도 보장하면서 개발을 하고자 jacocoTestCoverageVerification 에서도 위와 동일하게 클래스들을 제외했습니다. ➡️ 제외하지 않을 경우 쓸데 없는 모든 클래스의 테스트를 해야할 수 있기 때문

✨ 참고 사항

  • 아직 QuestionSet 등 테스트를 진행하지 않은 클래스가 있기 때문에 브랜치 커버리지, 메서드 커버리지 모두 0 으로 설정해뒀습니다. 조만간 필요한 테스트코드를 모두 작성한 후에 커버리지 퍼센티지를 정해야 할 것 같습니당!

  • build.gradle 내의 태스크들 간의 실행 프로세스를 지정했기 때문에 develop-ci.yml 37번 줄에서 run: ./gradlew clean build test jacocoTestReportrun: ./gradlew clean build test 로 변경했습니다.

🙋 변경 전

스크린샷 2024-09-02 오후 11 43 01

🙋 변경 후

스크린샷 2024-09-04 오후 2 12 12

⏰ 현재 버그

x


✏ Git Close

@adorableco adorableco added the 🛠️ Fix Something isn't working label Sep 4, 2024
@adorableco adorableco self-assigned this Sep 4, 2024
@adorableco adorableco linked an issue Sep 4, 2024 that may be closed by this pull request
2 tasks
Copy link

github-actions bot commented Sep 4, 2024

Unit Test Results

14 tests   14 ✔️  2s ⏱️
  6 suites    0 💤
  6 files      0

Results for commit 5b6e932.

Copy link
Contributor

@RinRinPARK RinRinPARK left a comment

Choose a reason for hiding this comment

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

규우우욷👍

@adorableco adorableco merged commit 29cb88c into develop Sep 4, 2024
2 checks passed
@adorableco adorableco deleted the seyeon/swm-169 branch September 4, 2024 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ Fix Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🛠️ Fix: jacoco 테스트 레포트 불필요한 클래스 제외
2 participants