テスト対象のフォームごとに確認するように変更

[kiy0taka]

概要(Overview・Refs Issue)

デフォルト値を使ったテストtestInValidDataが通るので、個別にフォームの値を変更してもすべて $this->form->isValid() は false になる。

ec-cube/tests/Eccube/Tests/Form/Type/Admin/OrderTypeTest.php

Lines 81 to 85 in 36cbdeb

	public function testInValidData()
	{
	$this->form->submit($this->formData);
	$this->assertFalse($this->form->isValid());
	}

方針(Policy)

フォームごとに確認するように修正しました。

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5469 (064e59c) into 4.1 (25e7b1c) will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                4.1    #5469      +/-   ##
============================================
+ Coverage     68.53%   68.59%   +0.05%     
  Complexity     6162     6162              
============================================
  Files           463      463              
  Lines         25305    25305              
============================================
+ Hits          17343    17358      +15     
+ Misses         7962     7947      -15     

Flag	Coverage Δ
E2E	57.94% <ø> (ø)
Unit	76.15% <ø> (+0.11%)	⬆️

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

Impacted Files	Coverage Δ
src/Eccube/Controller/Admin/AdminController.php	78.96% <0.00%> (+5.53%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 25e7b1c...064e59c. Read the comment docs.

[chihiro-adachi]

@kiy0taka
ありがとうございます。マージしました。