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类有一个公共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)

我不知道我在这里做错了什么。在我看来，我需要三件事：

对A进行间谍活动，以便我可以模拟/然后返回静态方法。
模拟我感兴趣的私有静态方法（在这种情况下为method2）。
使用反射访问method1，然后调用它。

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

Answer 1

我在讨论之前的代码时遇到了一些问题我们谈论它的可测试性：

method2 从不实际返回null。如果其中任何输入为null，则由于Java尝试取消装箱NullPointerException，您将获得null。因此，您尝试测试的条件毫无意义。

隐藏这些方法真的没有价值。它们似乎进行业务级别的计算，因此具有测试价值。

我会在最初的情况下说这么多：我没有得到与你相同的错误。从我在Java 8中看到的内容，您错过了invoke方法调用的参数，您需要通过该参数调用method1。

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

然而，我根本不建议这样做。让我们把它分成几个测试。

首先，让我们放松对这些方法的限制。他们可能是私人的，因为我们不会在其他地方暴露他们的行为，但如果我们将其削弱到包装私有，那么可能没有任何伤害。从这些方法中删除private标记。

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

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

测试私有静态方法，该方法在Java内部使用另一个私有静态方法

1 个答案: