使用 Moq 和 MSTest 测试异常的正确方法

Question

Moq 与 MsTest 的行为有点混淆。

编辑：这不是“我如何测试？”的问题。或“我如何断言？”，这是一个草稿本，可以查看 MoQ 是如何工作的，所以不要关注异常类型等。

我认为更好的问题可能是 =>“Moq Throws 的行为是否类似于 MsTest ExpectedExceptionAttribute？”也就是说，他们期望测试或 SUT 中出现异常？

我想知道 MoQ “Throws”在与 MsTest 一起使用时是如何工作的。不使用 MsTest 预期异常属性会更好吗？在测试中执行 try..catch 会更好吗？我还有一些关于这个的问题。

我正在模拟一个数据库调用，当发生错误时我想返回零 (0)。

TestMethod 直接使用 MsTest 异常属性，而 throws 异常使用 Moq。它仅在我在 SaveCart 方法中抛出异常时有效，而不是在我返回零时有效。

我想了解底层行为，因为感觉好像我不应该，也不想在 SaveCart 方法中引发异常。

这是有问题的测试：

[TestMethod]
[ExpectedException(typeof(ApplicationException))]
public void CartRepoSaveCartExceptionShouldReturnZero()
{
     _cartDatabaseMock.Setup(c => c.SaveCart(_cart))
                                   .Throws<ApplicationException>();

    var result = _cartRepository.SaveCart(_cart);

    Assert.AreEqual(result, _cartSaveExceptionValue);
}

这是基本的 SaveCart，它不会引发导致测试失败的异常：

public long SaveCart(Cart cart )
{
    long returnValue;

    try
    {
        returnValue = _cartDatabase.SaveCart(cart);
    }
    catch (Exception)
    {
        return 0;
    }
    return returnValue;
}

这是一个基本的 SaveCart，因为它引发了异常，所以测试可以在其中工作：

public long SaveCart(Cart cart )
{
    long returnValue;

    try
    {
        returnValue = _cartDatabase.SaveCart(cart);
    }
    catch (Exception)
    {
        throw new ApplicationException();
    }
    return returnValue;
}

如果问题解释不清楚，请随意建议一个更好的标题。

Answer 1

当被测单元抛出异常时，您应该使用ExpectedExceptionAttribute。

在您的第一个示例中，该方法没有引发任何异常，因此测试失败。

由于您的被测方法没有抛出任何异常，您根本不需要使用此属性...（只需验证此场景中的返回值）

当您想要验证是否引发了异常并且想要验证是否发生了一些额外的操作时，请使用以下模式：

[TestMethod]
[ExpectedException(typeof(<The specific exception>))]
public void FooTest()
{
    //arrange

    try
    {
       // act
    }
    catch(<the specific exception>)
    {
       // some asserts
       throw;
    }
}

如果出现以下情况，上述 sn-p 将失败：

1. 引发错误异常
2. 没有引发异常
3. 您的一个断言失败。

顺便说一句，由于您在方法中的捕获不是Exception 而不是ApplicationException，我建议您将设置更改为：

_cartDatabaseMock.Setup(c => c.SaveCart(_cart)).Throws<Exception>();

Answer 2

您是对的 - 第二个测试“SaveCart”有效，因为它引发了异常，而第一个测试失败，因为您转为 0。从您对先前答案的回复中，我相信您已经知道所有这些。如果您要询问第一次测试失败的行为......它是这样的：

1. SaveCart 被调用
2. 它返回一个异常（您的起订量设置的结果）
3. 您的 try catch 捕获了异常（您这样做是为了改变结果）
4. 您的 try catch 返回 0（结果现在是 0，因为您打算更改它）
5. Assert 根据 _cartSaveExceptionValue 检查您的结果
6. 您会收到一个失败测试，​​说明类似于“消息：Assert.AreEqual 失败。预期。实际。”

如果你想仔细检查这个...你可以试试下面的测试

1. 注释掉 [ExpectedException(typeof())]
2. 将 Assert.AreEqual(result, _cartSaveExceptionValue) 更改为 Assert.AreEqual(result, 0);
3. 测试应该通过，因为您将“结果”（又名 0）与 0 进行比较

我希望这能回答您的问题。

Answer 3

catch (Exception)
        {
            return 0;
        }

你不是在抛出异常，而是在吞下异常，那么你为什么会期待异常呢？它与MOQ 无关。您的测试和代码不同步。

顺便说一句，这是一个不好的做法，吞下异常。

catch (Exception)
        {
            throw new ApplicationException();
        }

这也是一种代码味道。您正在捕获各种异常，然后抛出不同的类型。