ec-cube
ec-cube copied to clipboard
ref #5569 メール設定のバリデートをする
概要(Overview・Refs Issue)
#5569 の修正
方針(Policy)
メールを送信するしかテストの方法がないのでdummyメールを送信する
実装に関する補足(Appendix)
テスト(Test)
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
- [ ] 既存機能の仕様変更はありません
- [ ] フックポイントの呼び出しタイミングの変更はありません
- [ ] フックポイントのパラメータの削除・データ型の変更はありません
- [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
- [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
- [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません
レビュワー確認項目
- [ ] 動作確認
- [ ] コードレビュー
- [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
- [ ] 互換性が保持されているか
- [ ] セキュリティ上の問題がないか
- [ ] 権限を超えた操作が可能にならないか
- [ ] 不要なファイルアップロードがないか
- [ ] 外部へ公開されるファイルや機能の追加ではないか
- [ ] テンプレートでのエスケープ漏れがないか
Codecov Report
Merging #5648 (35598cd) into 4.2 (16561c7) will increase coverage by
0.17%
. The diff coverage is93.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%) |
:arrow_down: |
Unit | 77.64% <97.72%> (+0.11%) |
:arrow_up: |
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%) |
:arrow_up: |
src/Eccube/Form/Type/Install/Step3Type.php | 98.70% <91.66%> (-1.30%) |
:arrow_down: |
src/Eccube/Util/StringUtil.php | 96.80% <96.96%> (+0.06%) |
:arrow_up: |
.../Service/PurchaseFlow/Processor/PointProcessor.php | 95.83% <0.00%> (+1.95%) |
:arrow_up: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
落ちてるテストは時間がかかりすぎた結果だと思うので問題ないと思います
@nobuhiko PRで作成いただきありがとうございます。 コンフリクトが発生しておりますので、解消をお願いできませんでしょうか。 お手数おかけしますがよろしくおねがいします。 (合わせてメインブランチの再取り込みもお願いします)
コンフリクト解消済み
インストール時のメール設定の有効性確認はユーザビリティが低いと判断し、本件はクローズします。