feat: PR 생성 시 테스트 자동 실행하는 CI 워크플로우 추가#389
Open
pooreumjung wants to merge 2 commits into
Open
Conversation
resolved #385 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
코드 리뷰 반영: EATSSU_JWT_SECRET_TEST 하드코딩 값 제거, openssl로 매 실행마다 생성 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#️⃣ Issue Number
📝 요약(Summary)
develop,main으로 생성될 때./gradlew test를 자동 실행하는 CI 워크플로우(.github/workflows/ci.yml) 추가deploy.yml은 빌드 시-x test로 테스트를 완전히 스킵하고 있어, 테스트가 자동으로 실행되는 지점이 전혀 없었음 → 이번 PR로 PR 단계에서의 테스트 게이트 신설application-test.yml이 실제 MySQL 연결을 요구하므로, GitHub Actions service container로 임시 MySQL 8.0을 띄워 테스트 실행 (health check로 DB 준비 대기)openssl rand -base64 64로 생성해$GITHUB_ENV에 주입deploy.yml(push 트리거, 빌드+배포)은 변경하지 않고 완전히 별도 워크플로우로 분리💬 공유사항 to 리뷰어
./gradlew test를 미리 실행해 검증했습니다. 84개 중 77개 통과, 나머지 7개는 이 PR과 무관한 기존 버그(Ratings.of()NPE,NicknameValidator정규식 버그)로 확인되어 각각 별도 fix 이슈(fix: Ratings.of()에서 amountRating/tasteRating null 시 NullPointerException 발생 #387, fix: NicknameValidator 시작/끝 문자 검증 정규식에 숫자가 허용되어 있어 검증 누락 #388)로 분리했습니다. 즉 이 PR을 머지하면 CI가 곧바로 실패 상태로 뜰 수 있으니, fix: Ratings.of()에서 amountRating/tasteRating null 시 NullPointerException 발생 #387/#388이 먼저 머지되거나 병행 진행이 필요합니다.root계정을 그대로 사용합니다. CI 한정이라 문제는 없지만, 운영/dev 계정과 권한 차이가 있을 수 있다는 점 참고 바랍니다.✅ PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.