Skip to content

[Infra]: Playwright E2E baseline 구축 - #13

Merged
hxm2n merged 2 commits into
mainfrom
feature/#12-playwright-e2e-baseline
Jun 17, 2026
Merged

[Infra]: Playwright E2E baseline 구축#13
hxm2n merged 2 commits into
mainfrom
feature/#12-playwright-e2e-baseline

Conversation

@hxm2n

@hxm2n hxm2n commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

요약

  • Playwright E2E baseline을 추가했습니다.
  • production-equivalent next start 서버 기준 route smoke test를 추가했습니다.
  • PR/push에서 실행되는 최소 CI를 추가했습니다.

포함 범위

  • @playwright/test devDependency 추가
  • playwright.config.ts 추가
  • pnpm test:e2e script 추가
  • /, /오혜민, /dev, /dev/component-preview smoke test 추가
  • Playwright 산출물 gitignore 추가
  • GitHub Actions CI 추가

제외 범위

  • 제품 UI 구현
  • Auth/Login UI
  • Dropdown 접근성 리팩토링
  • route policy abstraction / env flag 정책 계층

검증

  • pnpm lint
  • pnpm build
  • pnpm test:e2e
  • git diff --check

Closes #12

hxm2n added 2 commits June 16, 2026 13:18
Constraint: Playwright route smoke tests must run against production-equivalent Next start without adding product UI scope.\nRejected: Browser-based smoke tests | Route status and SSR content checks do not require browser binaries yet.\nConfidence: high\nScope-risk: narrow\nDirective: Keep this PR limited to test infrastructure and route smoke coverage; defer auth UI, dropdown a11y, and route policy abstraction.\nTested: pnpm lint; pnpm build; pnpm test:e2e; git diff --check\nNot-tested: GitHub Actions execution before PR checks run remotely
Constraint: PR review flagged exact Node version stability and missing minimal workflow permissions.\nRejected: Keeping exact 20.20.2 in CI | setup-node should track valid Node 20 releases reliably.\nConfidence: high\nScope-risk: narrow\nDirective: Keep Playwright webServer build ownership single-sourced in CI; do not expand this PR into product UI work.\nTested: pnpm lint; pnpm build; pnpm test:e2e; git diff --check\nNot-tested: None
@hxm2n

hxm2n commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator Author

Code Review Summary\n\n최종 독립 리뷰 결과입니다.\n\n- code-reviewer recommendation: APPROVE\n- architect status: CLEAR\n- final recommendation: APPROVE\n\n검증 evidence:\n- pnpm lint PASS\n- pnpm build PASS\n- pnpm test:e2e PASS\n- git diff --check PASS\n- GitHub Actions Lint, build, and E2E smoke PASS\n\n리뷰 반영:\n- CI Node 버전을 20.x로 변경\n- workflow 최소 권한 contents: read 추가\n- CI에서 Playwright webServer가 중복 build하지 않도록 정리\n\n현재 PR은 issue #12 범위와 일치하며, 제품 UI/Auth/Dropdown/route policy abstraction 범위는 포함하지 않았습니다.

@hxm2n
hxm2n merged commit 76c4b3c into main Jun 17, 2026
1 check passed
@hxm2n
hxm2n deleted the feature/#12-playwright-e2e-baseline branch June 17, 2026 06:28
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.

[Infra]: Playwright E2E baseline 구축

1 participant