注文画面 配送方法を選択した際の不具合修正

[tatsuishi]

概要(Overview・Refs Issue)

複数の配送方法を設定している場合で、「注文手続き画面」で配送方法をプルダウン選択した場合、
ボタンを押下できず、また送料・手数料が更新されない不具合がありました。

支払方法フォームのエラーが出ないように、選択していた支払方法が選択肢からなくなった場合は先頭の支払い方法をセットし、変更した旨のメッセージを出すように修正しました。

再現方法

[配送方法１]
名称：宅配便
販売種別：販売種別A
支払方法：代金引換(手数料200円)・銀行振込
送料：1000円
[配送方法２]
名称：メール便
販売種別：販売種別A
支払方法：銀行振込
送料：0円

配送方法を上記のように設定し、
「販売種別A」を設定した商品をカートに入れ、「注文手続き画面」に進みます。

支払方法は初期状態で「代金引換」が選択されています。

その状態で配送方法を「メール便」に変更します。

→ 「メール便」では「代金引換」が選択肢にないので、Paymentのフォームがエラーになり、添付画像の状態が発生します。

配送方法選択(切り替え)前

配送方法選択(切り替え)後

テスト（Test)

修正反映後の挙動

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

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

レビュワー確認項目

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

[codecov]

Codecov Report

Attention: Patch coverage is 44.44444% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 82.33%. Comparing base (e44c828) to head (9d31101).
Report is 13 commits behind head on 4.3.

Files	Patch %	Lines
...chaseFlow/Processor/PaymentChangePostValidator.php	0.00%	10 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##              4.3    #6142      +/-   ##
==========================================
- Coverage   82.35%   82.33%   -0.03%     
==========================================
  Files         480      481       +1     
  Lines       26426    26444      +18     
==========================================
+ Hits        21764    21772       +8     
- Misses       4662     4672      +10     

Flag	Coverage Δ
E2E	82.33% <44.44%> (-0.03%)	⬇️
Unit	82.33% <44.44%> (-0.03%)	⬇️

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.

[dotani1111]

@tatsuishi
PRありがとうございます！
動作確認してマージしたいと思います 👍

[ji-eunsoo]

@tatsuishi
Unitテストで落ちているのですが、一度ご確認いただけないでしょうか？

[dotani1111]

Amazonpayなどの決済プラグインの場合について、動作確認が必要ですね。

[dotani1111]

Amazonpayは問題なさそうでした。
GMOの決済プラグインでも動作検証します。

[dotani1111]

@tatsuishi
こちらのテストが落ちてしまっているようです。
https://github.com/EC-CUBE/ec-cube/actions/runs/8685427600/job/24227991479?pr=6142

testPaymentWithErrorのassertに失敗しているようですが、こちら修正可能でしょうか？