代码之家 › 专栏 › 技术社区 › jpoh

如何对重载函数进行单元测试?

overloading unit-testing

8

jpoh · 技术社区 · 17 年前

public class MyClass
{

    DatabaseDependency _depend;

    public MyClass(DatabaseDependency depend)
    {
        _depend = depend;
    }

    public string DoSomething(DBParameter database)
    {
        var result = _depend.GetResults(database, ...);
        string response = String.Empty;        

        // some logic to process the result 

        return response;
    }

}

现在,我想向DoSomething添加一个重载,以便它接受连接字符串而不是DBParameter参数(假设DatabaseDependency已经有一个接受连接字符串的GetResults重载)。

我的问题:我有几个单元测试,它们描述了用于处理DatabaseDependency.GetResults结果的各种逻辑路径。当我将重载添加到DoSomething中时,我基本上会重构代码,以便这两个重载都重用该逻辑(即,可能将其移动到私有方法)。进行单元测试的正确方法是什么?我是否需要同样多的单元测试来验证我正在添加的新重载的所有逻辑路径?

4 回复 | 直到 17 年前

1

7

Mark Seemann 17 年前

如果您的代码保持当前的外观,那么是的:您还需要对该方法进行单元测试,基本上重复您的测试工作。

但是,如果实现功能使一个方法简单地调用另一个方法是有意义的,那么可以将另一个方法设置为虚拟的。这将允许您创建一个特定于测试的子类,在该子类中,您只需验证虚拟方法是否由具有正确值的另一个方法调用。

经过证实的 方法正确调用另一个方法,并且您可以将测试工作集中在该方法上。

2

6

Brian Agnew 17 年前

如果您确信接受字符串的重载方法只是转换为连接对象,然后委托给原始对象,那么您应该再添加一个测试方法。

但是,如果重构底层重载方法,使其不发生委托,则会出现这种情况。在这个场景中,我会更加自信地复制这两种方法的所有测试。

我认为第一条路线是最务实的。不过,偶尔运行一次代码覆盖率分析是一个好主意,这将在以后指示是否需要更多的测试。

3

1

djna 17 年前

是的,将公共处理重构为私有方法——我假设不管测试考虑如何,您都会这样做,重复代码是不好的。有趣的是,思考测试如何引导我们做正确的事情。

然后对每个重叠的启动路径进行两个简单的测试。

4

0

ptomli 17 年前

取决于您是在进行黑盒测试还是白盒测试,以及您的应用程序是否同时使用这两个版本的方法。

如果您的应用程序只使用一种方法,则弃用另一种方法,并创建一个版本条件测试,当弃用的方法仍然存在于某个预先确定的版本时,该测试将失败。IE:强制在某个时候删除不推荐的方法。