Skip to content

Conversation

@nanasess
Copy link
Contributor

@nanasess nanasess commented Apr 15, 2025

概要(Overview・Refs Issue)

Chrome 129.0.6662.0 の何らかの改修が原因で、E2Eテストが大量に失敗するようになったための対策
Fixes #6344

方針(Policy)

主に以下のようなケースでテストが通らなくなった

  • 画面遷移直後に selector を参照すると、Facebook\WebDriver\Exception\StaleElementReferenceException が発生する
  • 画面遷移直後に selector を参照すると、画面遷移が完了しない
  • 画面遷移直後は selector の参照に失敗する(<selector name> was not found. となる)
  • 入力中に画面遷移してしまう
  • テストデータが残ったまま次のテストを実行してしまう

対策として、主に以下の対応をした

  • 画面遷移の前後に wait を入れる
  • 画面遷移の失敗を検知した場合はリトライする
  • 並び替えのテストを分離する

実装に関する補足(Appendix)

その他、 php-cs-fixer による修正を含みます

テスト(Test)

E2Eテストが通ることを確認

相談(Discussion)

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更はありません
  • フックポイントの呼び出しタイミングの変更はありません
  • フックポイントのパラメータの削除・データ型の変更はありません
  • twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

@codecov
Copy link

codecov bot commented Apr 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.82%. Comparing base (801f670) to head (6595e91).
Report is 9 commits behind head on 4.3.

Additional details and impacted files
@@            Coverage Diff             @@
##              4.3    #6353      +/-   ##
==========================================
- Coverage   82.83%   82.82%   -0.01%     
==========================================
  Files         480      480              
  Lines       27093    27093              
==========================================
- Hits        22442    22441       -1     
- Misses       4651     4652       +1     
Flag Coverage Δ
E2E 82.82% <ø> (-0.01%) ⬇️
Unit 82.82% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nanasess nanasess force-pushed the chrome-129-fix-problem branch from 8c71343 to 82a1e4e Compare April 15, 2025 09:11
refactor(tests):#	modified:   codeception/acceptance/EF09ThrottlingCest.php
 remove unnecessary wait statements in EA06# Changes not staged for commit:
Contents#	modified:   codeception/_support/AcceptanceTester.php
ManagementCest

This change eliminates redundant `wait` calls that were previously#	.php-version
 added before `amOnPage()` to#	CustomErrorPage42_20221119.tar.gz
 ensure proper page transitions. The code#	app/config/eccube/packages/purchaseflow.yaml.orig
 is now cleaner and relies on more robust mechanisms for handling page transitions.
@nanasess nanasess force-pushed the chrome-129-fix-problem branch from 35741a1 to 6595e91 Compare April 16, 2025 00:14
@dotani1111 dotani1111 merged commit 0222fee into EC-CUBE:4.3 Apr 16, 2025
125 of 130 checks passed
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.

管理画面のE2Eが失敗する

3 participants