单元测试遗留 C# 代码

Question

如何为这样的方法编写 NUnit 测试。这个方法本身是否值得重构？ 处理遗留代码中的此类场景的最佳方法是什么？

         public bool DoXYZ()
            {
                ABC abc= new ABC()
                XYZ xyz = new XYZ();
                if (xyz .IsSomeCondition(Session.SessionID))
                { return false; }
                else
                { return abc.IsSomeOtherCondition(SessionID.SessionID); }
            }

Answer 1

您可能需要重构它以引入用于依赖项注入的钩子。例如，包含 DoXYZ 方法的类可以获得 ABC 和 XYZ 的新属性。这些属性可以默认为 ABC 和 XYZ 的实例，但在单元测试中可以替换为模拟版本。

如果您更喜欢使用 IoC，这种方法也支持

Answer 2

我肯定会重构以通过参数注入会话 ID - 否则您将必须手动创建会话。

可以做成静态的吗？看起来很像，特别是当您注入 sessionid 时。

另外，您正在实现一个（短）命令调度程序，与 IoC 相比，它通常被认为是一种反模式（请参见上面 Joel Martinez 的回答）。

Answer 3

您有两种选择：

• 重构代码并按照 Joel 和 Chris 的建议使用依赖项注入。这可能涉及大量工作，但它将使您的代码干净且易于测试。
• 按照此问题中的要求使用高级框架来模拟非注入依赖项。

处理大型遗留代码的方法可能是同时使用这两种方法。

您还可以查看 MSDN 杂志中的这篇文章。

Answer 4

考虑到问题中的混淆代码，我只能提供一些

• 我看到的两条路径的指针 - 因此最少需要 2 个单元测试。
• 在方法中创建协作者通常会很麻烦。将依赖项作为构造函数/方法参数传递。这允许您派生一个假值并操纵分支（例如，使 xyz.IsSomeCondition 对于此测试返回 false）。如果 ABC 和 XYZ 是易于设置的简单类，那么它可能不是一个迫在眉睫的问题..您现在可以不用提取参数重构。
• 将 sessionId 提取到参数中以消除静态（全局）变量访问，这些访问通常会保留其先前的值，从而导致测试之间的依赖关系

。

public bool DoXYZ(ABC abc, XYZ xyz, Guid sessionId) 
{    if (xyz.IsSomeCondition(sessionId))    
        return false; 

     return abc.IsSomeOtherCondition(sessionId);  
}