[JUnit] 戻り値のないvoidメソッドはテストする必要はあるのか？

現在JUnitについて勉強しています。

質問があるのですが、
結論から言うと、

戻り値のないvoidメソッドはテストする必要はあるのでしょうか？
もしする必要があればどのようなテストを書けば良いでしょうか？

今過去に自分が書いたプロダクトコードのテストコードを書いてみようと思考錯誤しています。
例えば下記のようなSampleクラスのmethメソッドはどのようなテストコードが書けるでしょうか？
あるいはテストコードを書く必要はないのでしょうか？

Java
1public class Sample
2{
3	String name = "Sugai";	// Member Variable
4
5	 void meth()
6	{
7		String name = "Tochihara";	
8		System.out.println(this.name);	// 何が出力されますか？
9	}
10	  public static void main(String[] args)
11		{
12		  Sample d=new Sample();
13		  d.meth();
14		}
15
16}

テストの意図としてはこのような感じになるのかなと思って書いています。

Java
1public class SampleTest {
2
3	@Test
4	public void testEqual() {
5		Sample d = new Sample();
6		assertEquals("Sugai", d.meth());  //コンパイルエラー
7
8	}
9
10}

しかし戻り値がないのでassertEqualsメソッドが使えなくて困っています。当たり前ですが。。
どなたかアドバイスいただけるとありがたいです。

行動規範の内容に同意します

回答2件

ベストアンサー

返り値が void メソッドでもテストする項目はいろいろあります。

メソッド呼び出しの副作用の内容を確認する。

(System.out.println の内容や log 出力内容をテストする)
2. excepiton が発生する条件をテストする。

参考情報：

JUnitで標準出力の内容をテストするには？ https://teratail.com/questions/3694
JUnit with System.out.println https://limzhenghong.wordpress.com/2015/03/18/junit-with-system-out-println/
JUnit test for System.out.println() http://stackoverflow.com/questions/1119385/
Log4jのログ出力内容をJUnitでテストする方法 http://qiita.com/sifue/items/489d2d1ffed98571b88c

↓では System.out.println の内容と例外発生の条件をテストしてみています。
(例外発生のテストの為に meth2(String[]) というメソッドも追加定義しています)

Sample2.java

java
1public class Sample2 {
2    String name = "Sugai"; // Member Variable
3
4    void meth() {
5        String name = "Tochihara";
6        System.out.println(this.name); // 何が出力されますか？
7    }
8
9    void meth2(String[] args) {
10        if (args.length == 0) {
11            throw new IllegalArgumentException("args.length == 0");
12        }
13        System.out.println(this.name + " " + args[0]);
14    }
15
16    public static void main(String[] args) {
17        Sample2 d = new Sample2();
18        d.meth();
19        d.meth2(args);
20    }

Sample2Test.java

java
1import static org.junit.Assert.assertEquals;
2import static org.junit.Assert.assertTrue;
3import static org.junit.Assert.fail;
4
5import java.io.ByteArrayOutputStream;
6import java.io.PrintStream;
7
8import org.junit.After;
9import org.junit.Before;
10import org.junit.Test;
11
12public class Sample2Test {
13
14    private final ByteArrayOutputStream outContent = new ByteArrayOutputStream();
15    private final ByteArrayOutputStream errContent = new ByteArrayOutputStream();
16
17    @Before
18    public void setUpStreams() {
19        System.setOut(new PrintStream(outContent));
20        System.setErr(new PrintStream(errContent));
21    }
22
23    @After
24    public void cleanUpStreams() {
25        System.setOut(null);
26        System.setErr(null);
27    }
28
29    @Test
30    public void testMath() {
31        Sample2 s2 = new Sample2();
32        s2.meth();
33        assertEquals("Sugai\n", outContent.toString());
34    }
35
36    @Test
37    public void testMath2WithNull() {
38        try {
39            Sample2 s2 = new Sample2();
40            s2.meth2(null);
41            fail();
42        } catch (NullPointerException e) {
43            assertTrue(true);
44        }
45    }
46
47    @Test
48    public void testMath2WithNEmpty() {
49        try {
50            Sample2 s2 = new Sample2();
51            String[] args = {};
52            s2.meth2(args);
53            fail();
54        } catch (IllegalArgumentException e) {
55            assertEquals("args.length == 0", e.getMessage());
56        }
57    }
58
59    @Test
60    public void testMath2() {
61        Sample2 s2 = new Sample2();
62        String[] args = { "xxx" };
63        s2.meth2(args);
64    }
65
66    @Test
67    public void testMain() {
68        try {
69            Sample2.main(null);
70            fail();
71        } catch (NullPointerException e) {
72            assertTrue(true);
73        }
74    }
75}

eclipse で unit test を実行させたときのスクリーンショット：

投稿2015/10/18 11:42

katoy

総合スコア22324

k499778

2015/10/18 12:05

katoyさん回答ありがとうございます。丁寧でわかりやすく大変参考になります。１つ質問なのですが、 testMath2メソッドにはassertEqualsのようなtrue,falseを判定するメソッドはいらないのでしょうか？確かにargs変数の中身をnullにすると例外が発生しテストに失敗するので、正常に処理が流れること自体がテスト成功という意味でとくにtrue,falseを判定するメソッドは必要ないのかなとも思ったのですが。初歩的な質問で申し訳ないのですが、もしよろしければお答えください。