junitテストコード書き方

Question

 public String Test01(String in){
  if(in == null || in.length() != 13){
    return null;
  String phoneNumber = res(in);
   if(phoneNumber .length() != 15){
    return null;
   }
 }
}


//引数桁テスト
＠Test　public void 01(){
assertNull("12345", target.test001(null));
assertNotNull(target.test001("123456789012345"));
}

junitテストコードを書くとき、Test01メソッドの引数に
ついてのテストコードは作成できますが、Test01の中のif(phoneNumber .length() != 15)
に対してはテストコードを書いて実行することはできないでしょうか？

テストコードで引数を１５にしてしまうと if(in == null || in.length() != 13)でreturn null;
でそれ以上進めない。

junitというのは Test01メソッドでいうとpublic String Test01(String in){}の最初の引数inに対してのみの試験となりますか？

Comments

[m.ts10806]

そもそもどういうテストケースを作ったのでしょうか？
機能仕様次第なのでどういう試験が必要かは設計次第ということになります。
この内容だけでは他者には何とも言えません。
設計者に聞くべきところもあります。コードだけでは話ができません。

[kanekoK]

if(phoneNumber .length() != 15)に対してテストコードの作成をしたいです。

[m.ts10806]

設計した人に聞いてください。
そもそもTest01()というメソッドもコンパイルエラーがでる内容ですし（returnの後に同じスコープ内にコードがある）、やはりコードだけでは話ができません。

[kanekoK]

テスト設計書には引数１５に対しての試験項目が存在します。ですが、その場合junitでどのように実行できるかがわかりません。記載したテストコードではjava.lang.AssertErrorが出てしまいます。

[m.ts10806]

>テスト設計書には引数１５に対しての試験項目が存在します。

テスト設計書があるのなら余計に設計者に聞くべき内容です。
何の事情も知らない赤の他人にはどうにもできません。
それにテストしたい機能自体もご自身が組まれたものではないように見受けられます。
他者がどうもできない条件が揃いすぎてます。

[momon-ga]

> 記載したテストコードではjava.lang.AssertErrorが出てしまいます。
それはテストが失敗している＝実装が正しくない
ということです。

そもそもですが、target.test001("123456789012345")
は、NotNullになることを確認したいのですか？
どのような値になることを想定していますか？

[kanekoK]

数字以外が含まれているかいないかの確認をしたいです。

Answer 1

public String Test01(String in){
  if(in == null || in.length() != 13)
    return null;

  String phoneNumber = res(in);
  if(phoneNumber.length() != 15)
    return null;

　// phoneNumber.length() == 15 のときの処理はここから
}

if(in == null || in.length() != 13){　このブロックの中に書いてしまっているせいではないですか？
今のソースだとinがnullもしくはin.length()が13以外だとnullを返して、13だったとしても何もしていないんじゃないですか？

Comments

[kanekoK]

Junitがエラーが出てしまって。。すいません、確かに記載したコードも間違っておりました。しゅうせいしましたが、以前エラーのままです。。

Answer 2

JUnitは、テストしたいメソッドに値を渡したときの戻り値が想定通りかどうか、のテストをするためのツールです。

テスト設計書とは別に、設計書がありませんか？
もし設計書があれば、具体的な処理ロジックが書いてあるはずなので、その内容をテストできるテストコードを書いてください。

実装コードでres()メソッドを呼び出しているかどうか、は考慮しません。
想定通りの結果にならないときに何でだろうかと処理ロジックを確認する過程でres()メソッドの中を見る、ということはあります。

とはいえ、稀に絶対phoneNumberの値が15桁以外にならないはずなのに15桁以外ならエラー、というロジックがあって、それはJUnitではテストが出来ない、ということもありますが…。
その時はテスト仕様書を書いた人と話し合ってください。

Comments

[kanekoK]

ここ設計書がなにもないんです。。いきなりコードです。。

[deaf_tadashi]

それは厳しい…
JUnitでテスト必須、ということならパラメータが13桁以外はエラーになる確認と、res()の戻り値が正しいパターンの2件だけ書いてとりあえず終わりにするしかないかと。

res()メソッドの中身を読んで15桁以外が返ってくるパターンが把握できるなら、そのパターンも書く、くらいしかアドバイスが出来ないです。

[kanekoK]

どうしたらいいか、まず設計を詳しく聞いてみます！

Answer 3

	public String Test01(String in) {
		if (in == null || in.length() != 13) {
			return null;
			String phoneNumber = res(in);
			if (phoneNumber.length() != 15) {
				return null;
			}
		}
	}

ですので、まずは in に入る文字列は13文字である必要があるようです。
res() メソッドに特定の13文字をいれた結果（これに関しては私たちではわかりません）、phomeNumberの結果を判定してください。