在Nunit,C#中验证ArgumentException及其消息

时间:2013-07-23 16:51:17

标签: c# exception nunit assert

在我在Nunit的测试程序中,我想通过验证消息来验证它是否正在获得写入Argument Exception。

    [Test]
    public void ArgumentsWorkbookNameException()
    {
        const string workbookName = "Tester.xls";
        var args = new[] { workbookName, "Sheet1", "Source3.csv", "Sheet2", "Source4.csv" };
        Assert.Throws(typeof(ArgumentException), delegate { var appargs = new ApplicationArguments(args); }, "Invalid ending parameter of the workbook. Please use .xlsx");

    }

在对此进行测试后,当我在主程序中修改消息时,这不起作用。

        int wbLength = args[0].Length;

        // Telling the user to type in the correct workbook name file.
        if (args[0].Substring(wbLength-5,5)!=".xlsx")
        {
            throw new ArgumentException(
                "Invalid ending parameter of the workbook. Please use .xlsx random random");
        }

单位测试仍然通过,无论我是否更改了消息。

我该怎么办?或者C#中没有这样的东西。我的同事说Ruby和RSPEC都有类似的选项,但他对C#并不是100%肯定。

4 个答案:

答案 0 :(得分:47)

使用fluent界面创建断言:

Assert.That(() => new ApplicationArguments(args), 
    Throws.TypeOf<ArgumentException>()
        .With.Message.EqualTo("Invalid ending parameter of the workbook. Please use .xlsx random random"));

答案 1 :(得分:21)

我同意Jon的观点,“这种测试不必要地变脆”。但是,至少有两种方法可以检查异常消息:

1:Assert.Throws返回一个异常,因此你可以为它的消息做一个断言:

var exception = Assert.Throws<ArgumentException>(() => new ApplicationArguments(args));
Assert.AreEqual("Invalid ending parameter of the workbook. Please use .xlsx random random", exception.Message);

2:您还可以使用ExpectedException属性。但是,请注意,属性在整个测试代码中等待异常,而不仅仅是在抛出异常的代码中除外。因此,不建议使用此属性。

[Test]
[ExpectedException(typeof(ArgumentException), ExpectedMessage = "Invalid ending parameter of the workbook. Please use .xlsx random random")]
public void ArgumentsWorkbookNameException()
{
    const string workbookName = "Tester.xls";
    var args = new[] { workbookName, "Sheet1", "Source3.csv", "Sheet2", "Source4.csv" };
    new ApplicationArguments(args);
}

答案 2 :(得分:4)

Assert.Throws中的message参数不是预期的异常消息;如果测试失败,则包含断言失败的错误消息。

我不相信NUnit支持开箱即用的异常消息测试,我认为这样的测试无论如何都是不必要的。如果真的想要编写自己的帮助方法,你可以这样做,但我个人不会鼓励它。 (我很少指定测试失败消息,除非它包含一些诊断信息。如果测试失败,我将继续查看测试,因此消息不会增加太多。)

鼓励你使用泛型重载,而简单地使用lambda表达式:

Assert.Throws<ArgumentException>(() => new ApplicationArguments(args));

(顺便说一下,如果这是你的实际代码,还有其他问题 - 尝试传入new[] { "xyz" }作为参数......)

答案 3 :(得分:0)

在 .NET Core 3.1 MSTest 项目中,我就是这样做的。

[TestMethod]
public async Task SaveItemAsync_NameIsNull_ThrowsException()
{
    var item = new Item
    {
        Name = null
    };

    var result = await Assert.ThrowsExceptionAsync<ArgumentException>(() => _service.SaveItemAsync(item));
    Assert.AreEqual("The item's name must be set.", result.Message);
}