Skip to content

test: 알림·마이페이지 회원 여정 E2E 시나리오 추가 - #719

Merged
kanghaeun merged 6 commits into
devfrom
test/716-notification-mypage-e2e
Sep 28, 2026
Merged

kanghaeun merged 6 commits into
devfrom
test/716-notification-mypage-e2e

Conversation

@kanghaeun

@kanghaeun kanghaeun commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

작업 요약

  • 알림 목록·마이페이지·닉네임 수정·탈퇴의 회원 여정을 Playwright 로 고정
  • 요청 내용과 실패 경로 위주로 11케이스 선별

작업 세부 내용

시나리오

파일 케이스 검증
notificationList.spec.ts 알림 3종 클릭 읽음 요청 ids·목적지 이동 (create·result·create?highlightItem)
notificationList.spec.ts 공지 클릭 읽음 요청만 발생·URL 유지
notificationList.spec.ts 모두 읽음 all: true 요청·토스트·버튼 소멸
notificationList.spec.ts 500 후 다시 시도 에러 카드에서 재조회로 복구
mypageMember.spec.ts 회원 화면·로그아웃 프로필·계정 메뉴 렌더·로그아웃 후 쿠키 삭제·로그인 도착
nicknameEdit.spec.ts 중복 닉네임 available: false 응답을 카탈로그 문구로 안내·버튼 비활성
nicknameEdit.spec.ts 수정 성공 PATCH body·마이페이지 복귀·새 닉네임 반영
withdraw.spec.ts 탈퇴 확정 DELETE 요청·세션 폐기·로그인 도착
withdraw.spec.ts 403 거절 카탈로그 문구 토스트·다이얼로그 닫힘·화면 유지
  • highlightItem 케이스는 최종 URL 대신 네비게이션 요청으로 검증
    • 원인: 목적지 페이지가 쿼리를 읽은 뒤 URL 에서 지워 최종 URL 단언이 타이밍에 흔들림
  • 로그아웃 스텁은 Set-Cookie 로 토큰을 삭제
    • 원인: 실서버가 httpOnly 토큰을 응답 헤더로 지우는 방식과 동일하게 맞춤

목 스텁 keep-alive 유휴 타임아웃 비활성화

  • 원인: 서버가 유휴 소켓을 닫는 순간 serverApi 가 재사용하면 ECONNRESET 으로 SSR 이 에러 화면으로 떨어짐
    • 결과: 기존 토너먼트 spec 2개가 이 원인으로 흔들림

storageState 인트로 온보딩 열람 추가

범위

연관 이슈

closes #716

Summary by CodeRabbit

  • 테스트
    • 회원 마이페이지에서 프로필·계정 메뉴 확인, 닉네임 변경, 로그아웃 및 회원 탈퇴 흐름을 검증하는 시나리오를 추가했습니다.
    • 탈퇴 성공 시 로그인 화면으로 이동하고 세션 쿠키가 삭제되는 동작과, 게스트 계정의 탈퇴 제한 및 오류 안내를 확인합니다.
    • 알림 유형별 이동 및 읽음 처리, 전체 읽음 처리, 알림 조회 실패 후 재시도 동작을 검증합니다.

@vercel

vercel Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
piki Ready Ready Preview Sep 28, 2026 10:26am UTC
1 Skipped Deployment
Project Deployment Actions Updated
piki-notify Skipped Skipped Sep 28, 2026 10:26am UTC

@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 550ffac7-1f41-4a17-a8d1-3c19b6d28598

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 3a332d95-021c-4df8-8c99-22c32e6ba175

📥 Commits

Reviewing files that changed from the base of the PR and between b809c7a and e40c7f5.

📒 Files selected for processing (7)
  • apps/web/e2e/mocks/notification.ts
  • apps/web/e2e/setup/auth.setup.ts
  • apps/web/e2e/setup/mockApiServer.ts
  • apps/web/e2e/specs/mypage/mypageMember.spec.ts
  • apps/web/e2e/specs/mypage/nicknameEdit.spec.ts
  • apps/web/e2e/specs/mypage/withdraw.spec.ts
  • apps/web/e2e/specs/notification/notificationList.spec.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

알림 fixture와 알림 목록 E2E 테스트를 추가했습니다. 회원 마이페이지에서 정보 표시, 닉네임 수정, 로그아웃, 탈퇴 동작을 검증하는 테스트도 추가했습니다. E2E 인증 상태와 모의 API 서버 설정도 변경했습니다.

Changes

알림 E2E

Layer / File(s) Summary
알림 fixture와 이동 검증
apps/web/e2e/mocks/notification.ts, apps/web/e2e/specs/notification/notificationList.spec.ts, apps/web/e2e/setup/auth.setup.ts, apps/web/e2e/setup/mockApiServer.ts
알림 종류별 fixture를 추가했습니다. 알림 클릭 시 읽음 요청의 대상 ID와 이동 경로, highlightItem 값을 확인합니다. 인증 상태에 온보딩 완료 값을 추가하고 모의 API 서버의 keepAliveTimeout을 설정했습니다.
읽음 처리와 조회 오류 재시도
apps/web/e2e/specs/notification/notificationList.spec.ts
공지 알림의 읽음 요청과 전체 읽음 요청을 확인합니다. 알림 조회 오류 안내와 재시도 후 목록 표시도 검증합니다.

회원 마이페이지 E2E

Layer / File(s) Summary
회원 정보와 닉네임 수정
apps/web/e2e/specs/mypage/mypageMember.spec.ts, apps/web/e2e/specs/mypage/nicknameEdit.spec.ts
회원 정보와 로그아웃 흐름을 검증합니다. 중복 닉네임 안내와 수정 비활성화, 사용 가능한 닉네임의 PATCH 요청 및 마이페이지 복귀를 확인합니다.
회원 탈퇴
apps/web/e2e/specs/mypage/withdraw.spec.ts
탈퇴 성공 시 로그인 페이지 이동과 토큰 쿠키 삭제를 확인합니다. USER_GUEST_CANNOT_WITHDRAW 응답 시 오류 안내와 탈퇴 페이지 유지를 검증합니다.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to e40c7

The added notification and My Page E2E coverage has no identified merge-blocking issue.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning 직접 연결된 활성 이슈 [#716]의 코딩 요구사항을 모두 충족하지 않습니다. 구현된 notificationList.spec.ts는 세 가지 이동 목적지만 검증합니다. 이슈의 TOURNAMENT_COMPLETED 그룹 결과와 ITEM_PARSING_* 위시 이동은 제외되어 있습니다. 빈 상태 재시도, 웹뷰 푸시 배너, 읽음 항목 흐림 처리는 검증… [#716]의 미충족 자동화 테스트를 추가해야 합니다. 알림의 그룹 결과·위시 이동, 빈 상태 재시도, 웹뷰 배너 또는 제외 사유를 검증해야 합니다. 닉네임의 기존 값과 탈퇴 접두사 입력에서 검사 요청이 없고 저장이 비활성화되는지 검증해야 합니다. 디바운스 후 닉네임 검사 요청이 1회인지 단언해야 합니다. 탈퇴 게스트 진입 시 마이페이지로 리다이렉트되는지 검증해야 합니다.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 알림과 마이페이지의 회원 여정 E2E 시나리오를 추가하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Out of Scope Changes check ✅ Passed 변경 파일은 [#716]의 E2E 테스트와 테스트 지원 코드에 한정됩니다. 알림 목 데이터, 알림 목록 테스트, 마이페이지·닉네임·탈퇴 테스트가 목적과 직접 연결됩니다. storageState의 온보딩 상태와 SSR mock 서버의 keepAliveTimeout 변경도 로그아웃·탈퇴 E2E 실행을 지원합니다. 별도의 무관한 제품 기능 변경은 확인되지…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 7…
Full details: Linked Issues check

Explanation

직접 연결된 활성 이슈 [#716]의 코딩 요구사항을 모두 충족하지 않습니다. 구현된 notificationList.spec.ts는 세 가지 이동 목적지만 검증합니다. 이슈의 TOURNAMENT_COMPLETED 그룹 결과와 ITEM_PARSING_* 위시 이동은 제외되어 있습니다. 빈 상태 재시도, 웹뷰 푸시 배너, 읽음 항목 흐림 처리는 검증하지 않습니다. nicknameEdit.spec.ts는 중복 및 사용 가능 닉네임만 검증합니다. 기존 닉네임, 탈퇴 접두사, 디바운스 후 검사 요청 1회 조건이 없습니다. 탈퇴 회원 흐름은 성공 및 403 실패를 검증하지만 게스트의 마이페이지 리다이렉트는 없습니다. 회원 마이페이지와 로그아웃, 전체 읽음 확인 및 오류 재시도는 구현되어 있습니다.

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@vercel
vercel Bot temporarily deployed to Preview – piki-notify September 28, 2026 10:25 Inactive
@kanghaeun
kanghaeun merged commit 87b8e8c into dev Sep 28, 2026
7 checks passed
@kanghaeun
kanghaeun deleted the test/716-notification-mypage-e2e branch September 28, 2026 10:28

This branch was successfully deployed

1 active and 1 inactive deployments
Preview – piki — 8ccaf530 Deployed Sep 28, 2026 by vercel[bot]
Preview – piki-notify — 8ccaf530 Deployed Sep 28, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: 알림·마이페이지 회원 여정 E2E 시나리오 추가

2 participants