テスト用Fixturesの生成でユニーク項目がユニークになるようにする

[kazuyoshikakihara]

概要(Overview・Refs Issue)

#5613
[4.2] ランダムで落ちるテストを修正したい
関連です。

Fixturesの生成時に、Customerのメールアドレスなど、本来ユニークな値が生成されなければならない項目があります。
これまではここがFakerのword生成頼りになっていたのですが、生成候補に限りがあるため、稀に重複する値が生成されておりました。

重複する値が生成されてしまった場合には値がユニークになるまで再生成するようにコードを修正しました。

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

NonMember のメールアドレスまでユニークにする必要性はなかったかもしれません。
が、NonMemberのメールアドレスの重複で問題が起きるようなら、それはそれで別のテストを作るべき項目と考えられますので、とりあえずユニークになるようにしました。

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5679 (9dc1957) into 4.2 (16561c7) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                4.2    #5679   +/-   ##
=========================================
  Coverage     78.82%   78.82%           
  Complexity     6260     6260           
=========================================
  Files           469      469           
  Lines         21010    21009    -1     
=========================================
  Hits          16561    16561           
+ Misses         4449     4448    -1     

Flag	Coverage Δ
E2E	64.77% <ø> (+<0.01%)	⬆️
Unit	77.53% <ø> (ø)

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

Impacted Files	Coverage Δ
.../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.

[chihiro-adachi]

@kazuyoshikakihara
原因特定＆修正ありがとうございます！
めっちゃ助かります。