Question

对私有方法测试的意义有不同的看法，例如here和here。我个人认为这是有道理的，问题是如何正确地做到这一点。在C ++中，您可以使用#define hack或制作测试类friend，在C＃中有InternalsVisibleToAttribute，但在Java中，我们必须使用reflection或者使它们成为“可用于测试“和annotate them as such以便明确意图。两者的缺点应该很清楚。

我认为应该有更好的东西。从

开始

public class Something {
    private int internalSecret() {
        return 43;
    }
}

能够在测试代码中调用私有方法（如

）会很高兴

@MakeVisibleForTesting Something something = new Something();
Assert.assertEquals(43, something.internalSecret());

此处注释会使用反射以静默方式将所有调用转换为something的私有方法。我想知道Lombok是否可以做到（并会问作者）。

很有可能做那么多魔术证明太复杂了，无论如何都需要一些时间，所以我正在寻找一些替代方案。也许用@Decapsulate之类的东西注释正在测试的类，并使用注释处理器生成类似于Decapsulated_Something的类{/ 1}

public class Decapsulated_Something {
    public Decapsulated_Something(Something delegate) {
        this.delegate = delegate
    }
    public boolean internalSecret() {
        // call "delegate.internalSecret()" using reflection
    }
    ...
}

允许使用

Decapsulated_Something something = new Decapsulated_Something(new Something());
Assert.assertEquals(43, something.internalSecret());

我对注释处理没有太多经验，所以我先问一下：

实施起来有多复杂？
我忘记了什么？
您对此有何看法？

Answer 1

执行此实现似乎很麻烦。它可能不值得。而只是使方法包默认。

但是，如果您决定调用私有方法，则可以在Decapsulated_something类中使用setAccessible以允许通过反射进行调用。所以这很简单。

Answer 2

我将回答“一般”问题:-)只需要几行代码就可以通过反射访问方法，并且有很多库，工具，API等提供了这样做的方法。您可以在自己的代码中使用许多不同的技术。例如字节码操作，反射，类扩展等。但我倾向于保持简单。虽然测试私有方法可能很有用，但您也可能只想测试一些方法。因此，设计复杂的东西可能有点过头了。我只是使用已建立的API，或编写一个快速方法来访问我感兴趣的私有方法，并让它完成。

Answer 3

有很多方法可以采取

不要测试私有方法，因为它们是隐藏的实现细节，这些细节永远不会对调用者产生影响。
使方法包本地化，以便调用者无法访问它们，但您可以在同一个包中访问它们，即单元测试。
使单元测试内部类或提供包本地内部类。不确定这是一个改进！
使用反射来访问类的方法。这就像标记一个方法rpivate当它不是，并且是一个混乱恕我直言。你应该只是将一个私有的方法标记为私有。

Answer 4

几年前我参与了一个项目，该项目生成了类，以便更容易对私有方法进行单元测试。 http://java.net/projects/privateer/

它生成了额外的类，比调用反射更容易，例如如果您有MyClass.myPrivateMethod（），它将生成一个_MyClass类，允许直接调用myPrivateMethod。

它从来没有真正完成，并且在一些情况下有用，但总的来说，除非绝对必要，否则我不建议测试私有方法。通常将它们重新设计为实用程序类（如果您担心用户使用它们，可以使用包访问）是更好的选择。

Answer 5

能够在测试代码中调用私有方法（如
）会很高兴
@MakeVisibleForTesting Something something = new Something();
Assert.assertEquals(43, something.internalSecret());

有一种方法注释，请查看dp4j的@TestPrivates：

@Test
@TestPrivates 
//since the method is annotated with JUnit's @Test this annotation is redundant. 
// You just need to have dp4j on the classpath.
    public void somethingTest(){
      Something something = new Something();
      int sthSecret = something.internalSecret();
       Assert.assertEquals(43, sthSecret); //cannot use something.internalSecret() directly because of bug [dp4j-13][2] 
    }

用于测试私有方法的Java工具？

5 个答案: