3

在编写单元测试时,我最挣扎的一件事是我要测试什么,不测试什么。

所以给定下面的代码,我应该写什么测试:

public static class Enforce
{
    public static T ArgumentNotNull<T>(T parameter, string parameterName) where T : class
    {
        if (parameterName.IsNullOrWhiteSpace())
            throw new ArgumentNullException("parameterName");

        if (parameter.IsNull())
            throw new ArgumentNullException(parameterName);

        return parameter;
    }

    public static string ArgumentNotNullOrEmpty(string parameter, string parameterName)
    {
        ArgumentNotNull(parameter, parameterName);

        if (parameter.IsNullOrEmpty())
            throw new ArgumentException(parameterName);

        return parameter;
    }

    public static string ArgumentNotNullOrWhiteSpace(string parameter, string parameterName)
    {
        ArgumentNotNull(parameter, parameterName);

        if (parameter.IsNullOrWhiteSpace())
            throw new ArgumentException(parameterName);

        return parameter;
    }

    public static T NotNull<T>(T instance) where T : class
    {
        instance.IfNullThrow<T, NullReferenceException>(String.Format(EnforceResources.TypeNotNull, typeof(T).FullName));

        return instance;
    }
}

}

我应该为ArgumentNotNull<T>测试抛出的异常编写一个测试,然后再编写一个测试未抛出的异常吗?我想我的问题是,我是否应该编写测试预期的测试然后完全相反?

    [Fact]
    public void ShouldThrowArgumentNullException()
    {
        object arg = null;

        Assert.Throws<ArgumentNullException>(() => { Enforce.ArgumentNotNull(arg, "arg"); });
    }

    [Fact]
    public void ShouldNotThrowArgumentNullException()
    {
        var arg = "Test";

        Assert.DoesNotThrow(() => { Enforce.ArgumentNotNull(arg, "arg"); });
    }
4

2 回答 2

1

您应该尽最大努力测试您的所有代码。这意味着所有公共方法和其中的场景都可以尽可能多地覆盖其所有内部逻辑。

所以你应该编写 3 个测试:一个用于第一个异常,一个用于第二个异常,一个不抛出任何异常。

关于这些测试的命名约定,这可能很有趣:https ://stackoverflow.com/a/1594049/1373170

于 2013-03-17T07:37:05.770 回答
1

单元测试的概念是确保您的代码完全按照您的预期工作,

因此,如果您希望它抛出异常,则应测试它是否存在正确的异常。

以下是有关编写单元测试的一些基本概念和技巧:

http://www.slideshare.net/homespothq/unit-testing-concepts-and-best-practices

于 2013-03-17T07:42:27.103 回答