不要かつ困惑的なFormTypeのCouponTypeのConstraintsの追加をやめる

[kazuyoshikakihara]

概要(Overview・Refs Issue)

CouponType の discount_rate のConstraintsについて。

最初にFormをbuildする際には

 new Assert\Range([
     'min' => 1,
     'max' => 100,
]),

と宣言されていますが、その後、POST_SUBMITのEventListenerの中で改めて

new Assert\Range([
    'min' => 0,
    'max' => 100,
]),

とmin値を変えてConstraints（validate）が追加宣言されています。

どちらのminが正しいのかプログラムからは判断がつきませんが、実質的に'min' => 1のほうが有効になっているので、POST_SUBMIT時の追加宣言は不要かつ困惑的と考え、削除します。

テスト（Test)

CouponType.php に対する自動テスト用のテストコードを新たに作成しました。

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

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

レビュワー確認項目

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

[chihiro-adachi]

@kazuyoshikakihara
ありがとうございます！
修正確認しました。