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

[BE/feat] CI개선 : JPA Entity와 Schema 검증 및 테스트 커버리지 제한 추가 #329

Merged
merged 7 commits into from
Jun 29, 2024

Conversation

hwangdaesun
Copy link
Collaborator

@hwangdaesun hwangdaesun commented Jun 28, 2024

⚠️ 관련 이슈

#321

📢 주요 변경사항

  • 스키마 불일치 이슈로 인해 서버가 과부화되는 문제가 발생하여 CI시 스키마 검증하는 것이 필요하다고 판단 이에, SchemaValidationTest 클래스를 생성하여 CI시 resources/gaebaljip-develop-environment/mariadb-init/01_schema.sql과 JPA Entity가 일치하는 지 검증
  • Approve된 이후 커밋이 추가되면 해당 커밋은 워크 플로우에 반영되지 못 함 이에 CI/CD Trigger를 Approve 방식에서 PR이 Merge되면 수행되는 방식으로 변경
  • 현재 개발 서버이기때문에 always나 on-failure로 할 필요가 없다고 판단 이에 docker-compose.yml의 restart 속성을 no로 수정
  • 테스트 코드 작성이 되어있지 않아 테스트 코드 작성을 강제 및 독려하기 위해 CI Step에 Jacoco를 이용한 최소 테스트 커버리지 제한 추가 및 PR Comment에 테스트 커버리지 리포트 생성

@hwangdaesun hwangdaesun requested a review from LJH098 June 28, 2024 04:36
@hwangdaesun hwangdaesun self-assigned this Jun 28, 2024
@hwangdaesun hwangdaesun changed the title [BE/feat] CI/CD 개선: JPA Entity 검증, Trigger 수정, Docker-compose 설정 업데이트, 최소 테스트 커버리지 제한 추가 [BE/feat] CI/CD 개선: JPA Entity와 Schema 검증 및 최소 테스트 커버리지 제한 추가 Jun 28, 2024
@hwangdaesun hwangdaesun changed the title [BE/feat] CI/CD 개선: JPA Entity와 Schema 검증 및 최소 테스트 커버리지 제한 추가 [BE/feat] CI개선 : JPA Entity와 Schema 검증 및 테스트 커버리지 제한 추가 Jun 28, 2024
Copy link
Collaborator

@LJH098 LJH098 left a comment

Choose a reason for hiding this comment

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

와우 굿굿 근데 Jacoco가 뭐하는 녀석인가요??

@hwangdaesun
Copy link
Collaborator Author

와우 굿굿 근데 Jacoco가 뭐하는 녀석인가요??

Jacoco는 Java 코드 커버리지 체크하는 라이브러리인데, 테스트 코드를 돌리면 해당 커버리지에 대해서 아래와 같이 html 형식으로 만들어줍니다

image

추가로, 테스트 커버리지가 30% 아래면 CI 통과 못 하게끔 막아뒀습니다~

@hwangdaesun hwangdaesun merged commit 9ff1c0f into be Jun 29, 2024
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