BDD 方式的 JUnit 测试应该如何集中

Question

我正在测试一些代码进行练习，并发现了奇怪的情况。

有一个 ChannelRegistry 包含所有通信通道引用，以及 PrimaryConsumer 需要将其自身附加到在调用初始化（）时在运行时选择的通道之一。 所以我完成了第一个测试，如下：

@RunWith(MockitoJUnitRunner.class)
public class PrimaryConsumerTest {

    private @Mock ChannelsRegistry communicationRegistry;
    private PrimaryConsumer consumer;

    @Before
    public void setup() {
        consumer = new PrimaryConsumer(communicationRegistry);
    }

    @Test
    public void shouldAttachToChannel() throws Exception {
        consumer.initialize();
        verify(communicationRegistry).attachToChannel("channel", consumer);
    }
} 

我正在检查是否调用了附加方法。为了让它变成绿色，我像这样放置 impl：

public void initialize() {
    communicationRegistry.attachToChannel("channel", this);
}

现在下一个测试：按名称获取通道 id 并附加到此特定通道。我希望我的测试描述类的行为而不是其内部结构，所以我不希望我的测试是“shouldGetSpecificChannel”。相反，我检查它是否可以附加到运行时选择的通道：

    @Test
    public void shouldAttachToSpecificChannel() throws Exception {
        String channelName = "channel";
        when(communicationRegistry.getChannel("channel_name")).thenReturn(channelName);
        consumer.initialize();
        verify(communicationRegistry).attachToChannel(channelName, consumer);
    }

该测试立即通过，但实现被搞砸了（“通道”硬编码）。

这里有 2 个问题：

• 可以对这种行为进行 2 次测试吗？也许我应该在第一次测试中立即获取通道？如果是这样，它如何映射到在单个测试中测试单个事物？

• 如何应对这种情况：测试绿色，impl“硬编码”？我应该用不同的频道名称编写另一个测试吗？如果是这样，我应该在更正 impl 后删除它（因为它变得无用？）

更新： 只是一些澄清。 我在这里对“通道”进行了硬编码，

public void initialize() {
    communicationRegistry.attachToChannel("channel", this);
}

只是为了使第一个测试快速通过。但是，当运行第二次测试时，它立即通过。我不验证是否调用了存根方法，因为我认为不应显式验证存根。 罗德尼说测试是多余的，这就是你的意思吗？如果是，我应该在第一次测试的一开始就制作存根吗？

Answer 1

1. 更多的测试通常比太少的测试更好，所以两次测试就可以了。更好的问题是这两个测试是否多余：是否存在任何情况或输入组合会使其中一个测试失败，但另一个测试不会失败？那么这两个测试都是需要的。如果它们总是一起失败或成功，那么您可能只需要其中之一。

2. 什么时候您需要为 channelName 使用不同的值？听起来这是一个与这些特定测试无关的配置设置。没关系，也许您会在集成测试中在更高级别上测试该配置。我更关心的是为什么它首先是硬编码的：它应该被注入到你的类中（可能通过构造函数）。然后您可以测试不同的频道名称——也可以不测试。无论哪种方式，您都不想仅仅为了测试而更改代码（如果这意味着在完成后将其更改回来）。

Answer 2

对于多次测试的问题，基本上与罗德尼相同。根据您的更新，我建议一两件事。

首先，您在两个测试中使用了相同的数据。 Kent Beck 在关于 TDD 的书中提到了“三角测量”的使用。如果您在第二种情况下使用了不同的参考数据，那么如果您不做任何额外的工作，您的代码将无法通过。

另一方面，他还提到消除所有重复，而重复包括代码和测试之间的重复。在这种情况下，您可以将两个测试保持原样，并通过替换被测类中的文字来重构代码中的字符串 "channel" 与测试中的相同字符串之间的重复调用您的 communicationRegistry.getChannel()。经过这次重构之后，您现在将字符串文字放在一个且只有一个位置：测试。

两种不同的方法，相同的结果。您使用哪一种取决于个人喜好。在这种情况下，我会采取第二种方法，但这只是我的情况。

提醒查看罗德尼对多次测试问题的回答。我猜你可以删除第一个。

谢谢！

布兰登