Formのテストを実装しました。

[mkmmr]

概要（このPRの対応範囲）

• ToDoFormとToDoUpdateFormのテストを実装しました。

やったこと

• テスト機能の実装
  • ToDoFormTest クラス
    • 正しい値を入力した時にバリデーションエラーとならないこと
    • nullを入力した時にバリデーションエラーとなること
    • 空文字を入力した時にバリデーションエラーとなること
    • limitDateに有効な型以外を入力した時にバリデーションエラーとなること
  • ToDoUpdateForm クラス
    • 正しい値を入力した時にバリデーションエラーとならないこと
    • nullを入力した時にバリデーションエラーとならないこと
    • 空文字を入力した時にバリデーションエラーとならないこと
    • limitDateに有効な型以外を入力した時にバリデーションエラーとなること

[mkmmr]

Formに関するテストが3点失敗しております。
いずれもバリデーションメッセージの言語に関するものです。

IntelliJでは日本語で問題なくテストできたのですが、Github Actionsではバリデーションメッセージが英語になるらしく、それでテストが失敗しているようです。
ValidationMessages_ja.propertiesを作成し、テストの時だけを読み込むことができないかとも考えたのですが、調べてもよくわかりませんでした。

何かいいやり方がありましたらご教示いただけますと幸いです。
そもそもConstraintViolationではなくBindingResultを使ったらよかったのでしょうか……。

1. build/classes/java/test/com/raisetech/todo/form/ToDoFormTest.class:80 | java.lang.AssertionError: Expecting ArrayList: [("task", "must not be blank"), ("task", "size must be between 1 and 256")] to contain only: [("task", "1 から 256 の間のサイズにしてください"), ("task", "空白は許可されていません")] element(s) not found: [("task", "1 から 256 の間のサイズにしてください"), ("task", "空白は許可されていません")] and element(s) not expected: [("task", "must not be blank"), ("task", "size must be between 1 and 256")]

2. build/classes/java/test/com/raisetech/todo/form/ToDoFormTest.class:57 | java.lang.AssertionError: Expecting ArrayList: [("limitDate", "must not be blank"), ("task", "must not be blank")] to contain only: [("limitDate", "空白は許可されていません"), ("task", "空白は許可されていません")] element(s) not found: [("limitDate", "空白は許可されていません"), ("task", "空白は許可されていません")] and element(s) not expected: [("limitDate", "must not be blank"), ("task", "must not be blank")]

3. build/classes/java/test/com/raisetech/todo/form/ToDoUpdateFormTest.class:63 | java.lang.AssertionError: Expecting ArrayList: [("task", "size must be between 1 and [25](https://github.com/mkmmr/todo/runs/8074180953?check_suite_focus=true#step:6:27)6")] to contain only: [("task", "1 から 256 の間のサイズにしてください")] element(s) not found: [("task", "1 から 256 の間のサイズにしてください")] and element(s) not expected: [("task", "size must be between 1 and 256")]

[yoshi-koyama]

このあたり必要でしょうか？
なくても動くと思いました。

[mkmmr]

なくても動きました。
不要なアノテーションを削除しました。 1dad501

[yoshi-koyama]

こちらもinjectするモックがないので不要かと思います。

[mkmmr]

ToDoForm.java内にgetLimitDateメソッドがあって、下記2点をおこなっています。

• String型からLocalDate型への変換
• バリデーションチェック

「limitDateに有効な型以外を入力した時にバリデーションエラーとなること」という名前のテストでgetLimitDateメソッドを使用しているので、モック化が必要だと思い記述しています。

なくてもテストできるでしょうか？

[yoshi-koyama]

なしでテストしてみてくれますか？
要らないと思います。

[mkmmr]

@InjectMocksがなくても動きました。
必要だと思っていました。Junitの理解がまだまだ足りないようです……。

@InjectMocksを削除しました。 474b89a

[yoshi-koyama]

となるとこれもいらないですね。

[mkmmr]

こちらも上記と同じ理由で記載しています。

[yoshi-koyama]

たしかにロケール（locale）の問題が絡んでくるか・・・。
以下でうまくいくかどうか試してみてくれますか？
①テストコード内にロケールを指定する
https://stackoverflow.com/questions/16466258/how-to-customize-the-locale-for-javax-validation

static {
    Locale.setDefault(Locale.JAPAN);
}

②ワークフロー内でubuntuのロケールを変更する
test_gradle.yml に手を加えてlocaleを変更してみてほしいです。
参考：https://qiita.com/tailak/items/77b90a4df07e4f6a1fc2

上村さんのローカルPCでも使えると思いますがlocaleコマンドでロケール情報が出力されますのでワークフロー内でlocaleコマンドを実行してロケールが正しく変更できているかどうか確認しつつ進めてほしいです。

[mkmmr]

ありがとうございます。

1. テストコード内にロケールを指定する 7d4a11e
2. ワークフロー内でubuntuのロケールを変更する（こちらを参考にしました） f1c8175

この2つを行ったところ、Github上でも無事にテストが成功するようになりました。