web-dev-qa-db-ja.com

PowerMockito.whenNew（）の使用はモックされず、元のメソッドが呼び出されます

以下のようなコードがあります。

Class A {
  public boolean myMethod(someargs) {
    MyQueryClass query = new MyQueryClass();
    Long id = query.getNextId();
    // some more code
  }
}
Class MyQueryClass     {
  ....
  public Long getNextId() {
    //lot of DB code, execute some DB query
    return id;
  }
}

現在、A.myMethod(someargs)のテストを書いています。実際のメソッドquery.getNextId()をスキップして、代わりにスタブ値を返したいです。基本的に、MyQueryClassをモックしたい。

だから私のテストケースでは、私は使用しています：

MyQueryClass query = PowerMockito.mock(MyQueryClass.class);
PowerMockito.whenNew(MyQueryClass.class).withNoArguments().thenReturn(query);
when(query.getNextId()).thenReturn(1000000L);

boolean b = A.getInstance().myMethod(args);

//asserts

テストクラスの最初に@RunWith(PowerMockRunner.class)と@PrepareForTest({MyQueryClass.class})を使用しました。

しかし、テストをデバッグするとき、MyQueryClassクラスの実際のメソッドgetNextId()を呼び出しています。

ここで何が欠けていますか？私はMockitoとPowerMockitoが初めてなので、誰でも助けてくれますか。

javajunitmockitopowermock
73
user3942446

constructorが呼び出されるクラスを、構築中のクラスの代わりに@PrepareForTestアノテーションに入れる必要があります 新しいオブジェクトの模擬構成 。

あなたの場合：

✗@PrepareForTest(MyQueryClass.class)

✓@PrepareForTest(A.class)

より一般的な：

✗@PrepareForTest(NewInstanceClass.class)

✓@PrepareForTest(ClassThatCreatesTheNewInstance.class)

163
TrueDub

@TrueDubが受け入れられた応答で言及したように、コンストラクターが呼び出されるクラスを@PrepareForTestに追加する必要があります。

ただし、これを行うと、eclemmaおよびSonarによって報告されたそのクラスのカバレッジはそのクラスに対してゼロになります

Powermockito wiki

JavassistをByteBuddy（＃727）に置き換えますが、この古い問題の解決に役立つはずです。しかし、現在、JaCoCo On-the-fly計装でPowerMockを使用する方法はありません。 IDEでコードカバレッジを取得するための回避策はありません。

したがって、ここでの解決策は、実際のコードをリファクタリングして、そのクラスのインスタンスを返す静的ファクトリーを使用し、静的にモックすることです。

1
ACV