[symfony4.4] 4.0ブランチの変更を取り込み

[okazy]

概要(Overview・Refs Issue)

experimental/symfony4.4 ブランチへ 4.0 ブランチの変更を取り込みました。

マージ後に composer update しています。

以下のテストは #4722 で修正しています。

1. EA03ProductCest: Ea0305-uc01-t01(& uc01-t02/uc02-t01/uc04-t01) カテゴリ登録/編集/削除
2. EA03ProductCest: Ea0307-uc01-t01(& uc01-t02) カテゴリcsv登録
3. EA06ContentsManagementCest: Ea0603-uc01-t01(& uc01-t02/uc01-t03/uc01-t04/uc01-t05) ページ管理

方針(Policy)

4.0ブランチを取り込み、テストを通す

実装に関する補足(Appendix)

テスト（Test)

個人リポジトリで全てのケースで一度以上通ることを確認

相談（Discussion）

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

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

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

以下のテストが落ちる問題が発生中

There was 1 failure:

1) Eccube\Tests\Web\ProductControllerTest::testProductFavoriteAddThroughLogin
Failed asserting that false is true.

/home/runner/work/ec-cube/ec-cube/tests/Eccube/Tests/Web/ProductControllerTest.php:247

[okazy]

以下のテストが高確率で落ちる。

1. EA10PluginCest: Test_install_enable_update_disable_remove_local
2. EA10PluginCest: Test_enhance_plugin_entity

[okazy]

#4722 #4727 でプラグイン関連のエラーは安定して通るようになりました。

[okazy]

#4723 (comment)

Symfony4.4.15から以下のテストが落ちるようになった。

There was 1 failure:

1) Eccube\Tests\Web\ProductControllerTest::testProductFavoriteAddThroughLogin
Failed asserting that false is true.

/home/runner/work/ec-cube/ec-cube/tests/Eccube/Tests/Web/ProductControllerTest.php:247

手動では再現せず、テストの時だけ落ちる

大まかな流れは以下

1. product_detailで未ログイン状態でお気に入りボタンを押下して、mypage_loginへリダイレクト
2. mypage_loginでログイン情報を入力してログインボタンを押下
3. mypage_loginでログイン処理が行われ、product_add_favoriteへリダイレクト
4. product_add_favoriteでお気に入り処理が行われ、product_detailへリダイレクト

(3) でログイン成功しないと (4) へ遷移しないはずなのに、 (4) ではログインできていない状態となっておりもう一度mypage_loginへリダイレクトされてしまう。

Security あるいは session あたりが怪しいかと思いましたが、Symfony4.4.15の変更内容にはそれらしいのはありませんでした。
https://symfony.com/blog/symfony-4-4-15-released

助けてください・・・

[nanasess]

@okazy

Symfony4.4.15から以下のテストが落ちるようになった。

安直ですが、 submit の後に wait を入れてみては？

// ログインをクリック
$client->submit($loginForm);

[okazy]

MOCKSESSIDの仕様で、アクセス発生時の時刻がexpiresにセットされるようです。

expiresと同時刻でcookieが期限切れになるよう仕様変更をされたため、常にセッションが破棄される状態となっているようです。
symfony/symfony@9d187c0
他のテストが落ちていない理由はまだわかりません。

[okazy]

#4723 (comment)
こちらの問題、 #4730 で解決します。

[okazy]

全部通った 🎉
ご助力ありがとうございました 🙏