Skip to content

test(e2e): cover default tab of user edit page by AtCoder verification status - #4187

Merged
KATO-Hiro merged 2 commits into
stagingfrom
#4186
Oct 18, 2026
Merged

KATO-Hiro merged 2 commits into
stagingfrom
#4186

Conversation

@KATO-Hiro

@KATO-Hiro KATO-Hiro commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

close #4186

Summary by CodeRabbit

  • テスト
    • 未認証ユーザーが編集画面から適切にリダイレクトされることを確認するテストを追加しました。
    • ユーザーの認証状態に応じて、編集画面で最初に選択されるタブが異なることを確認するテストを追加しました。
    • AtCoder認証済みユーザーが基本情報タブを開いた際、ユーザー名が表示されることを確認するテストを追加しました。

KATO-Hiro and others added 2 commits October 18, 2026 09:54
…n status

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: AtCoder-NoviSteps/AtCoderNoviSteps/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: cd512d08-09c9-4484-a2dd-8654cc9a1a53
📥 Commits

Reviewing files that changed from the base of the PR and between 395a542 and e0ba1c6.

📒 Files selected for processing: 2
  • e2e/helpers/auth.ts
  • e2e/user_edit.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

未検証ユーザー向けのログインヘルパーを追加しました。ユーザー編集画面のE2Eテストで、Aliceとguestの初期タブ、およびguestのユーザー名を検証します。

Changes: ユーザー編集タブのE2Eテスト

Layer / File(s) Summary
ログイン対象の追加
e2e/helpers/auth.ts
loginAsUnverifiedUserを追加し、Aliceで共通のログイン処理を呼び出します。loginAsUserの説明をAtCoder認証済みユーザー向けに変更します。
編集画面のタブ検証
e2e/user_edit.spec.ts
タブ取得と編集画面の表示処理を共通化します。Aliceとguestの初期タブを検証し、guestは「基本情報」タブを開いた後にユーザー名を確認します。

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: river0525

Merge Risk: ⚪ Minimal · up to e0ba1

The new tests match the expected default tabs for Alice and guest. No merge-blocking issue was identified.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Docstring Coverage Warning Docstring coverage is 60.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed Passed checks: 4 passed
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed タイトルは、AtCoder認証状態に応じたユーザー編集ページの初期タブをE2Eテストで検証する変更を正確に要約しています。
Linked Issues check Passed Issue #4186 は user/edit のタブに関する E2E テスト修正を求めています。PR は未認証ユーザーのリダイレクトを維持し、未検証ユーザー Alice では「基本情報」タブ、AtCoder 認証済みユーザー guest では「AtCoder IDを設定」タブが初期選択されることを aria-selected で検証します。guest の基本情報タブ選択後のユーザー名検証も維…
Out of Scope Changes check Passed 変更は e2e/helpers/auth.ts のログインヘルパー追加と e2e/user_edit.spec.ts の user/edit E2E テストに限定されています。Alice と guest の認証状態を分ける変更、タブ取得とページ安定化の共通化、関連するアサーション追加は issue #4186 のテスト修正を直接支援します。無関係な変更は確認できません。
Full details: Linked Issues check

Explanation

Issue #4186 は user/edit のタブに関する E2E テスト修正を求めています。PR は未認証ユーザーのリダイレクトを維持し、未検証ユーザー Alice では「基本情報」タブ、AtCoder 認証済みユーザー guest では「AtCoder IDを設定」タブが初期選択されることを aria-selected で検証します。guest の基本情報タブ選択後のユーザー名検証も維持しています。確認できるコーディング要件の未達はありません。

  • Fix all pre-merge checks with AI
✨ Finishing Touches: 💡 1
📝 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.

@KATO-Hiro KATO-Hiro left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@KATO-Hiro
KATO-Hiro merged commit ce923f1 into staging Oct 18, 2026
3 checks passed
@KATO-Hiro
KATO-Hiro deleted the #4186 branch October 18, 2026 10:00

This branch was successfully deployed

1 active deployment
preview — e0ba1c6f Deployed Oct 18, 2026 by KATO-Hiro via preview #9009
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.

test: fix e2e in user/edit tab

1 participant