Skip to content

[#40] 배포 단계 테스트 실행 추가#41

Merged
1117mg merged 1 commit intodevfrom
feat/40-build-run-test
Feb 4, 2026
Merged

[#40] 배포 단계 테스트 실행 추가#41
1117mg merged 1 commit intodevfrom
feat/40-build-run-test

Conversation

@1117mg
Copy link
Member

@1117mg 1117mg commented Feb 4, 2026

작업 내용

  • 배포 단계에서 테스트 돌려서 api.yaml 생성하도록 함

참고

  • Gradle 캐싱을 위한 jdk 설치 단계 추가

관련 이슈

@1117mg 1117mg requested review from earlgrey02 and mjk25 February 4, 2026 12:11
@1117mg 1117mg self-assigned this Feb 4, 2026
@1117mg 1117mg added the feature 기능 구현 label Feb 4, 2026
@1117mg 1117mg linked an issue Feb 4, 2026 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

🤖 Gemini Code Review

✅ 변경 사항 요약

  • CI/CD 배포 워크플로우에 JDK 21 환경 설정이 추가되었습니다.
  • Docker 이미지 빌드 전 API 모듈 테스트 실행 단계가 도입되었습니다.

🚨 치명적인 이슈 (Critical Issues)

  • 🛑 .github/workflows/deploy.yml
    • Problem: Run tests 단계(:api:test)는 테스트를 실행할 뿐, 배포를 위한 최종 실행 가능한 아티팩트(예: JAR 파일)를 생성하지 않는다. 이후 Build and push API 단계에서 Dockerfile이 워크스페이스에서 사전 빌드된 JAR 파일을 필요로 할 경우, 아티팩트가 누락되어 Docker 빌드가 기능적으로 실패한다.
    • Fix: 테스트 실행 단계와 Docker 빌드 단계 사이에 build 또는 bootJar 태스크를 명시적으로 실행하여 최종 아티팩트 생성을 보장해야 한다. 예: ./gradlew :api:bootJar --no-daemon --info.

✏️ 파일별 상세 리뷰

.github/workflows/deploy.yml

  • 📍 Issue: 배포 안정성 확보를 위해 테스트 단계를 추가한 것은 타당하나, 테스트만으로는 아티팩트 생성이 보장되지 않아 후속 Docker 빌드 단계의 실패를 야기하는 논리적 오류가 발생한다.
  • Improvement: JDK 설정 시 java-version: 21을 명시했으므로, 향후 Gradle 빌드가 해당 버전으로 일관되게 실행되는지 확인이 필요하다. (현재는 테스트 단계의 기능적 오류가 더 심각함.)

Copy link
Collaborator

@mjk25 mjk25 left a comment

Choose a reason for hiding this comment

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

👍👍

@1117mg 1117mg merged commit ab180ef into dev Feb 4, 2026
1 check passed
@1117mg 1117mg deleted the feat/40-build-run-test branch February 4, 2026 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

빌드 단계 테스트 실행 추가

2 participants