ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

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

Open nobuhiko opened this issue 2 years ago • 2 comments

概要(Overview・Refs Issue)

#5569 の修正

方針(Policy)

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

実装に関する補足(Appendix)

テスト(Test)

相談(Discussion)

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

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

レビュワー確認項目

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

nobuhiko avatar Aug 26 '22 15:08 nobuhiko

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%) :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.

codecov-commenter avatar Aug 27 '22 04:08 codecov-commenter

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

nobuhiko avatar Aug 27 '22 22:08 nobuhiko

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

shinya avatar May 19 '23 07:05 shinya

コンフリクト解消済み

shinya avatar May 24 '23 07:05 shinya

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

xuelian311 avatar Jul 07 '23 00:07 xuelian311