Skip to content

composer update#4585

Merged
okazy merged 2 commits intoEC-CUBE:4.0from
nanasess:composer-update
Jun 26, 2020
Merged

composer update#4585
okazy merged 2 commits intoEC-CUBE:4.0from
nanasess:composer-update

Conversation

@nanasess
Copy link
Copy Markdown
Contributor

@nanasess nanasess commented Jun 23, 2020 •
edited
Loading

概要(Overview・Refs Issue)

composer update "symfony/*" を適用

Details
  • 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)

テスト（Test)

TODO

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

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

レビュワー確認項目

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

@nanasess nanasess mentioned this pull request Jun 23, 2020
Merged
11 tasks
@okazy okazy added this to the 4.0.x milestone Jun 23, 2020
@okazy okazy added the dependencies Pull requests that update a dependency file label Jun 23, 2020
@okazy okazy mentioned this pull request Jun 24, 2020
Closed
11 tasks
@okazy
Copy link
Copy Markdown
Contributor

okazy commented Jun 24, 2020 •
edited
Loading

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

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

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

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

Details 
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
Copy link
Copy Markdown
Contributor Author

nanasess commented Jun 24, 2020

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

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

@okazy
Copy link
Copy Markdown
Contributor

okazy commented Jun 24, 2020

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

@nanasess
Copy link
Copy Markdown
Contributor Author

nanasess commented Jun 25, 2020

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

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

@nanasess
Copy link
Copy Markdown
Contributor Author

nanasess commented Jun 25, 2020

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

@nanasess nanasess mentioned this pull request Jun 25, 2020
Open
@nanasess
Copy link
Copy Markdown
Contributor Author

nanasess commented Jun 25, 2020

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

okazy
okazy approved these changes Jun 26, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@okazy okazy left a comment •
edited
Loading

Choose a reason for hiding this comment

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

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

@okazy okazy merged commit a7ff5a5 into EC-CUBE:4.0 Jun 26, 2020
@okazy
Copy link
Copy Markdown
Contributor

okazy commented Jun 26, 2020

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

@okazy okazy mentioned this pull request Jun 30, 2020
Open
@okazy okazy modified the milestones: 4.0.x, 4.0.4 Aug 21, 2020
@okazy okazy mentioned this pull request Aug 24, 2020
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@okazy okazy okazy approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

dependencies Pull requests that update a dependency file Status: ready-for-merge

Projects

None yet

Milestone

4.0.4

Development

Successfully merging this pull request may close these issues.

2 participants

@nanasess @okazy