java - 进行单元测试时来自 Mockito 的 MissingMethodInitationException

Question

我有一个类，其中包含一个公共(public)静态方法getProduct(String name):

public class ProductManager {
  public static Product getProduct(String name) {
        Product prod = findProduct(name);
        return prod;
  }
}

另一个CustomerService类使用上面的ProductManager:

public class CustomerService {
   ...
   public void handleProduct() {
     Product appleProd = ProductManager.getProduct("apple");
     doHandle(appleProd);
   }
}

我在 CustomerService 类中对 handleProduct() 方法进行单元测试。我用mockito模拟测试中的 ProductManager.getProduct("apple") 部分:

public class CustomerServiceTest {
  @Test
  public void testHandleProduct() {
    Product mockProd = Mockito.mock(Product.class);

    // MissingMethodInvocationException
    when(ProductManager.getProduct("apple")).thenReturn(mockProd);
    ...
  }
}

但是，当我运行测试时，我从 Mockito 得到了 MissingMethodInitationException:

org.mockito.exceptions.misusing.MissingMethodInvocationException: 
when() requires an argument which has to be 'a method call on a mock'.
For example:
    when(mock.getArticles()).thenReturn(articles);

Also, this error might show up because:
1. you stub either of: final/private/equals()/hashCode() methods.
   Those methods *cannot* be stubbed/verified.
   Mocking methods declared on non-public parent classes is not supported.
2. inside when() you don't call method on mock but on some other object.

它提示在 when() 内部我没有调用方法，但我确实调用了 public static 方法 ProductManager.getProduct("apple") in when(...)，为什么 Mockito 向我提出这个错误？我不明白。

Answer 1

Mockito 无法模拟静态方法。使它成为一个实例方法，您的代码就可以工作。

还有其他框架允许这样做(例如 Powermock)，但是这是相当糟糕的做法，也是糟糕设计的标志。您应该创建一个实例并进行依赖项注入(inject)。如果一个方法很小，可以在测试其他类时间接测试它(例如 Math.max())，则不需要模拟。

在您发布的代码中，您有 getProduct()，但在堆栈跟踪中它是 getArticles() - 我认为该代码只是一个简化的示例，而堆栈跟踪是实际的。

这里有几篇文章解释了测试/模拟静态方法的问题:

• https://softwareengineering.stackexchange.com/questions/231594/mocking-static-methods
• http://misko.hevery.com/2008/12/15/static-methods-are-death-to-testability/
• http://blog.christianposta.com/testing/java-static-methods-can-be-a-code-smell/
• Why does Mockito not mock static methods?