在过去的几年里，我一直认为在Java中，单元测试时广泛使用反射。由于其中一些需要检查的变量/方法是私有的，因此读取它们的值似乎是必要的。我一直认为反射API也是用于这个目的的。
上周，我不得不测试一些包并编写一些JUnit测试。像往常一样，我使用反射来访问私有字段和方法。但是我的主管审核代码时并不是很满意，并告诉我反射API不适用于这种“黑客”行为。相反，他建议修改生产代码中的可见性。
使用反射真的是不好的做法吗？我实在无法相信-
编辑：我应该提到，我被要求所有测试都在名为test的单独包中（因此使用受保护的可见性等并不是可行的解决方案）
我认为，反射（Reflection）应该只在最后情况下被使用，仅用于测试遗留代码或无法更改的API。如果你正在测试自己的代码，需要使用反射意味着你的设计不可测试，所以你应该修复它而不是使用反射。
如果你需要访问单元测试中的私有成员变量，通常意味着该类的接口不合适并且/或试图完成过多任务。因此，要么应该重新设计其接口，要么将某些代码提取到单独的类中，在那里这些有问题的方法/字段访问器可以被公开访问。
请注意，一般而言，使用反射会导致代码更难理解和维护，并且更容易出现错误。有些错误通常由编译器检测，但是使用反射会导致这些错误仅作为运行时异常出现。
更新：如@tackline所指出的，这仅涉及在自己的测试代码中使用反射，而不是测试框架的内部。JUnit（和可能所有其他类似的框架）使用反射来识别和调用测试方法-这是一种合理且局部化的使用反射。如果不使用反射，将很难或不可能提供相同的功能和便利。但是，它完全封装在框架实现中，因此不会使我们自己的测试代码变得复杂或受到影响。
仅出于测试目的而修改生产API的可见性是非常不好的行为。该可见性很可能由于合法原因而设置为其当前值，不应更改。
在单元测试中使用反射大多数情况下是可以的。当然，你应该设计可测试的类，以便需要较少的反射操作。
例如Spring框架有ReflectionTestUtils，但其目的是设置依赖项的模拟对象，在这些依赖项原本被Spring注入的情况下。
这个话题比“做与不做”更深刻，并关系到 何时 应进行测试 - 是否需要测试对象的内部状态；是否应该质疑正在测试的类的设计等等。
从测试驱动设计的角度来看，这是一种不好的实践。我知道您没有标记此为TDD，也没有特别要求，但TDD是一种良好的实践，而这种方法与其背道而驰。
在TDD中，我们使用测试来定义类的接口 - 公共接口 - 因此，我们编写的测试直接与公共接口交互。 我们关注该接口；适当的访问级别是设计和良好代码的重要组成部分。 如果您发现自己需要测试某些私有内容，则通常意味着存在设计问题。
public class ClassUnderTest {
protect void methodExposedForTesting() {}
}
在你的单元测试中
class ClassUnderTestTestable extends ClassUnderTest {
@Override public void methodExposedForTesting() { super.methodExposedForTesting() }
}
如果你有一个受保护的构造函数：
ClassUnderTest test = new ClassUnderTest(){};
我赞同Bozho的想法：
仅仅为了测试而修改生产API的可见性是非常糟糕的。
但如果你尝试着做可能最简单的事情，那么写反射API样板可能比较好，至少在你的代码还很新/变化时。我的意思是，每次更改方法名或参数时，手动更改测试中的反射调用的负担太重，而且方向错误。
我曾经陷入过为了测试而放松可访问性的陷阱，然后无意中从其他生产代码中访问了私有方法，因此我想到了dp4j.jar：它会注入反射API代码(Lombok-style)，这样你就不需要更改生产代码，也不需要自己编写反射API；dp4j在编译时将单元测试中的直接访问替换为等效的反射API。这里有一个使用JUnit的dp4j示例。
我认为你的代码应该通过两种方式进行测试。你应该通过单元测试测试你的公共方法，这将作为我们的黑盒测试。由于你的代码被分解成可管理的函数（良好的设计），你需要使用反射来单元测试各个部分，以确保它们独立于进程工作，我能想到的唯一方法是使用反射，因为它们是私有的。
至少，在单元测试过程中这是我的想法。
除了其他人已经说过的，还要考虑以下几点：
//in your JUnit code...
public void testSquare()
{
Class classToTest = SomeApplicationClass.class;
Method privateMethod = classToTest.getMethod("computeSquare", new Class[]{Integer.class});
String result = (String)privateMethod.invoke(new Integer(3));
assertEquals("9", result);
}
public void testSquare()
{
String result = new NumberUtils().computeSquare(new Integer(3));
assertEquals("9", result);
}