Skip to content

[YS-39] chore: JaCoCo 및 소나클라우드 연동#10

Merged
chock-cho merged 9 commits intodevfrom
feature/YS-39
Dec 31, 2024
Merged

[YS-39] chore: JaCoCo 및 소나클라우드 연동#10
chock-cho merged 9 commits intodevfrom
feature/YS-39

Conversation

@Ji-soo708
Copy link
Member

@Ji-soo708 Ji-soo708 commented Dec 29, 2024

💡 작업 내용

  • 코드 커버리지 측정을 위한 JaCoCo 및 소나클라우드 연동

✅ 셀프 체크리스트

  • PR 제목을 형식에 맞게 작성했나요?
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있나요?
  • 테스트는 잘 통과했나요?
  • 빌드에 성공했나요?
  • 본인을 assign 해주세요.
  • 해당 PR에 맞는 label을 붙여주세요.

🙋🏻‍ 확인해주세요

  • 일단은 domain/usecase를 위주로 테스트한다는 전제 하에 구성했습니다
  • 제가 임의로 커버리지 통과 기준을 설정했는데 이 부분은 좀 더 논의가 필요할 것 같습니다. 추가적으로 제안하고 싶은 사항이 있으면 코멘트로 남겨주세요!

🔗 Jira 티켓


https://yappsocks.atlassian.net/browse/YS-39

@Ji-soo708 Ji-soo708 added the ⚙️ CHORE config, workflow.yaml label Dec 29, 2024
@Ji-soo708 Ji-soo708 self-assigned this Dec 29, 2024
@github-actions github-actions bot changed the title chore: JaCoCo 및 소나클라우드 연동 [YS-39] chore: JaCoCo 및 소나클라우드 연동 Dec 29, 2024
@sonarqubecloud
Copy link

Copy link
Member Author

@Ji-soo708 Ji-soo708 left a comment

Choose a reason for hiding this comment

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

주소
https://sonarcloud.io/project/overview?id=YAPP-Github_25th-Web-Team-2-BE

플로우
test → 테스트 실행 및 Jacoco 커버리지 측정
jacocoTestReport → Jacoco 리포트 생성 (HTML 및 XML)
jacocoTestCoverageVerification → 커버리지 기준 검증 (기준에 미달하면 경고)

측정 기준

  • usecase 테스트코드만 측정
  • 클래스 커버리지 최소 70% 이상이어야 통과
  • 브랜치 커버리지 최소 70% 이상이어야 통과

위와 같이 설정했습니다! 혹시 더 설명이 필요한 부분이 있거나 수정하고 싶은 부분이 있다면 코멘트로 남겨주세요~

@Ji-soo708 Ji-soo708 requested a review from chock-cho December 29, 2024 07:50
Copy link
Member

@chock-cho chock-cho left a comment

Choose a reason for hiding this comment

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

소나 클라우드라는 툴을 처음 알아서 관련 레퍼런스들 좀 서치해가며 차차 공부해보고 있는데, 되게 유용하다는 걸 알게 되었습니다 ☺️
수고 많으셨습니다!

@chock-cho chock-cho merged commit f25b368 into dev Dec 31, 2024
3 checks passed
@chock-cho chock-cho deleted the feature/YS-39 branch December 31, 2024 07:02
Ji-soo708 added a commit that referenced this pull request Jan 26, 2025
* chore: add Jacoco test coverage reporting and verification

* chore: update ci script for jacoco

* fix: sonar sources path

* chore: update version in ci script

* chore: add upload problems report as artifact step

* chore: add continue on error in ci

* chore: update version

* chore: update Java toolchain configuration

* chore: delete upload problems report step
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙️ CHORE config, workflow.yaml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants