Skip to content

test: 병렬 Vite 테스트 캐시 충돌 방지 - #328

Merged
xodn09 merged 1 commit into
developfrom
fix/test-vite-cache-isolation
Oct 21, 2026
Merged

xodn09 merged 1 commit into
developfrom
fix/test-vite-cache-isolation

Conversation

@xodn09

@xodn09 xodn09 commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

변경 요약

fix/test-vite-cache-isolation → develop

릴리스 PR #327의 CI 실행에서 예약 취소 조건 테스트가 시작 중 종료된 원인을 수정합니다.

  • 실패 근거: CI 39525878971의 npm test에서 ENOTEMPTY: directory not empty, rmdir '.../node_modules/.vite/deps'. merchant-reservation-terms.test.mjs가 테스트 등록 전 종료되어 1,123개 중 1,102개만 집계됐습니다.
  • Node 테스트 워커들이 서로 다른 설정의 Vite 서버로 같은 의존성 캐시를 삭제·재생성하고 있었습니다. 프로젝트 설정은 NODE_TEST_CONTEXT가 있는 경우에만 서버별 임시 캐시를 할당합니다.
  • 프로젝트 설정을 읽지 않는 합성 서버도 기본 캐시를 서버별로 격리합니다. 명시한 캐시 경로는 그대로 존중합니다.
  • 일반 개발/운영 build의 기본 캐시, API, 애플리케이션 동작, CI의 테스트 수·병렬 실행·실패 판정은 변경하지 않습니다. 테스트를 생략하거나 재시도로 숨기지 않습니다.
  • 캐시 경로 분리, 동시 SSR 로딩, 명시적 경로 유지, 일반 build 설정 유지에 대한 회귀 검사를 추가합니다.
  • PR fix: 상점주 예약 선택과 관리 화면 UI 개선 #326 첫 실행의 별도 장소 신고 화면 대기 시간 초과까지 해결했다고 주장하지 않습니다.

검증

  • 변경 범위에 맞는 lint, type check, build 또는 직접 검증을 수행했습니다.

  • 수행하지 않은 검증과 이유를 적었습니다.

  • 캐시 격리 회귀 검사와 기존 예약 취소 조건 검사: 26개 통과.

  • 전체 npm test 두 번 연속 실행: 각 1,127개 통과, 실패·skip 없음. 기존 1,123개 및 새 캐시 검사 4개입니다.

  • npm run lint, TypeScript 포함 npm run build, git diff --check 통과.

  • 운영 데이터 변경, Safari/Firefox·실기기 QA는 수행하지 않았습니다. 운영 배포 전 CI를 통과시키기 위한 테스트 인프라 수정입니다.

  • 최신 HEAD 1b384ed의 원격 CI에서 verify·ec2-image 모두 성공했습니다. 전체 브라우저 23개 스위트를 포함합니다.

Web API 계약

  • Web API 영향 없음
  • 영향 operation을 아래에 METHOD /path 형식으로 모두 기록했습니다.
  • 계약표 갱신 여부 검토: endpoint/schema/payload 변경이 없어 해당 없음입니다.
  • 제거·legacy API를 구현 완료로 표시하지 않았습니다.
  • 파괴적 mutation의 확인 수준 검토: 운영 쓰기를 실행하지 않으며 기존 확인 흐름을 변경하지 않습니다.

영향 operation:

  • 없음

관련 이슈

릴리스 반영 결과

  • develop 1eb3234로 병합했고 release PR #327에서 release 21b498c로 병합했습니다.
  • 최신 develop push, release PR, release push CI 모두 verify·ec2-image가 성공했습니다. 테스트 1,127개 및 전체 브라우저 23개 스위트를 포함합니다.
  • Docker 자동 배포 성공과 컨테이너 Healthy·공개 HTTP 검사 성공을 확인했습니다. 일반 운영 빌드의 기본 캐시를 유지하는 검사는 통과했습니다.

@xodn09
xodn09 merged commit 1eb3234 into develop Oct 21, 2026
2 checks passed
@xodn09 xodn09 mentioned this pull request Oct 21, 2026
13 tasks done
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.

1 participant