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

[Feat] jacoco & SonarCloud 도입 #8

Merged
merged 12 commits into from
Jun 1, 2024
Merged

[Feat] jacoco & SonarCloud 도입 #8

merged 12 commits into from
Jun 1, 2024

Conversation

jaeuk520
Copy link
Contributor

@jaeuk520 jaeuk520 commented Jun 1, 2024

#️⃣ 이슈

📝 작업사항

  • CI 스크립트 작성
  • Jacoco 도입
  • Sonarcloud 도입

@jaeuk520 jaeuk520 added the 인프라/데브옵스 인프라 / 데브옵스 label Jun 1, 2024
Copy link

sonarqubecloud bot commented Jun 1, 2024

@jaeuk520 jaeuk520 merged commit 07793fe into develop Jun 1, 2024
2 checks passed
@jaeuk520
Copy link
Contributor Author

jaeuk520 commented Jun 1, 2024

테스트코드에서 Thread.sleep()가 문제되는 이유

  • 비동기 테스트에서 테스트 코드는 테스트를 계속하기 전에 비동기 작업이 완료될 때까지 기다릴 수 있도록 작성된다.

  • 이 경우 Thread.sleep을 사용하면 불안정하고, 느리고, 부정확한 테스트 결과를 만들어낼 수 있다. 환경이나 부하에 따라 예기치 않게 실패할 수 있는 취약한 테스트를 생성할 수 있기 때문에 지양하는게 좋다.

@jaeuk520
Copy link
Contributor Author

jaeuk520 commented Jun 1, 2024

리팩토링: 런타임 예외를 발생시킬 수 있는 메서드 호출이 하나만 있도록 수정

  • 코드에서 런타임 예외가 발생하는지 확인할 때는 테스트된 코드 내에서 여러 메서드 호출을 피하고 예외가 발생할 것으로 예상되는 메서드 호출만을 명시하는 것이 좋다.

  • 테스트의 명확성을 높이고 테스트 하고자 하는 메서드가 아닌 다른 메서드가 예외를 잘못 발생시키는 일을 방지한다.

@jaeuk520 jaeuk520 changed the title feat: jacoco & SonarCloud 도입 [Feat] jacoco & SonarCloud 도입 Jun 3, 2024
@jaeuk520 jaeuk520 deleted the feature/ci branch September 4, 2024 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
인프라/데브옵스 인프라 / 데브옵스
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant