在方法级别隔离单元测试和存根内部方法调用是正确的吗？

Question

package com.company.domain;

public class Foo {
    private FooHelper helper; 

    public setHelper(FooHelper helper) {
        this.helper = helper;
    }

    public Double method1(Integer a, Integer b) {
        ... // logic here
        Integer var = method2(a);  // internal call to method2
        ... // more logic here
    }

    protected Integer method2(Integer c) {
        ... // logic here and return an Integer
    }
}

package com.company.domain;

@RunWith(MockitoJUnitRunner.class)    
public class FooTest {
    @Mock private FooHelper fooHelperMock; // mocking dependencies as usual
    @InjectMocks @Spy Foo foo; // this way we can stub internal calls

    @Test
    public void method1Test() {
        doReturn(2).when(foo).method2(1); //stub the internal method call
        ... // stub fooHelperMock method calls here

        Double result = foo.method1(1, 2);

        assertEquals(new Double(1.54), result);
        verify(foo, times(1)).method2(1);
        ... // verify fooHelperMock method calls here
    }

    @Test
    public void method2Test() {
        ... // stub fooHelper method calls here
        assertEquals(new Integer(5), foo.method2(3));
        ... // verify fooHelperMock method calls here
    }
}

Answer 1

通常，测试类的内部函数调用会使测试过于脆弱。当单元测试时，您只想看到类作为一个整体正在做它应该做的事情。你应该故意忽略内部。

忽略内部有很多有用的好处。主要是，如果您更改了方法--只要公共接口保持不变--您的测试仍然会通过。耶。这是您希望单元测试具有的稳定性类型。脆弱的单元测试几乎比没有单元测试更糟糕。一个脆弱的测试必须在每次代码更改时进行更改，即使一切都还在工作。这只为您提供额外的工作。

不过也有例外。在以下情况下，我会考虑取消内部函数调用：

总之，我强烈建议您不要在方法级别进行测试。类应该是一个独立的单元。如果这个班在工作，一切都很好。这种方法为您提供了安全所需的测试和更改事物所需的灵活性。

@raspacorp

实际上，如果您只测试一个用例，那么有人可以对var=2进行硬编码，然后通过您的测试用例。但是，您的工作是设计测试用例，以覆盖足够多的不同用例，从而合理地满足它的行为是适当的。