5

よそから利用されないようにprivateにしたフィールドAがあります。
ですが、テストのためにそのフィールドにアクセスする必要が出てきました。
そのためにjava.lang.reflectを用いてアクセスすることにしました。

そのフィールドはテストしたい自作クラスBの中にあり、自作クラスC型の変数です。
また、プログラムの仕様上自作クラスBをテストするためには
事前に自作クラスCのインスタンスを作ってセットしておく必要があります。

テストにはJUnit4を使っていて、
setUp内で自作クラスCのインスタンスを作ってセットしています。
ですが、テスト中にクラスBクラスCのインスタンスがないと言ってきます。

そこで、確認のために適当に作ったプログラムで
privateなフィールドへの値セットと取得等を試してみました。

確認用に作ったSetCallTestクラス

package test;

import java.lang.reflect.Field;

public class SetCallTest {
    private HasPrivate hasPrivate;
    private Nakami nakami;

    public static void main(String[] args) {
        SetCallTest test = new SetCallTest();
        System.out.println("Nakamiをセットします。");
        test.setNakami();
        System.out.println("セットしたNakamiを取得します。");
        test.getNakami();
        System.out.println("もともと入れてあるNakamiを取得します。");
        test.getArimasu();
    }

    public SetCallTest() {
        hasPrivate = new HasPrivate();
        nakami = new Nakami();
    }

    public void setNakami() {
        try {
            Field privateField = HasPrivate.class.getDeclaredField("nakami");
            privateField.setAccessible(true);
            privateField.set(hasPrivate, nakami);
        } catch (NoSuchFieldException | SecurityException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        } catch (IllegalArgumentException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        } catch (NullPointerException e) {
            e.printStackTrace();
        }
    }

    public void getNakami() {
        try {
            Field privateField = HasPrivate.class.getDeclaredField("nakami");
            privateField.setAccessible(true);
            Nakami nakami2 = (Nakami) privateField.get(hasPrivate);
            System.out.println("取得したnakami:" + nakami2);
            System.out.println("HasPrivateからもらったnakami:" + hasPrivate.getNakami());
            System.out.println("Nakamiのnakami:" + nakami2.nakami);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    public void getArimasu() {
        try {
            Field arimasu = HasPrivate.class.getDeclaredField("arimasu");
            arimasu.setAccessible(true);
            Nakami nakami2 = (Nakami) arimasu.get(hasPrivate);
            System.out.println("取得したarimasu:" + nakami2);
            System.out.println("HasPrivateからもらったariamsu:" + hasPrivate.getArimasu());
            System.out.println("arimasuのnakami:" + nakami2.nakami);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

自作クラスBの代わりのHasPrivateクラス

package test;

public class HasPrivate {
    private Nakami nakami;
    private Nakami arimasu=new Nakami();

    public Nakami getNakami(){
        return nakami;
    }

    public Nakami getArimasu(){
        return arimasu;
    }
}

自作クラスCの代わりのNakamiクラス

package test;

public class Nakami {
    public String nakami="中身";
}

HasPrivate#arimasuSetCallTestから取得できますが、
HasPrivate#nakamiNullPointerExceptionになります。
SetCallTestからセットできていないんだろうと思いつつ
HasPrivatenakami取得用のpublicなメソッドを用意して呼び出してみましたが、
やはり同様です。

出力ログ

Nakamiをセットします。
セットしたNakamiを取得します。
取得したnakami:test.Nakami@6d06d69c
HasPrivateからもらったnakami:test.Nakami@6d06d69c
Nakamiのnakami:中身
もともと入れてあるNakamiを取得します。
取得したarimasu:test.Nakami@7852e922
HasPrivateからもらったariamsu:test.Nakami@7852e922
arimasuのnakami:中身

確認用コード間違えすぎてすみません。
本物のコードの一部抜粋したものを以下に載せます。

package com.example.sakaguchi.test.all.model;

import android.support.test.runner.AndroidJUnit4;
import android.test.suitebuilder.annotation.LargeTest;

import org.json.JSONException;
import org.json.JSONObject;
import org.junit.Before;
import org.junit.Test;
import org.junit.runner.RunWith;

import java.lang.reflect.Method;
import java.util.ArrayList;

import com.example.sakaguchi.model.PriceItem;
import com.example.sakaguchi.model.PriceListDataLoader;
import com.example.sakaguchi.model.PriceListData;

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertNull;

/**
 * PhotoListDataのテスト.
 * ダミーのjsonを渡してそれ通りにデータを取得等できるかをテストする。
 * 現状はとりあえず以下の3パターン。
 * ・集合写真
 * ・スナップ写真・データ販売あり・現像販売あり
 * ・セッション切れ
 */
@LargeTest
@RunWith(AndroidJUnit4.class)
public class PriceListDataLoaderTest {
    PriceListDataLoader mPriceListDataLoader;
    PriceListData mPriceListData;
    JSONObject mPriceListJson, mSessionOutJson;

    @Before
    public void setUp() {
        mPriceListData = new PriceListDataLoader();

        mPriceListData = new PriceListData();
        Field priceListData = HasPrivate.class.getDeclaredField("mPriceListData");
    priceListData.setAccessible(true);
    priceListData.set(mPriceListDataLoader, mPriceListData);

        try {
            //json作成
        } catch (JSONException e) {
            e.printStackTrace();
        }
    }

    /*以下テスト用メソッドが続く。
    * mPriceListDataLoaderのこれまたprivateなメソッドを呼び出す。
    * そのメソッドは引数のjsonからデータを取得・加工したものをPriceListDataに入れる。
    * そのPriceListDataをチェックして想定通りのデータが入っているかどうか確認したい。
    * ですが、そもそもsetUpでmPriceListDataLoaderに設定したはずの
    * mPriceListDataがnullだと言われてしまう。*/
}

今本番の環境がないので本物のコードを試せないのですが、
こういうことをやっていてうまくいかなくて
JUnit4を使わずに試してみようと作ったのが確認用のプログラムです。
そちらは適当すぎていろいろ間違えていましたが、きちんと動作するよう修正したコードは
これとやっていることはそんな変わらないと思うのですが、
本物ではなにかやり方を間違えているのでしょうか?

3 件の回答 3

2

setNakamigetNakamigetArimasuの各メソッドにはそれぞれ

HasPrivate hasPrivate = new HasPrivate();

という行がありますが、このhasPrivateはそれぞれ別のオブジェクトになっています。

setNakami(nakami);

と呼び出すと、HasPrivateオブジェクトが作成され、nakamiがプライベートなフィールドにセットされますが、 setNakamiの呼び出しの終了とともに、せっかくプライベートなフィールドにnakamiをセットしたHasPrivateオブジェクトが破棄されています。 そして、次の

getNakami(nakami);

の呼び出しでは、先ほどsetNakami(nakami);でフィールドにセットしたオブジェクトとは異なるHasPrivateオブジェクトを改めて作り直しており、新たに作られたHasPrivateオブジェクトのnakamiフィールドは作りたてなのでnullのままで、そのため、

Nakami nakami2 = (Nakami) privateField.get(hasPrivate);

としてフィールドを読み取ってもnakami2nullになります。

setNakamigetNakamigetArimasuの各メソッドでそれぞれ別のHasPrivateオブジェクトを使っており、setNakamiを呼び出した意味がなくなってしまっているのが原因だと思われます。

おそらくですが、このようなことがしたいのではないでしょうか:
https://ideone.com/w3Xui9

3
  • プログラムをコピペで作っていたら間違えてました。 もう少し本物風に書き換えます。 Commented 2015年8月28日 15:32
  • 提示してくださったコードと違って本物は書き換えたコードのようにHasPrivateNakamiにあたるにあたるクラスのインスタンスを入れる変数をクラスのフィールドとして定義しているのですが、それは関係あるでしょうか? Commented 2015年8月28日 15:39
  • それはただまた私がコードを間違えてしまっていただけでした。すみません。 Commented 2015年8月29日 3:10
2

メソッド setNakami() の先頭にある

HasPrivate hasPrivate = new HasPrivate();

hasPrivate = new HasPrivate();

にすることで、提示していただいたコードは動作します。

ですがこれはJUnit4を使うことなく、Javaをそのまま実行しても動作するコードですね。
JUnitを利用する前提として、動作しているクラスやメソッドに対して、JUnit用のクラスを用意して実行するもので、そして実行結果を評価するものです。

JUnit4に関しては簡単なサンプルコードとして以下のサイトがあります。
http://nanasess.net/mt/2008/04/junit4.html

JUnit4の使い方とできることを簡単にまとめているスライドです。
http://www.slideshare.net/ichikaz3/junit4

これらも参考にしてください。

1
  • ですね。 また間違えてました。すみません。 ここで提示しているコードは確認用に作ったもので、「そもそもreflectでprivateなフィールドにアクセスしたこともなかったから確認してみよう。」というものでした。 適当すぎて間違えすぎて自分の勘違いがありましたが。 Commented 2015年8月29日 2:45
1

本日実際のコードを確認したところ、
うまく行かなかったのはreflectするときにクラスを間違えていたからでした。
きちんと確認していれば気がつけたミスですみません。

package com.example.sakaguchi.test.all.model;

import android.support.test.runner.AndroidJUnit4;
import android.test.suitebuilder.annotation.LargeTest;

import org.json.JSONException;
import org.json.JSONObject;
import org.junit.Before;
import org.junit.Test;
import org.junit.runner.RunWith;

import java.lang.reflect.Field;
import java.lang.reflect.Method;
import java.util.ArrayList;

import com.example.sakaguchi.model.PriceItem;
import com.example.sakaguchi.model.PriceListData;
import com.example.sakaguchi.model.PriceListDataLoader;

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertNull;

/**
 * PhotoListDataのテスト.
 * ダミーのjsonを渡してそれ通りにデータを取得等できるかをテストする。
 * 現状はとりあえず以下の3パターン。
 * ・集合写真
 * ・スナップ写真・データ販売あり・現像販売あり
 * ・セッション切れ
 */
@LargeTest
@RunWith(AndroidJUnit4.class)
public class PriceListDataLoaderTest {
    PriceListDataLoader mPriceListDataLoader;
    PriceListData mPriceListData;
    JSONObject mPriceListJson, mSessionOutJson;

    @Before
    public void setUp() {
        mPriceListDataLoader = new PriceListDataLoader();

        try {
            mPriceListData = new PriceListData();
            Field field = PriceListDataLoader.class.getDeclaredField("mPriceListData");
            //ここをPriceListDataLoaderではなく誤ってPriceListDataと書いてしまっていた。
            field.setAccessible(true);
            field.set(mPriceListDataLoader, mPriceListData);
        } catch (NoSuchFieldException e) {
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        }

        try {
            //json作成
        } catch (JSONException e) {
            e.printStackTrace();
        }
    }

    /*以下テスト用メソッドが続く。
    * mPriceListDataLoaderのこれまたprivateなメソッドを呼び出す。
    * そのメソッドは引数のjsonからデータを取得・加工したものをPriceListDataに入れる。
    * そのPriceListDataをチェックして想定通りのデータが入っているかどうか確認する。*/
}

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。