composer update #4585

nanasess · 2020-06-23T04:41:25Z

概要(Overview・Refs Issue)

composer update "symfony/*" を適用

Updating symfony/flex (v1.4.8 => v1.8.4)
Updating symfony/asset (v3.4.35 => v3.4.42)
Updating symfony/class-loader (v3.4.35 => v3.4.42)
Updating symfony/css-selector (v3.4.35 => v3.4.42)
Updating symfony/debug (v3.4.35 => v3.4.42)
Updating symfony/polyfill-mbstring (v1.12.0 => v1.17.1)
Updating symfony/var-dumper (v3.4.35 => v3.4.42)
Updating symfony/polyfill-ctype (v1.12.0 => v1.17.1)
Updating symfony/twig-bridge (v3.4.35 => v3.4.42)
Updating symfony/polyfill-util (v1.12.0 => v1.17.1)
Updating symfony/polyfill-php56 (v1.12.0 => v1.17.1)
Updating symfony/polyfill-php70 (v1.12.0 => v1.17.1)
Updating symfony/http-foundation (v3.4.35 => v3.4.42)
Updating symfony/event-dispatcher (v3.4.35 => v3.4.42)
Updating symfony/http-kernel (v3.4.35 => v3.4.42)
Updating symfony/filesystem (v3.4.35 => v3.4.42)
Updating symfony/config (v3.4.35 => v3.4.42)
Updating symfony/twig-bundle (v3.4.35 => v3.4.42)
Updating symfony/routing (v3.4.35 => v3.4.42)
Updating symfony/web-profiler-bundle (v3.4.35 => v3.4.42)
Updating symfony/stopwatch (v3.4.35 => v3.4.42)
Updating symfony/monolog-bridge (v3.4.35 => v3.4.42)
Updating symfony/dependency-injection (v3.4.35 => v3.4.42)
Updating symfony/debug-bundle (v3.4.35 => v3.4.42)
Updating symfony/debug-pack (v1.0.7 => v1.0.8)
Updating symfony/doctrine-bridge (v3.4.35 => v3.4.42)
Updating symfony/dotenv (v3.4.35 => v3.4.42)
Updating symfony/polyfill-apcu (v1.12.0 => v1.17.1)
Updating symfony/cache (v3.4.35 => v3.4.42)
Updating symfony/expression-language (v3.4.35 => v3.4.42)
Updating symfony/inflector (v3.4.35 => v3.4.42)
Updating symfony/property-access (v3.4.35 => v3.4.42)
Updating symfony/options-resolver (v3.4.35 => v3.4.42)
Updating symfony/intl (v3.4.35 => v3.4.42)
Updating symfony/polyfill-intl-icu (v1.12.0 => v1.17.1)
Updating symfony/form (v3.4.35 => v3.4.42)
Updating symfony/finder (v3.4.35 => v3.4.42)
Updating symfony/framework-bundle (v3.4.35 => v3.4.42)
Updating symfony/console (v3.4.35 => v3.4.42)
Updating symfony/maker-bundle (v1.14.3 => v1.19.0)
Updating symfony/orm-pack (v1.0.7 => v1.0.8)
Updating symfony/polyfill-iconv (v1.12.0 => v1.17.1)
Updating symfony/polyfill-php72 (v1.12.0 => v1.17.0)
Updating symfony/polyfill-intl-idn (v1.12.0 => v1.17.1)
Updating symfony/proxy-manager-bridge (v3.4.35 => v3.4.42)
Updating symfony/security (v3.4.35 => v3.4.42)
Updating symfony/security-bundle (v3.4.35 => v3.4.42)
Updating symfony/serializer (v3.4.35 => v3.4.42)
Updating symfony/templating (v3.4.35 => v3.4.42)
Updating symfony/translation (v3.4.35 => v3.4.42)
Updating symfony/validator (v3.4.35 => v3.4.42)
Updating symfony/process (v3.4.35 => v3.4.42)
Updating symfony/web-server-bundle (v3.4.35 => v3.4.42)
Updating symfony/workflow (v3.4.35 => v3.4.42)
Updating symfony/yaml (v3.4.35 => v3.4.42)
Updating symfony/dom-crawler (v3.4.35 => v3.4.42)
Updating symfony/browser-kit (v3.4.35 => v3.4.42)
Updating symfony/phpunit-bridge (v3.4.35 => v3.4.42)

実装に関する補足(Appendix)

SameSite cookie support #4519 にて移植したテストケースの random_session_id のテストがエラーになる問題が解決している模様
- see [HttpFoundation] workaround PHP bug in the session module symfony/symfony#36490

テスト（Test)

TODO

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

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

レビュワー確認項目

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

okazy · 2020-06-24T07:18:54Z

ライブラリのバージョンアップはテストで品質を担保したいのでしっかりめにテスト結果を見ていければと思います。

以下のテストが落ちているので動作の確認が必要です。

19) Eccube\Tests\Web\Admin\Product\CategoryControllerTest::testMoveSortNoAndShow
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'子2-2'
+'孫2'

/home/runner/work/ec-cube/ec-cube/tests/Eccube/Tests/EccubeTestCase.php:108
/home/runner/work/ec-cube/ec-cube/tests/Eccube/Tests/Web/Admin/Product/CategoryContorllerTest.php:370

以下のエラーは再実行で通りました。

There was 1 failure:

---------
1) EA04OrderCest: Ea0405-uc06-t01_ー括受注のステータス変更
 Test  codeception/acceptance/EA04OrderCest.php:order_ー括受注のステータス変更
 Step  See "検索結果：8件が該当しました","#search_form #search_total_count"
 Fail  Failed asserting that any element by '#search_form #search_total_count' on page /admin/order
Elements: 
+ <span> 検索結果：7件が該当しました
contains text '検索結果：8件が該当しました'

Scenario Steps:

 29. $I->see("検索結果：8件が該当しました","#search_form #search_tot...") at codeception/acceptance/EA04OrderCest.php:392
 28. // 発送済みに変更した件数を確認します。新規受付: 2件 +  発送済み: 6件
 27. $I->click("#search_form #search_submit") at codeception/_support/Page/Admin/OrderManagePage.php:226
 26. $I->checkOption({"id":"admin_search_order_status_5"}) at codeception/_support/Page/Admin/OrderManagePage.php:225
 25. $I->see("受注一覧受注管理",".c-pageTitle") at codeception/_support/Page/Admin/AbstractAdminPageStyleGuide.php:27
 24. $I->amOnPage("/admin/order") at codeception/_support/Page/Admin/AbstractAdminPage.php:32

There was 1 error:

---------
1) EF05MypageCest: Ef0503-uc01-t01 mypage お届け先編集削除
 Test  codeception/acceptance/EF05MypageCest.php:mypage_お届け先編集削除
                                                                                                                                                                                                                                                                                                                                        
  [Facebook\WebDriver\Exception\StaleElementReferenceException] stale element reference: element is not attached to the page document
  (Session info: chrome=83.0.4103.97)
  (Driver info: chromedriver=83.0.4103.39 (ccbf011cb2d2b19b506d844400483861342c20cd-refs/branch-heads/4103@{#416}),platform=Linux 5.3.0-1028-azure x86_64)  
                                                                                                                                                                                                                                                                                                                                        

Scenario Steps:

 23. $I->see("お届け先は登録されていません。","#page_mypage_delivery >...") at codeception/acceptance/EF05MypageCest.php:267
 22. $I->acceptPopup() at codeception/_support/Page/Front/CustomerAddressListPage.php:48
 21. $I->click("div.ec-addressList div:nth-child(1) a.e...") at codeception/_support/Page/Front/CustomerAddressListPage.php:47
 20. $I->see("マイページ/お届け先一覧","div.ec-pageHeader h1") at codeception/_support/Page/Front/CustomerAddressListPage.php:26
 19. $I->see("大阪市西区","div.ec-addressList div:nth-child(...") at codeception/acceptance/EF05MypageCest.php:261
 18. $I->click("div.ec-RegisterRole__actions button") at codeception/_support/Page/Front/CustomerAddressEditPage.php:96

再実行します。

nanasess · 2020-06-24T08:07:51Z

以下のテストが落ちているので動作の確認が必要です。

composer update "symfony/*" を実行した場合はテスト成功するので、 Doctrine のバグ or 仕様変更の可能性

okazy · 2020-06-24T08:45:54Z

以下のケース以外のテストが通ることを確認しました。
Eccube\Tests\Web\Admin\Product\CategoryControllerTest::testMoveSortNoAndShow

nanasess · 2020-06-25T07:52:34Z

Doctrine のバグ or 仕様変更の可能性

この PR が原因の模様
doctrine/DoctrineBundle#1099

nanasess · 2020-06-25T08:11:25Z

doctrine/doctrine-bundle は更新しないよう、 composer update "symfony/*" を適用しました

nanasess · 2020-06-25T08:20:45Z

@okazy テストが落ちる件は #4592 に分割しました

okazy

ありがとうございます！
Eccube\Tests\Web\Admin\Product\CategoryControllerTest::testMoveSortNoAndShow
が通っていることを確認しました。問題ないと思います。

okazy · 2020-06-26T00:22:22Z

ありがとうございました。テストを通すために優先して取り込ませていただきました。

Apply composer update

7af426d

nanasess mentioned this pull request Jun 23, 2020

Fix testcase to session_regenerate_id() #4586

Merged

11 tasks

okazy added this to the 4.0.x milestone Jun 23, 2020

okazy added the dependencies Pull requests that update a dependency file label Jun 23, 2020

okazy mentioned this pull request Jun 24, 2020

PHP7.4.8, 7.3.20 もバグでエラーになるためテストをスキップ #4589

Closed

11 tasks

Apply composer update "symfony/*"

6fe761c

nanasess mentioned this pull request Jun 25, 2020

doctrine/doctrine-bundle 1.12.3 support #4592

Open

okazy approved these changes Jun 26, 2020

View reviewed changes

okazy added the Status: ready-for-merge label Jun 26, 2020

okazy merged commit a7ff5a5 into EC-CUBE:4.0 Jun 26, 2020

okazy mentioned this pull request Jun 30, 2020

【週刊】取り込んだプルリクエスト #4595

Open

okazy modified the milestones: 4.0.x, 4.0.4 Aug 21, 2020

okazy mentioned this pull request Aug 24, 2020

easycorp/easy-log-handler を必須ライブラリに変更 #4669

Merged

11 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

composer update #4585

composer update #4585

nanasess commented Jun 23, 2020 •

edited

okazy commented Jun 24, 2020 •

edited

nanasess commented Jun 24, 2020

okazy commented Jun 24, 2020

nanasess commented Jun 25, 2020

nanasess commented Jun 25, 2020

nanasess commented Jun 25, 2020

okazy left a comment •

edited

okazy commented Jun 26, 2020

composer update #4585

composer update #4585

Conversation

nanasess commented Jun 23, 2020 • edited

概要(Overview・Refs Issue)

実装に関する補足(Appendix)

テスト（Test)

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

レビュワー確認項目

okazy commented Jun 24, 2020 • edited

nanasess commented Jun 24, 2020

okazy commented Jun 24, 2020

nanasess commented Jun 25, 2020

nanasess commented Jun 25, 2020

nanasess commented Jun 25, 2020

okazy left a comment • edited

Choose a reason for hiding this comment

okazy commented Jun 26, 2020

nanasess commented Jun 23, 2020 •

edited

okazy commented Jun 24, 2020 •

edited

okazy left a comment •

edited