主持人注:这里已经有39个答案了(有些已经删除了)。在你发表你的答案之前，考虑一下你是否可以为讨论添加一些有意义的东西。你很可能只是在重复别人已经说过的话。
我偶尔发现自己需要将类中的私有方法设为public，只是为了为它编写一些单元测试。
通常这是因为该方法包含类中其他方法之间共享的逻辑，并且单独测试逻辑更整洁，或者另一个原因可能是我想测试同步线程中使用的逻辑，而不必担心线程问题。
其他人发现他们这样做是因为我不喜欢吗?我个人认为，公开一个方法的好处超过了它在类之外没有提供任何服务的问题……
更新
谢谢大家的回答，似乎引起了大家的兴趣。我认为普遍的共识是测试应该通过公共API进行，因为这是使用类的唯一方式，我非常同意这一点。在我上面提到的几个案例中，我会这样做，这是不常见的情况，我认为这样做的好处是值得的。
然而，我可以看到，每个人都指出它不应该真的发生。再仔细想想，我觉得改变你的代码来适应测试是一个坏主意——毕竟我认为测试在某种程度上是一个支持工具，而改变一个系统来“支持一个支持工具”是明显的坏做法。
当前回答
在您的更新中，您说使用公共API进行测试很好。 这里实际上有两所学校。
黑盒测试 黑盒学派认为，类应该被视为一个黑盒，没有人可以看到其中的实现。测试这一点的唯一方法是通过公共API——就像类的用户将使用它一样。 白盒测试。 白盒学派认为自然地使用关于类实现的知识，然后测试类以了解它应该工作。
在这次讨论中我真的不能支持任何一方。我只是想知道有两种不同的方法来测试一个类(或库或其他)会很有趣。
其他回答
注意: 这个答案最初是针对以下问题发布的:单独的单元测试是否是通过getter公开私有实例变量的好理由?这是合并到这个，所以它可能是一个特定于这里提出的用例。
一般来说，我通常都支持重构“生产”代码，以使其更容易测试。然而，我不认为这是一个好的决定。一个好的单元测试(通常)不应该关心类的实现细节，只关心它的可见行为。与其将内部堆栈暴露给测试，不如测试类在调用first()或last()后是否按照预期的顺序返回页面。
例如，考虑以下伪代码:
public class NavigationTest {
private Navigation nav;
@Before
public void setUp() {
// Set up nav so the order is page1->page2->page3 and
// we've moved back to page2
nav = ...;
}
@Test
public void testFirst() {
nav.first();
assertEquals("page1", nav.getPage());
nav.next();
assertEquals("page2", nav.getPage());
nav.next();
assertEquals("page3", nav.getPage());
}
@Test
public void testLast() {
nav.last();
assertEquals("page3", nav.getPage());
nav.previous();
assertEquals("page2", nav.getPage());
nav.previous();
assertEquals("page1", nav.getPage());
}
}
许多答案建议只测试公共接口，但恕我直言，这是不现实的——如果一个方法执行的操作需要5个步骤，那么您将希望分别测试这5个步骤，而不是一起测试。这需要测试所有五个方法，否则它们(除了测试之外)可能是私有的。
测试“私有”方法的通常方法是给每个类一个自己的接口，并将“私有”方法设为公共的，但不将它们包含在接口中。这样，它们仍然可以被测试，但不会使界面膨胀。
是的，这将导致文件和类膨胀。
是的，这确实使公共和私有说明符变得多余。
是啊，这真让人头疼。
不幸的是，这是我们为使代码可测试而做出的众多牺牲之一。也许未来的语言(或者甚至是c# /Java的未来版本)将具有使类和模块可测试性更加方便的特性;但与此同时，我们得跳过这些障碍。
There are some who would argue that each of those steps should be its own class, but I disagree - if they all share state, there is no reason to create five separate classes where five methods would do. Even worse, this results in file- and class-bloat. Plus, it infects the public API of your module - all those classes must necessarily be public if you want to test them from another module (either that, or include the test code in the same module, which means shipping the test code with your product).
在java中，还可以选择将其包设置为private(即去掉可见性修饰符)。如果您的单元测试与被测试的类在同一个包中，那么它应该能够看到这些方法，并且比将方法完全公开要安全一些。
我通常将这些方法保留为受保护的，并将单元测试放在相同的包中(但在另一个项目或源文件夹中)，在那里它们可以访问所有受保护的方法，因为类装入器将把它们放在相同的名称空间中。
就我个人而言，我宁愿使用公共API进行单元测试，我当然不会为了方便测试而将私有方法设为公共。
如果您真的想单独测试私有方法，那么在Java中可以使用Easymock / Powermock来实现这一点。
你必须务实，你也应该意识到为什么事情很难测试的原因。
“倾听测试”——如果测试很困难，这是否能告诉你关于你的设计的一些东西?你能不能重构到这样一个地方，对这个方法的测试将是微不足道的，并且很容易通过公共api进行测试?
以下是Michael Feathers在《有效使用遗留代码》一书中所说的话
“很多人花了很多时间试图弄清楚如何解决这个问题……真正的答案是，如果你想测试一个私有方法，这个方法不应该是私有的;如果公开方法让您感到困扰，很可能是因为它是单独责任的一部分;它应该上另一门课。”[有效地使用遗留代码(2005)，作者:M. Feathers]
