确定高级方法的单元测试代码覆盖率

Question

When I have a method like below,

double Process(double payable)
{
    var step1Result = Step1(payable);
    var step2Result = Step2(step1Result);
    return Step3(step2Result);
}

is it required to have a unit test for Process() when I already have unit tests for the inner three public methods? If the answer is not required, then does it mean that the Process() method has to be maintained through manual code review?

Answer 1

If Process is public then Yes, you will need unit tests for Process. If someone is fixing a bug and forgets to call Step2, Process will not work anymore. If you have written code, it has to be tested.

If it is not public, then as long as it is being covered via some other method calling it, for example, another public method then you should be fine. Basically if the Process method is being used and you put a breakpoint and run all your tests and it does not hit the breakpoint, then that means it is not covered and tested. Of course you can use visual studio's test coverage to indicate if it is covered instead of using a breakpoint.