JUnitの引数の確認の仕方

Question

Eclipse Mars
JUnit

car.set(MAX_E);

上記のようなメソッドで、引数にセットする値が間違えていないかチェックする方法をご教示いただけませんか？

補足
Aシステムの閉塞を確認したいので引数にAをいれるべきだが間違えてBを入れてしまっていた場合まずいですよね？引数の中身の確認をしたいのですがどのようにするのでしょうか？

テスト時のことです。

Comments

[euledge]

テストのときでしょうか？それとも運用での実行時にということでしょうか？

[退会済みユーザー]

えっと テストケースをテストするテストケースということでしょうか（笑)?

[退会済みユーザー]

つまりプログラムがあなたの頭とハンドシェイク通信をするテストケースを作りたいのですか?→補足

Answer 1

テストっていうのは、下記のようなクラスの実装があって、(下記はあくまでもクラス名とメソッド名を質問者の書いたものに合わせただけのものです)

public class Car {
  public boolean set(String value){
     if(....) {
       return true;
     }
     return false;
  }
}

引数にある値を入れたときに、その関数が正しく処理を行うかどうかを判定するものです。
したがってMAX_Eが正しいかどうかをテストするのではなく MAX_Eを入れたときにcar.setの返り値が期待通り(trueなのかfalseなのか)になっているかどうかということをテストしなければいけません。

これをJUnitでは以下のように表現をします。

assertTrue(car.set(MAX_E));

Comments

[sobue]

ありがとうございます。補足に記載しました。

[euledge]

car.set()に引数Aが入るべきだが、Bが入ったら困るのであれば、JUnitでテストを行うものではなく、Aしか入らないような実装をするべきだと思います。 MAX_E が正の整数でなければいけないのであれば if (MAX_E>0) {car.set(MAX_E);} という実装を書くべきです。もしくは上記の回答のcar.setの関数内のif文で判定を入れるかです。後者の場合のJunitでのテストの方法を上記の回答で示しています。

[sobue]

では、MAX_E以外の整数値が入っていた場合はどうしたらよいのですか？部品化しているので引数は違うものが入り、複数は正常に終了する想定があるはずです。正常に終了する想定だが、仕様上ここではこの引数ではないときがあるので、それを検証したいのです。

[euledge]

何となく議論が噛み合ってない気がします。sobueさんがテストと言っているのは、実装されたシステムを実際に動かしたときの結合テストのことを言っていますか?

[sobue]

単体テストで検証するような内容だと思います。実際単体テスト検証中です。

[euledge]

MAX_Eと言うのは変数ではなく定数ですか?
また、単体テストをされているのであれば該当するJunitのテストコードをお示し頂けると、論点がクリアになると思います。

[sobue]

定数です。定数ですが他の定数も入る想定です。がここではほかの定数は仕様上入ってはいけないものになります。

[euledge]

引数がMAX_E(例えば1000と定義されてたとします)でない場合、car.setはどのように動作するのが正解ですか?

例外を出すのか、関数の返り値としてfalseを出すのか 等

[sobue]

戻り値は　booleanです。
trueが正常値です

[euledge]

では、car.set()はMAX_Eで定義された値以外が入っていればfalseを返すという仕様ということで良いですね。
こちらのモヤモヤが解消できたので、回答示せると思います。もう少しお付き合いください。

Answer 2

前回の回答の繰り返しになりますが、
仕様に対する実装コードがあって、その中の検証したいメソッドにある引数を与えた時、そのメソッドが仕様通りの値を返すかどうかをテストコードとして記述します。
したがって、正しい値を入れたときの検証と正しくない値を入れた時の検証を書けばよいのです。

実装側のコードが以下のようなものであるとします。

// 仕様:引数valueがMAX_Eである場合はtrue,そうでない場合はfalseを返す
public class Car {
  public boolean set(Integer value){
     if(value.equals(MAX_E)) {
       return true;
     } else {
       return false;
     }
  }
}

上記実装の仕様からCar.set()のテストとしては

• car.setにMAX_Eが渡されたときTrueを返す
• car.setにMAX_E以外が渡されたときFalseを返す

ということを検証することになりますのでそれをJunitでコードにします。
したがってテストコードは以下のようになります。

import static org.junit.Assert.*;

import org.junit.Test;

public class JunitCarTest {
  @Test
  public void 関数setに正しい引数が渡されたときTrueを返す() {
    Car car = new Car();
    assertTrue(car.set(MAX_E));
  }
  @Test
  public void 関数setに正しい引数が渡されないときFalseを返す() {
    Car car = new Car();
    int invalidValue = -1; // MAX_Eでない値であれば何でもよい。
    assertFalse(car.set(invalidValue));
  }
}

もし、car.setの実装が下記のように無条件でtrueを返す(仕様通りではない)実装であったとしたら、
先のテストコードの 「関数setに正しい引数が渡されないときFalseを返す」のassertFalse(invalidValue)の部分が失敗するので実装が誤っていることが検証できます。

  public boolean set(Integer value){
     return true;
  }

Comments

[sobue]

すみません。少し違います。
car.set(HALF_E);
でも、仕様上trueになります。
HALF_Eが入ってないMAX_Eであることを確認したいのです。

[euledge]

car.setに何が入ればtrueなのでしょうか？MAX_Eという表現ではなく具体的な値として (偶数ならtrueとか、1000以下の正数ならtrueとか)

[euledge]

上記回答の中で関数の定義の中の引数例として value という名前を用いました。この場合、関数を呼ぶ側で car.set(MAX_E)としてもcar.set(HALF_E)としても関数の中ではどちらもvalueという名前になるので区別はつかないです。あくまでもその変数の持つ値でしかテストは出来ないです。

[euledge]

例えば、お金を計算するようなメソッドに対して、同じ100という値であっても変数として 身長を表す height=100として定義された 変数heightを入れたくないのでチェックしたい。と言うことを望んでいらっしゃいますか?

[sobue]

そんなイメージです。

[euledge]

その要望は残念ながらJUnitでは実現できないです。
したがって、元のメソッドの実装側で工夫して行うしかないです。