void型のテストコードの書き方

Question

void型のテストコードの書き方がわかりません。

正しくcase文が動いているかをテストしたい場合はどのように書くと良いでしょうか？
よろしくお願いします。

    public void createBehaviorUrl(String behavior,String url,WebView webView){

        String topPage ="http://kuruma/AutonituveMaterials/";

        switch(behavior) {
            case "履歴から戻る":
                if(webView.canGoBack()) { webView.goBack(); }
                break;
            case "トップページ":
                urlLoad(topPage,webView);
                break;
            case "次のページ":
                urlLoad(nextHtml(url),webView);
                break;
            case "前のページ":
                urlLoad(backHtml(url),webView);
                break;
        }

    }

Comments

[defghi1977]

引数として使うWebViewのテスト用の子クラス作ってメソッド実行後の内部状態を採れるように細工しておくとか

[L85A2]

テストの為に、クラスを作成してしまうとテストとテストをしたいプログラムを別々に区切ることができなく、あまりよくないコードになってしまいそうですね。使用上仕方のない事なのかもしれませんが。

[defghi1977]

createBehaviorUrlメソッドのテストとしてはそうする他なく, たとえテストの為のクラスを作る行為が気になるとしても, テストを行うことによるメリットのほうが上回ると思います. ここらへんはテストを行うためのテクニックとして受け入れざるを得ないかと.(もちろん継承不可だと死にますが)

[L85A2]

確かに、テストの方がメリットは大きそうですね。

Answer 1

テストするのはメソッドを実行した結果だけです。
中の実装までは普通テストしません。
内部の switch がうまく分岐しているかしりたい場合にはデバッグ実行してブレークポイントでの変数の内容やデバッグ出力を見て判断します。

この質問の場合はメソッドを実行すると WebView がページを読み込むので読み終わるのを待って遷移先を見ればいいと思います。
[Android]WebViewのイベントが発生した時に任意の処理をさせる

Answer 2

Mockitoを導入し、Verfiyで確認するのが一般的かと思います。

Answer 3

画面遷移を検査したいということならばユニットテストではなくUIテストとして seleniumなどを用いるのではどうでしょうか？