我如何获得这些代码示例的完整OpenCover分支覆盖?

时间:2011-11-24 09:37:01

标签: unit-testing nunit code-coverage opencover

我想测试以下两个(不相关的)方法,并使用OpenCover 2.0实现完整的分支和语句覆盖。

public class Methods
{
    public static void MethodWithDelegate(SynchronizationContext context)
    {
        context.Send(delegate { Console.Beep(); }, null);
    }

    public static string MethodWithSwitchStatement(Type value)
    {
        string output = string.Empty;

        if (value != null)
        {
            switch (value.ToString())
            {
                case "System.Int32":
                    output = "int";
                    break;
                default:
                    output = "other type";
                    break;
            }
        }

        return output;
    }
}

我编写了以下(NUnit)测试,其中一个使用'Moq'模拟对象:

[Test]
public void Test_ShouldCoverMethodWithDelegate()
{
    var context = new Mock<SynchronizationContext>();

    Methods.MethodWithDelegate(context.Object);

    context.Verify(x => x.Send(It.IsAny<SendOrPostCallback>(), It.IsAny<object>()));
}

[Test]
public void Test_ShouldCoverSwitchStatement()
{
    Assert.That(Methods.MethodWithSwitchStatement(null), Is.EqualTo(string.Empty));
    Assert.That(Methods.MethodWithSwitchStatement(typeof(int)), Is.EqualTo("int"));
    Assert.That(Methods.MethodWithSwitchStatement(typeof(float)), Is.EqualTo("other type"));
}

但是,在通过OpenCover运行测试后,coverage.xml文件始终包含两个测试的访问次数为零的分支点。序列覆盖率显示为100%。

不是IL专家,我不确定如何编写进一步的测试以使分支覆盖率达到100%。

1 个答案:

答案 0 :(得分:2)

好的,首先让我们看一下IL中的第一个方法(我正在使用IL SPY

.method public hidebysig static 
void MethodWithDelegate (
    class [mscorlib]System.Threading.SynchronizationContext context
) cil managed 
{
// Method begins at RVA 0x2059
// Code size 41 (0x29)
.maxstack 8

IL_0000: nop
IL_0001: ldarg.0
IL_0002: ldsfld class [mscorlib]System.Threading.SendOrPostCallback so8254847.Methods::'CS$<>9__CachedAnonymousMethodDelegate1'
IL_0007: brtrue.s IL_001c

IL_0009: ldnull
IL_000a: ldftn void so8254847.Methods::'<MethodWithDelegate>b__0'(object)
IL_0010: newobj instance void [mscorlib]System.Threading.SendOrPostCallback::.ctor(object, native int)
IL_0015: stsfld class [mscorlib]System.Threading.SendOrPostCallback so8254847.Methods::'CS$<>9__CachedAnonymousMethodDelegate1'
IL_001a: br.s IL_001c

IL_001c: ldsfld class [mscorlib]System.Threading.SendOrPostCallback so8254847.Methods::'CS$<>9__CachedAnonymousMethodDelegate1'
IL_0021: ldnull
IL_0022: callvirt instance void [mscorlib]System.Threading.SynchronizationContext::Send(class [mscorlib]System.Threading.SendOrPostCallback, object)
IL_0027: nop
IL_0028: ret
} // end of method Methods::MethodWithDelegate

正如您所看到的,IL_0007上有一个条件分支,只有在设置了缓存的匿名委托时才会执行,否则它会通过设置委托的主代码然后调用它。

解决方案:运行测试两次 - 或者忘记它,因为它有点像.NET优化

现在针对第二个问题,这次最好看看C#中实际生成了什么 - 你编写了switch语句,但是编译器使用了ifs而不是

public static string MethodWithSwitchStatement(Type value)
{
    string output = string.Empty;
    if (value != null)
    {
        string a;
        if ((a = value.ToString()) != null && a == "System.Int32")
        {
            output = "int";
        }
        else
        {
            output = "other type";
        }
    }
    return output;
}

正如您所看到的,编译器已经使用switch语句引入了if null测试但是因为您已经有了这个测试,所以它永远不会被执行。

解决方案:删除初始if null测试(因为不需要)。