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

Test: jacoco test coverage에 맞게 테스트 코드 작성 #207

Merged
merged 3 commits into from
Sep 10, 2024

Conversation

adorableco
Copy link
Contributor

@adorableco adorableco commented Sep 9, 2024

PR

✨ 작업 내용

  • jacoco test coverage에 맞게 테스트 코드 작성했습니다.

✨ 참고 사항

  • 자바 lombok 과 같은 경우에는 getter, setter를 제외시키는 설정을 따로 할 수 있는데 kotlin의 프로퍼티는 따로 제외할 수 있는 방법이 없어서 method coverage 퍼센티지를 낮춰놓는 쪽으로 진행했습니다.
  • method coverage : 0.4 , branch coverage : 0.5
    • 개인적으로 branch coverage는 비율을 더 높여도 좋을 것 같습니다!
  • Fix: jacoco test Report 불필요 클래스 제외 #203 에서 gradle의 test 태스크에 jacocoTestReport 생성 프로세스를 모두 합쳤었는데 이렇게 하니 테스트 코드 작성 시에 한 메서드 씩 실행하는 경우 jacocoTestCoverageVerification 을 위반하는 불편함이 있었습니다. (다른 테스트 코드는 실행을 안하니 커버리지 0 달성) 따라서 ./gradlew test 커맨드 시에는 단순 테스트, ./gradlew jacocoTestReport 시에는 테스트 후 jacocoTestReport 를 생성하도록 프로세스를 변경했습니다.

사실 #201 이 PR 테스트해볼 때 전 결과가 계속 똑같이 나와서 이해를 잘못했었는데 알고보니 계속 ./gradlew test jacocoTestReport 로 실행을 해서 그런거더라구요..?!! 허허


⏰ 현재 버그

x


✏ Git Close

@adorableco adorableco added the ✅ Test test 관련(storybook, jest...) label Sep 9, 2024
@adorableco adorableco self-assigned this Sep 9, 2024
@adorableco adorableco linked an issue Sep 9, 2024 that may be closed by this pull request
7 tasks
Copy link

github-actions bot commented Sep 9, 2024

Unit Test Results

18 tests   18 ✔️  2s ⏱️
  7 suites    0 💤
  7 files      0

Results for commit e97038c.

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.

그대는 great girl
그나저나 fixture monkey 이름 참 귀여움🙈

@adorableco adorableco force-pushed the develop branch 13 times, most recently from 6f7e346 to 85aac61 Compare September 10, 2024 01:39
@adorableco adorableco merged commit c576c3a into develop Sep 10, 2024
2 checks passed
@adorableco adorableco deleted the seyeon/swm-170 branch September 10, 2024 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ Test test 관련(storybook, jest...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🧪 Test: Entity Class 테스트 코드
2 participants