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

Gradle test 작업에 fast-fail 옵션 추가 #987

Open
wants to merge 1 commit into
base: dev/be
Choose a base branch
from

Conversation

zeus6768
Copy link
Contributor

@zeus6768 zeus6768 commented Dec 23, 2024

⚡️ 관련 이슈

#955

📍주요 변경 사항

AS-IS

현재는 테스트 중에 실패하더라도 작업을 멈추지 않고 모든 테스트를 실행해요.

아래 스크린캡처 이미지처럼, 시간이 오래 걸려요.

Screenshot 2024-12-23 at 10 17 37 AM

TO-BE

아래의 gradle 옵션을 추가해요.

test {
   failFast = true
}

Gradle test 작업 실행 중 테스트가 실패하면 작업을 멈춰요.

Screenshot 2024-12-23 at 10 19 11 AM

이렇게하면 실패하는 CI의 작업 속도를 최대 99%(로컬 기준 48.973s → 358ms) 높일 수 있어요.

계산식: $\left( \frac{48973 - 358}{48973} \right) \times 100 \approx 99.27%$

🍗 PR 첫 리뷰 마감 기한

12/24 22:30

@zeus6768 zeus6768 added BE 백엔드 infrastructure 인프라 작업 labels Dec 23, 2024
@zeus6768 zeus6768 added this to the 7차 스프린트 💭 milestone Dec 23, 2024
@zeus6768 zeus6768 self-assigned this Dec 23, 2024
Copy link
Contributor

@HoeSeong123 HoeSeong123 left a comment

Choose a reason for hiding this comment

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

⚡⚡⚡⚡⚡⚡

Copy link
Contributor

@kyum-q kyum-q left a comment

Choose a reason for hiding this comment

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

멋져~~~~~~~

Copy link
Contributor

@jminkkk jminkkk left a comment

Choose a reason for hiding this comment

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

이건 어떻게 안거죠? 최고입니다 👍

Copy link
Contributor

@zangsu zangsu left a comment

Choose a reason for hiding this comment

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

제우스 덕분에 더이상 테스트를 돌리는 것이 무섭지 않아요~~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 infrastructure 인프라 작업
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

5 participants