[FE] ci 워크플로우 구축 #40
[FE] ci 워크플로우 구축 #40
Conversation
Yoonkyoungme
requested review from
hwinkr,
Largopie,
ehBeak,
ikjo39,
hw0603,
seokmyungham and
seunghye218
as code owners
Test Results0 tests 0 ✅ 0s ⏱️ Results for commit fc70cce. ♻️ This comment has been updated with latest results. |
| run: npm ci | ||
|
|
||
| - name: RTL을 실행해요 :) | ||
| run: npm run test -- --passWithNoTests |
There was a problem hiding this comment.
아무런 테스트를 진행하지 않는 경우에도, CI 파이프라인이 통과될 수 있도록 하기 위한 설정이군요!
설정 고생하셨습니다 빙봉~ 🔮
There was a problem hiding this comment.
맞습니다!
PR을 날린 시점에서 아래 사진처럼 테스트가 존재하지 않아서 CI 파이프라인이 실패하였었는데, 이러한 현상을 방지하기 위해 --passWithNoTests 옵션을 추가했습니다.
--passWithNoTests: 테스트 파일이 없더라도 테스트를 통과시키도록 하는 옵션
이 옵션을 사용하면, 테스트를 실행할 때 실제로 테스트 파일이 하나도 없더라도 Jest가 에러를 발생시키지 않고 성공 코드로 종료됩니다.
| uses: EnricoMi/publish-unit-test-result-action@v2 | ||
| if: always() | ||
| with: | ||
| files: frontend/test-results/results.xml |
There was a problem hiding this comment.
PR 본문에 테스트 결과 표시를 위해서는, jest-junit 의존성을 추가하고 테스트 완료 시, xml 파일 생성 후 테스트 결과를 작성하도록 해야 함!
(기록용 코멘트 입니다 ㅎㅎ)
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: "lts/*" |
There was a problem hiding this comment.
노드 버전 설정 관련돼서 설정하는 방법이 https://github.com/actions/setup-node 여기 명시돼있네요!
|
@seokmyungham 재즈! |
Yoonkyoungme
dismissed stale reviews from ikjo39, seunghye218, seokmyungham, hw0603, ehBeak, and Largopie
via
4cc9703
Jest 실행 시 `--passWithNoTests` 옵션 추가
- 'detect-changes' 작업 추가로 변경 사항 감지 및 조건부 실행 구현 - 'fe-test' 작업을 변경된 파일이 'frontend/**' 경로에 있을 때만 실행되도록 설정
Yoonkyoungme
force-pushed
the
feat/37-fe-ci
branch
from
4cc9703
to
fc70cce
Compare
관련 이슈
작업 내용
frontend디렉토리에 대해 GitHub Actions를 이용한 CI 워크플로우를 설정하였습니다.특이 사항
리뷰 요구사항 (선택)