Skip to content

Feat/admin util tests from develop#421

Merged
hexdrinker merged 6 commits into
developfrom
feat/admin-util-tests-from-develop
May 2, 2026
Merged

Feat/admin util tests from develop#421
hexdrinker merged 6 commits into
developfrom
feat/admin-util-tests-from-develop

Conversation

@hexdrinker

Copy link
Copy Markdown
Collaborator

개요

admin 영역의 유틸 함수 테스트를 전반적으로 보강하고, 실제 사용처에서 유틸 동작이 연결되는지 통합 테스트를 추가했습니다.
함께 포함된 기존 브랜치 변경(ShareBanner, Cast API, CI notify)도 포함되어 있습니다.

주요 변경사항

1) util 테스트 인프라/의존성 추가

  • apps/admin/package.json
    • vitest 테스트 스크립트 추가
    • jsdom, @testing-library/react, @testing-library/dom 추가
  • .pnp.cjs, yarn.lock 갱신

2) util 유닛 테스트 추가

  • apps/admin/src/utils/validation.test.ts
  • apps/admin/src/utils/replace.test.ts
  • apps/admin/src/utils/format.test.ts
  • apps/admin/src/utils/phone.test.ts
  • apps/admin/src/utils/link.test.ts
  • apps/admin/src/utils/app.test.ts
  • apps/admin/src/utils/boldText.test.ts
  • apps/admin/src/utils/highlight.test.tsx
  • apps/admin/src/utils/vConsole.test.ts

3) util 관련 통합 테스트 추가

  • apps/admin/src/components/ReservationTable/ReservationTable.integration.test.tsx
  • apps/admin/src/components/EnteranceTable/EnteranceTable.integration.test.tsx
  • apps/admin/src/components/MobileCardList/MobileCardList.integration.test.tsx
  • apps/admin/src/pages/AppStoreBridge/AppStoreBridge.integration.test.tsx

4) 유틸 동작 보정

  • apps/admin/src/utils/format.ts
    • 짧은 번호/옵셔널 매칭 구간에서 --가 생기지 않도록 안전 포맷팅 로직으로 개선

5) 브랜치에 함께 포함된 기존 변경

  • apps/admin/src/hooks/useShareBannerVisibility.ts
  • apps/admin/src/components/ShareBanner/ShareBanner.styles.ts
  • apps/admin/src/components/ShowCastInfoFormDialogContent/index.tsx
  • .github/workflows/notify-policy-changes.yml

검증

  • yarn workspace admin test
  • 결과: 13 files, 65 tests passed

참고

  • 워크트리에 /.cursor, /.omx는 untracked 상태이며 PR 변경 파일에는 포함하지 않았습니다.

@hexdrinker hexdrinker merged commit e42e81f into develop May 2, 2026
1 check passed
@hexdrinker hexdrinker deleted the feat/admin-util-tests-from-develop branch May 2, 2026 06:36
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