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

Question

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

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

@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);
}

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

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

该测试立即通过，但实现却被搞砸了(“ channel ”硬编码)。

这里有两个问题:

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

• 如何应对这种情况:测试绿色，impl“硬编码”？我应该用不同的 channel 名称编写另一个测试吗？如果是这样，我应该在更正 impl 后将其删除(因为它变得毫无用处？)

更新:只是一些澄清。我在这里硬编码了“ channel ”

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

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

Answer 1

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

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