ref #5569 メール設定のバリデートをする

[nobuhiko]

概要(Overview・Refs Issue)

#5569 の修正

方針(Policy)

メールを送信するしかテストの方法がないのでdummyメールを送信する

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5648 (35598cd) into 4.2 (16561c7) will increase coverage by 0.17%.
The diff coverage is 93.47%.

@@             Coverage Diff              @@
##                4.2    #5648      +/-   ##
============================================
+ Coverage     78.82%   78.99%   +0.17%     
- Complexity     6260     6261       +1     
============================================
  Files           469      469              
  Lines         21010    21019       +9     
============================================
+ Hits          16561    16604      +43     
+ Misses         4449     4415      -34     

Flag	Coverage Δ
E2E	64.74% <0.00%> (-0.03%)	⬇️
Unit	77.64% <97.72%> (+0.11%)	⬆️

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

Impacted Files	Coverage Δ
...rc/Eccube/Controller/Install/InstallController.php	16.09% <0.00%> (+1.18%)	⬆️
src/Eccube/Form/Type/Install/Step3Type.php	98.70% <91.66%> (-1.30%)	⬇️
src/Eccube/Util/StringUtil.php	96.80% <96.96%> (+0.06%)	⬆️
.../Service/PurchaseFlow/Processor/PointProcessor.php	95.83% <0.00%> (+1.95%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[nobuhiko]

落ちてるテストは時間がかかりすぎた結果だと思うので問題ないと思います

[shinya]

@nobuhiko
PRで作成いただきありがとうございます。
コンフリクトが発生しておりますので、解消をお願いできませんでしょうか。
お手数おかけしますがよろしくおねがいします。
（合わせてメインブランチの再取り込みもお願いします）

[shinya]

コンフリクト解消済み

[xuelian311]

インストール時のメール設定の有効性確認はユーザビリティが低いと判断し、本件はクローズします。