java - 测试在 Java 中内部使用另一个私有(private)静态方法的私有(private)静态方法

Question

我有一个 A 类，如下所示:

public class A {

  // there is a public API that uses both
  // method1 and method2, but that's not important
  // in this question.
  public Integer publicApi(Integer a, Integer b) {
    // a bunch of other stuff is done that is 
    // not related to method1 or method2 here.
    // However, method1 IS used somewhere in here.
    // For now, I don't want to test other
    // implementation details for this API, I only
    // am interested in testing method1.

    return method1(a, b);
  }

  private static Integer method1(Integer a, Integer b) {
    if(method2(a, b, 10) == null) {
      return -1;
    }
    return 10 + method2(a, b, 10);
  }

  private static Integer method2(Integer a, Integer b, Integer c) {
    // some complicated logic with multiple values 
    // to return (based on various conditions).

    // however, I want to simply test what happens
    // when this method returns null (which is a possibility).
    return a+b+c;
  }
}

正如给出的那样，我的 A 类有一个公共(public) API，它是(有点)复杂的逻辑，我不想测试(出于这个问题的目的)。假设我只对测试 method1 感兴趣。

这是我的测试代码(我使用的是 PowerMockito):

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

import java.lang.reflect.Method;
import static org.junit.Assert.assertNull;
import static org.powermock.api.mockito.PowerMockito.spy;
import static org.powermock.api.mockito.PowerMockito.when;
import static org.powermock.api.support.membermodification.MemberMatcher.method;

@RunWith(PowerMockRunner.class)
@PrepareForTest(A.class)
public class SimpleTest {

  @Test
  public void testMethod1Invocation() throws Exception {

    spy(A.class);

    when(A.class, method(A.class,
        "method2", Integer.class,
        Integer.class, Integer.class))
        .withArguments(1, 2, 10)
        .thenReturn(null);

    // use reflection to call method1
    Method method1 = A.class.getDeclaredMethod(
        "method1",
        Integer.class, Integer.class
    );

    method1.setAccessible(true);
    Object ret = method1.invoke(1, 2);
    assertEquals(-1, ret);
  }
}

但是，我收到以下错误:

com.apple.geo.tart.A.method1(java.lang.Integer, java.lang.Integer, java.lang.Integer)
java.lang.NoSuchMethodException: com.apple.geo.tart.A.method1(java.lang.Integer, java.lang.Integer, java.lang.Integer)
    at java.lang.Class.getDeclaredMethod(Class.java:2130)
    at com.apple.geo.tart.SimpleTest.testMethod1Invocation(SimpleTest.java:32)

我看不出我做错了什么。在我看来，这里我需要三样东西:

1. 监视 A 以便我可以模拟 when/thenReturn 静态方法。
2. 模拟我感兴趣的私有(private)静态方法(在本例中为 method2)。
3. 使用反射访问方法1，然后调用它。

然后如何模拟一个私有(private)静态方法 (method2)，以便我可以测试依赖于第一个的同一类的另一个私有(private)静态方法 (method1)？

Answer 1

在我们讨论它的可测试性之前，我遇到了一些关于这段代码的问题:

• method2 实际上永远不会 返回null。如果它的任何输入是 null，您将得到一个 NullPointerException，因为 Java 试图拆箱 null。因此，您尝试测试的条件毫无意义。
• 隐藏这些方法确实没有任何值(value)。它们似乎进行业务级计算，因此具有测试值(value)。

对于您的初始案例，我会说这么多:我没有得到与您相同的错误。从我对 Java 8 的观察来看，您缺少 invoke 方法调用的参数，您需要通过该参数传递 method1 。

Object ret = method1.invoke(method1, 1, 2);

但是，我根本不推荐这样做。让我们把它分成几个测试。

首先，让我们放宽对这些方法的限制。它们可能是私有(private)的，因为我们不会在其他地方暴露它们的行为，但如果我们将其弱化为包私有(private)，则可能不会有任何危害。从这些方法中删除 private 标记。

其次，实际测试这些方法!这些方法可能会受到 null 的影响，您应该针对该场景进行测试。

第三，只有在您测试了其他两种方法后，您才应该回到您的publicApi 方法。此时，您所关心的只是使用正确的参数简单地调用了该方法。