Skip to content

chore: 소나클라우드 추가#91

Draft
yeongunheo wants to merge 9 commits intodevelopfrom
feature/yeongun/apply-sonarcloud
Draft

chore: 소나클라우드 추가#91
yeongunheo wants to merge 9 commits intodevelopfrom
feature/yeongun/apply-sonarcloud

Conversation

@yeongunheo
Copy link
Contributor

No description provided.

@yeongunheo yeongunheo self-assigned this Sep 30, 2024
@github-actions
Copy link

github-actions bot commented Sep 30, 2024

Unit Test Results

64 tests   64 ✔️  3s ⏱️
16 suites    0 💤
16 files      0

Results for commit 01cec22.

♻️ This comment has been updated with latest results.

@yeongunheo yeongunheo marked this pull request as draft September 30, 2024 02:10
@jickDo
Copy link
Contributor

jickDo commented Oct 1, 2024

현재 상황이 소나클라우드가 작동되는 상환인지 판단이 잘안서서 브랜치 pull해서 이것저것 테스트 해보고 의견 남겨봅니다!

먼저 에러로그중 domain영역의 main시작점을 못찾는 다는 것을 확인하여

`tasks.bootJar {
enabled = false
}

tasks.jar {
enabled = true
}`

domain영역은 메인이 없다는 것을 명시적으로 선언하고, ./gradlew build jacocoTestReport sonar --scan --info 이 명령어를 통해 분석을 수행했습니다.

image

사진처럼 ui상 테스트가 출력되는것 까지 확인했습니다.

@jickDo
Copy link
Contributor

jickDo commented Oct 1, 2024

추가적으로, 테스트 환경에서 Failed가 많은것은 의존 방향에 대한 문제라는 생각이 듭니다.

제가 생각한 문제와 해결방법은, 현재 테스트 케이스 통과한 경우의 상황들은 전부 domain영역의 리소스 자원만 사용한 경우라고 보입니다.
즉, 실패 한 경우는 domain영역 외부 리소스를 사용한 경우라는 생각이들고, 여기서 말하는 외부 리소스는 external-api영역의 특정 서비스 들인것입니다.
증명을 해보고 싶어서 domain gradle에서 external-api영역을 import하려고 했지만 모듈 상호 참조 문제로 컴파일 자체가 불가하다고 합니다.

여기서 부터는 멀티모듈 구조를 개선하거나, 테스트 의존성에 대해 고민해보아야 할것같다는 생각이 들고, 이부분에 대해서는 영운님의 의견이 필요하다고 생각해 이부분까지만 진행했습니다!

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.

2 participants