Skip to content

feat: VRTをPlaywright toHaveScreenshot()で実装 (#36)#42

Open
topi-log wants to merge 6 commits into
mainfrom
feat/#36_vrt
Open

feat: VRTをPlaywright toHaveScreenshot()で実装 (#36)#42
topi-log wants to merge 6 commits into
mainfrom
feat/#36_vrt

Conversation

@topi-log

Copy link
Copy Markdown
Owner

Summary

  • e2e/vrt.spec.ts を新規追加。Playwright の toHaveScreenshot() でスクリーンショット比較テストを実装
  • 対象ページ: ホーム / プライバシーポリシー / 利用規約 / 記録一覧(記録あり・なし)/ エディタ
  • ベースラインスナップショット6枚をリポジトリにコミット
  • playwright.config.ts--no-sandbox(Docker対応)・animations: 'disabled'maxDiffPixelRatio: 0.02 を設定
  • package.jsontest:vrt(比較実行)と test:vrt:update(ベースライン更新)スクリプトを追加
  • record.spec.ts からCookie認証移行後に不要になった setAuthStorage(localStorage)を削除

Test plan

  • yarn test:vrt で全6件パスすること
  • UIを意図的に変更後に yarn test:vrt を実行するとテストが失敗すること
  • yarn test:vrt:update でスナップショットを更新後に再度パスすること

🤖 Generated with Claude Code

- e2e/vrt.spec.tsを新規追加(ホーム・プライバシーポリシー・利用規約・記録一覧・エディタ)
- ベースラインスナップショット6枚をコミット
- playwright.config.tsにno-sandbox・animations無効・maxDiffPixelRatio 2%を設定
- package.jsonにtest:vrt・test:vrt:updateスクリプトを追加
- record.spec.tsからCookie認証移行後に不要になったsetAuthStorage(localStorage)を削除

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 25, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
leaf-record Ready Ready Preview, Comment Apr 29, 2026 5:36am

- playwright.config.ts: `next start`(port 8001)に変更しStrictModeによるflakiness解消
- mobile(Pixel 5)プロジェクトを追加
- APIルートをglobパターン(`**/api/v1/...`)に統一
- モックデータをAPIレスポンス形式(nameフィールド)に修正
- waitForLoadStateをexplicitなlocator waitに置き換え
- home.spec.ts: モバイルでログインボタンがメニュー内のためskip追加
- スナップショット(chromium/mobile)を更新・追加

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@topi-log

Copy link
Copy Markdown
Owner Author

うーん、また考える

- `test:e2e` スクリプトを vrt.spec.ts を除外するよう変更
- CI に独立した `vrt` ジョブを追加(失敗時は差分をアーティファクトとしてアップロード)
- `update-vrt.yml` を追加: CI 環境(Ubuntu)でスナップショットを更新し、ブランチにコミットバックする `workflow_dispatch` ワークフロー

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
プッシュ時に --update-snapshots でスナップショットを生成し、差分があれば
[skip ci] コミットとしてブランチに自動プッシュ。
ローカル環境とCI環境のフォントレンダリング差異を解消する。
update-vrt.yml ワークフローは不要になったため削除。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
pull_request イベントへの permissions: contents: write 付与はワークフロー
全体がスキップされる原因となるため削除。
代わりに push トリガー専用の vrt-update.yml を追加し、push イベントの
書き込み権限でスナップショットをコミットバックする方式に分離。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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