Skip to content

fix(ci): handle no frontend test files#226

Open
kdh-92 wants to merge 1 commit intomainfrom
fix/ci-passWithNoTests
Open

fix(ci): handle no frontend test files#226
kdh-92 wants to merge 1 commit intomainfrom
fix/ci-passWithNoTests

Conversation

@kdh-92
Copy link
Copy Markdown
Owner

@kdh-92 kdh-92 commented Apr 9, 2026

Summary

  • vitest run --passWithNoTests 사용하여 테스트 파일이 없을 때 CI 실패 방지
  • ci.yml과 main.yml 모두 수정

Test plan

  • CI 워크플로우 통과 확인

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • CI/CD 파이프라인 최적화를 위해 프론트엔드 테스트 실행 도구 업데이트

Vitest exits with code 1 when no test files found. Use --passWithNoTests
flag to allow CI to pass until test files are added.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0767a23e-8a89-4c1b-a6be-717dc774a285

📥 Commits

Reviewing files that changed from the base of the PR and between 809ce07 and 07120ff.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • .github/workflows/main.yml

개요

두 개의 GitHub Actions 워크플로우 파일에서 프론트엔드 테스트 명령을 npm test에서 npx vitest run --passWithNoTests로 변경하여 Vitest 테스트 러너를 사용하고 테스트를 찾지 못할 때도 성공으로 처리합니다.

변경 사항

Cohort / File(s) Summary
GitHub Actions 워크플로우
.github/workflows/ci.yml, .github/workflows/main.yml
프론트엔드 테스트 단계에서 테스트 실행 명령을 npm test에서 npx vitest run --passWithNoTests로 변경하여 Vitest 기반 테스트 실행으로 전환하고 테스트 미발견 시에도 성공 처리

예상 코드 리뷰 난이도

🎯 1 (Trivial) | ⏱️ ~3 minutes

깡깡깡! 🐰 워크플로우를 손질하네,
Vitest로 갈아타는 중이야,
테스트 없어도 괜찮다며,
CI/CD가 더 부드러워졌구나! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: updating CI workflows to handle situations where no frontend test files are present by using the --passWithNoTests flag.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-passWithNoTests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kdh-92 kdh-92 enabled auto-merge (squash) April 9, 2026 07:44
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