15

监视正在单元测试的对象是否是代码异味？例如，假设我有一个LineCounter类，其工作是简单地计算字符串中的行数。--

class LineCounter {
    public int getNumLines(String string) {
        String metadata = getStringMetadata(string);

        // count lines in file
        return numLines;
    }

    /** Expensive operation */
    protected String getStringMetadata(String string) {
        // do stuff with string
    }
}

现在我想为此编写一个 JUnit 4 测试来测试该getNumLines方法，同时模拟昂贵的getStringMetadata调用。我决定使用 Mockito 的间谍机制来getStringMetadata返回一个虚拟值。

class LineCounterTests {
    @Test public void testGetNumLines() {
        LineCounter lineCounterSpy = Mockito.spy(new LineCounter());

        // Mock out expensive call to return dummy value.            
        Mockito.when(lineCounterSpy.getStringMetadata(Mockito.anyString()).thenReturn("foo");

        assertEquals(2, lineCounterSpy.getNumLines("hello\nworld");
    }
}

这是合理的做法吗？我觉得测试一个 Spy 对象而不是实际的类很奇怪，但我真的想不出反对它的理由。

4

2 回答 2

4

我将分两部分回答这个问题。首先，是的，模拟或监视被测类是代码异味。这并不意味着它不能正确完成，而是它容易发生风险，应尽可能避免。

WRT您的具体示例，我会看到如何正确使用间谍，但这将基于您在其他地方进行了完全单元测试的断言getStringMetadata。这就引出了一个问题，如果您在getStringMetadata其他地方进行了全面的单元测试，那么您必须知道如何测试它，因此为什么不在getNumLines没有间谍的情况下进行测试。

话虽这么说，millhouse但无论哪种方式，您都必须在某处对昂贵的代码进行单元测试。他的建议有助于隔离昂贵的代码并确保您只需测试/练习一次。

于 2013-10-09T10:34:54.353 回答
2

在这种情况下，存根由被测方法调用的方法是完全合法的。这甚至是我能想到的单独测试它的唯一方法。您只是不想为了测试的唯一目的将单个方法提取到它自己的类中。

不过要注意存根方法的副作用。存根返回值可能还不够，如果存根方法有副作用，那么您也必须存根副作用。在副作用非常复杂的某些情况下，它甚至可能是反对它的理由，但这很可能表明在测试类本身的实现中存在代码异味。

要回答你的问题，我觉得很容易找到它的理由，但很难找到反对它的理由。这是我每天都使用的技术，它帮助我将我的实现拆分为小方法，这些方法在完全隔离的情况下单独测试，而且我还没有看到它有任何限制。

于 2016-03-15T07:52:34.233 回答