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

Question

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

4

我有一个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)

我不明白我做错了什么，据我所知，这里需要3件事:

监听 A ，以便可以模拟静态方法的 when/thenReturn。
模拟我想要的私有静态方法（在本例中是 method2）。
使用反射访问 method1，然后调用它。

如何模拟一个私有的静态方法(method2)，以便可以测试另一个相同类别的私有静态方法(method1),该方法依赖于第一个方法?

- Nishant Kelkar

5

测试私有方法是绝对必要的吗？一般来说，您只需要为公共方法编写测试。您可以使用“verify”来查看是否在私有方法中调用了任何模拟对象。 - user2004685

异常提示说你正在寻找有3个参数的 method1 方法，但是你的代码只传入了 2 个参数。尝试重新编译你的项目 -- 看起来代码没有引用测试中的二进制文件。 - Sergey Grinev

@SergeyGrinev 我确实尝试过这样做，清除了构建目录，并重新使用Gradle组装了代码。但我仍然得到相同的错误。 - Nishant Kelkar

@所有人，问题已经找到。问题出在method1.invoke(...)上。它所需的第一个参数是要调用方法的对象，而不是实际的参数！当测试类的静态方法时，通常为null。将该方法调用更改为method1.invoke(null, 1, 2)即可得到预期结果。 - Nishant Kelkar

1个回答

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- Makoto · Accepted Answer

在讨论其可测试性之前，我对这段代码有一些问题：

method2 实际上永远不会返回null。如果其中任何输入为null，则由于Java尝试取消装箱null，您将获得NullPointerException。因此，您试图测试的条件是相当无意义的。
隐藏这些方法真的没有任何价值。它们似乎做了业务层面的计算，因此有被测试的价值。

至于您最初的情况，我只能说：我并没有遇到与你相同的错误。根据我在Java 8中所看到的，您在invoke方法调用中缺少一个参数，您需要传递 method1。

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

然而，我完全不建议这样做。 让我们将其分解成几个测试。

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

第二，实际测试这些方法！ 这些方法可以受到null的影响，您应该针对该场景进行测试。

第三，在您测试了其他两个方法之后，才应返回到您的publicApi方法。此时，您只关心该方法是否仅以正确的参数调用。