パラメーターを使用してコンストラクターをモックします


89

私は以下のようなクラスを持っています:

public class A {
    public A(String test) {
        bla bla bla
    }

    public String check() {
        bla bla bla
    }
}

コンストラクターのロジックA(String test)check()は、私がモックしようとしているものです。次のような呼び出しが必要です:new A($$$any string$$$).check()ダミー文字列を返します"test"

私は試した:

 A a = mock(A.class); 
 when(a.check()).thenReturn("test");

 String test = a.check(); // to this point, everything works. test shows as "tests"

 whenNew(A.class).withArguments(Matchers.anyString()).thenReturn(rk);
 // also tried:
 //whenNew(A.class).withParameterTypes(String.class).withArguments(Matchers.anyString()).thenReturn(rk);

 new A("random string").check();  // this doesn't work

しかし、それは機能していないようです。new A($$$any string$$$).check()のモックオブジェクトをフェッチする代わりに、コンストラクタロジックを実行していますA


模擬のcheck()メソッドは正しく機能していますか?
ベングラッサー2012年

@BenGlasser check()は問題なく動作します。whenNewがまったく機能していないようです。説明も更新しました。
Shengjie 2012年

回答:


93

あなたが投稿したコードは、MockitoとPowermockitoの最新バージョンで動作します。たぶんあなたはAを準備していませんか?これを試して:

A.java

public class A {
     private final String test;

    public A(String test) {
        this.test = test;
    }

    public String check() {
        return "checked " + this.test;
    }
}

MockA.java

import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.equalTo;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;

import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.Mockito;
import org.powermock.api.mockito.PowerMockito;
import org.powermock.core.classloader.annotations.PrepareForTest;
import org.powermock.modules.junit4.PowerMockRunner;

@RunWith(PowerMockRunner.class)
@PrepareForTest(A.class)
public class MockA {
    @Test
    public void test_not_mocked() throws Throwable {
        assertThat(new A("random string").check(), equalTo("checked random string"));
    }
    @Test
    public void test_mocked() throws Throwable {
         A a = mock(A.class); 
         when(a.check()).thenReturn("test");
         PowerMockito.whenNew(A.class).withArguments(Mockito.anyString()).thenReturn(a);
         assertThat(new A("random string").check(), equalTo("test"));
    }
}

両方のテストは、mockito 1.9.0、powermockito 1.4.12、およびjunit4.8.2で合格する必要があります。


24
また、コンストラクターが別のクラスから呼び出された場合は、次のリストに含めることに注意してくださいPrepareForTest
Jeff E

「PowerMockito.whenNew」が呼び出されたときに、なぜ自分自身を準備する必要があるのか​​、誰にも分かりますか?
udayanga

50

私の知る限り、mockitoでコンストラクターをモックすることはできず、メソッドのみをモックすることができます。しかし、Mockito googleコードページのwikiによると、クラス内にそのクラスの新しいインスタンスを返すメソッドを作成することにより、コンストラクターの動作をモックする方法があります。次に、そのメソッドをモックアウトできます。以下は、Mockitowikiからの直接抜粋です。

パターン1-オブジェクト作成に1行のメソッドを使用

パターン1(MyClassというクラスのテスト)を使用するには、次のような呼び出しを置き換えます。

   Foo foo = new Foo( a, b, c );

   Foo foo = makeFoo( a, b, c );

1行のメソッドを記述します

   Foo makeFoo( A a, B b, C c ) { 
        return new Foo( a, b, c );
   }

メソッドにロジックを含めないことが重要です。オブジェクトを作成する1行だけ。これは、メソッド自体が単体テストされることは決してないためです。

クラスをテストするようになると、テストするオブジェクトは実際にはMockitoスパイになり、このメソッドがオーバーライドされて、モックが返されます。したがって、テストしているのはクラス自体ではなく、クラスのごくわずかに変更されたバージョンです。

テストクラスには、次のようなメンバーが含まれている可能性があります

  @Mock private Foo mockFoo;
  private MyClass toTest = spy(new MyClass());

最後に、テストメソッド内で、makeFooの呼び出しを次のような行でモックアウトします。

  doReturn( mockFoo )
      .when( toTest )
      .makeFoo( any( A.class ), any( B.class ), any( C.class ));

コンストラクターに渡される引数を確認する場合は、any()よりも具体的なマッチャーを使用できます。

クラスのモックオブジェクトを返したいだけなら、これでうまくいくと思います。いずれにせよ、モックオブジェクトの作成について詳しくは、こちらをご覧ください。

http://code.google.com/p/mockito/wiki/MockingObjectCreation


21
+1、ソースコードを調整してモックを使いやすくする必要があるという事実は気に入らない。共有していただきありがとうございます。
Shengjie 2012年

22
よりテスト可能なソースコードを用意したり、コードを作成するときにテスト容易なアンチパターンを回避したりすることは決して悪いことではありません。よりテストしやすいソースを作成すると、自動的により保守しやすくなります。コンストラクター呼び出しを独自のメソッドで分離することは、これを実現する1つの方法にすぎません。
Dawood ibn Kareem 2012年

1
テスト可能なコードを書くのは良いことです。AはCにハードコードされた依存関係を持っているため、Aに依存するクラスBのテストを作成できるように、クラスAを再設計することを余儀なくされています。ええ、コードは最終的には良くなりますが、1つのテストを書き終えるために、いくつのクラスを再設計することになりますか?
マークウッド

私の経験では、@ MarkWoodの不格好なテスト経験は、一般的にいくつかの設計上の欠陥の兆候です。IRLは、コンストラクターをテストしている場合、コードがファクトリまたは依存性注入を求めて叫んでいる可能性があります。これら2つのケースの一般的なデザインパターンに従うと、コードのテストと一般的な操作がはるかに簡単になります。そこにたくさんのロジックがあるためにコンストラクターをテストしている場合は、おそらくポリモーフィズムのレイヤーが必要です。または、そのロジックを初期化メソッドに移動することもできます。
ベングラッサー

12

Powermockを使用せずに....それを理解するのに時間がかかったので、BenGlasserの回答に基づいた以下の例を参照してください..時間を節約できることを願っています...

元のクラス:

public class AClazz {

    public void updateObject(CClazz cClazzObj) {
        log.debug("Bundler set.");
        cClazzObj.setBundler(new BClazz(cClazzObj, 10));
    } 
}

変更されたクラス:

@Slf4j
public class AClazz {

    public void updateObject(CClazz cClazzObj) {
        log.debug("Bundler set.");
        cClazzObj.setBundler(getBObject(cClazzObj, 10));
    }

    protected BClazz getBObject(CClazz cClazzObj, int i) {
        return new BClazz(cClazzObj, 10);
    }
 }

テストクラス

public class AClazzTest {

    @InjectMocks
    @Spy
    private AClazz aClazzObj;

    @Mock
    private CClazz cClazzObj;

    @Mock
    private BClazz bClassObj;

    @Before
    public void setUp() throws Exception {
        Mockito.doReturn(bClassObj)
               .when(aClazzObj)
               .getBObject(Mockito.eq(cClazzObj), Mockito.anyInt());
    }

    @Test
    public void testConfigStrategy() {
        aClazzObj.updateObject(cClazzObj);

        Mockito.verify(cClazzObj, Mockito.times(1)).setBundler(bClassObj);
    }
}

6

mockitoを使用すると、withSettings()を使用できます。たとえば、CounterServiceに2つの依存関係が必要な場合は、それらをモックとして渡すことができます。

UserService userService = Mockito.mock(UserService.class); SearchService searchService = Mockito.mock(SearchService.class); CounterService counterService = Mockito.mock(CounterService.class, withSettings().useConstructor(userService, searchService));


私の意見では、最も簡単で最良の答えです。ありがとうございました。
エルドン

4

Mockitoには、final、static、およびprivateメソッドのテストに制限があります。

jMockitテストライブラリを使用すると、以下のように非常に簡単で簡単な作業をいくつか行うことができます。

java.io.Fileクラスのモックコンストラクタ:

new MockUp<File>(){
    @Mock
    public void $init(String pathname){
        System.out.println(pathname);
        // or do whatever you want
    }
};
  • パブリックコンストラクター名は$ initに置き換える必要があります
  • スローされる引数と例外は同じままです
  • 戻り値の型はvoidとして定義する必要があります

静的メソッドをモックします。

  • メソッドのモックシグネチャからスタティックを削除します
  • メソッドシグネチャはそれ以外は同じままです
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.