Question

我在使用mockito.spy方法遇到了麻烦。

我最近来到了一个＆＃34; old＆＃34;项目，我的第一个任务是在其中添加mockito，并进行真正的单元测试：）

该项目存在许多概念问题，但这不是重点;）

我解释了我的问题：

我有一个班级

public class Tutu{
  public Tutu(){
  }
}

public class Toto{
  public Toto(){
  }
  public int executeToto(Tutu tutu){
    //do some stuff
    return 5;
  }
}

public class Titi{
  private Toto toto;

  public Titi(){
     this.toto = new Toto();     
  }

  public void executeTiti(){
      //do some stuff
      Tutu tutu = new Tutu();
      int ret = this.toto.executeToto(tutu);
      //do some stuff
  }
}

在我的测试类TitiTest.java中我想只测试executeTiti，我不想测试executeToto的东西，因为这个类有自己的测试类TotoTest.java。

但正如你所看到的，toto是在titi构造函数中实例化的，所以我尝试这样的事情：（我也在我的测试中使用PowerMock，所以我使用PowerMockRunner，但它似乎不是问题）

@RunWith(PowerMockRunner.class)
public class TitiTest {

 @Test
 public void testExecuteTiti(){
   Toto toto = Mockito.spy(new Toto());
   Mockito.doReturn(2).when(toto).executeToto(Mockito.any(Tutu.class));

   Titi testedObject = new Titi();
   testedObject.executeTiti();
 }
}

但真正的方法总是在每次调用和ret = 5 :(

我错过了什么吗？我在stackoverflow上阅读了很多关于这个问题的文章并尝试了所有的解决方案，但它从来没有工作过，因为我觉得我做得对。

我使用junit4.11 / powermock1.5.4 / mockito1.9.5

Answer 1

Toto toto = Mockito.spy(new Toto());

请记住，您在此行中创建的Toto实例上的间谍/存根，而不是每个新创建的Toto。所以当你打电话时：

Titi testedObject = new Titi();
testedObject.executeTiti();

构造函数new Titi()本身会创建一个不受Mockito影响的Toto新实例，因此对this.toto.executeAction()的调用将始终返回5.

因为您使用PowerMockito运行，所以您可以选择stubbing Toto's constructor：

@RunWith(PowerMockRunner.class)
@PrepareForTest(Titi.class) // stub the calling class Titi, not Toto!
public class TitiTest {
  @Test public void testExecuteTiti() {
    Toto toto = Mockito.spy(new Toto());
    Mockito.doReturn(2).when(toto).executeToto(Mockito.any(Tutu.class));

    PowerMockito.whenNew(Toto.class).withAnyArguments().thenReturn(toto);

    Titi testedObject = new Titi();
    testedObject.executeTiti();
  }
}

但我最喜欢的选项是为Titi创建一个辅助构造函数，用于测试：

public Titi(){
  this.toto = new Toto();     
}

/** For testing only. Uses the passed Toto instance instead of a new one. */
Titi(Toto toto){
  this.toto = toto;
}

然后只需要你像这样调整你的测试：

@Test public void testExecuteTiti(){
  Toto toto = Mockito.spy(new Toto());
  Mockito.doReturn(2).when(toto).executeToto(Mockito.any(Tutu.class));

  Titi testedObject = new Titi(toto);
  testedObject.executeTiti();
}

Answer 2

你似乎缺少的是你的Toto类的间谍实际上从未被Titi类使用过。

我要做的是

1）重构Titi类以接受Toto作为构造函数中的依赖项。这样你就可以用任何Toto轻松创建Titi（并在你的单元测试中使用模拟）

2）如果选项1不可能，您可以执行以下操作：

public class Titi{
  private Toto toto;

  public Titi(){
     this.toto = new Toto();     
  }

  public void executeTiti(){
      //do some stuff
      Tutu tutu = new Tutu();
      int ret = getToto().executeToto(tutu);
      //do some stuff
  }

  //package private - used for overriding via spying 
  Toto getToto() {
      return toto;
  }
}

@RunWith(MockitoJUnitRunner.class)
public class TitiTest {

 @Test
 public void testExecuteTiti(){
   Toto toto = Mockito.mock(Toto.class);
   when(toto.executeToto(Mockito.any(Tutu.class)).thenReturn(2);

   Titi testedObject = new Titi();
   testedObject = spy(testedObject);
   doReturn(toto).when(testedObject).getToto();

   testedObject.executeTiti();
 }
}

Answer 3

这篇文章描述了使用单行方法或工厂辅助方法来测试不会注入协作者的类。 https://code.google.com/p/mockito/wiki/MockingObjectCreation

Mockito间谍方法不起作用

3 个答案: