期望测试中出现异常，但无论如何都想验证调用了 dispose

Question

我正在对一个演示应用程序进行单元测试，它是一个 POP3 客户端。 POP3 客户端实现了IDisposable，因此我尝试测试using 循环。

（我使用的是 nunit 2.5.2 和 Moq 4.0）

/// <summary>
/// Unsuccessful construct dispose cycle, no ILogger object given. 
/// Expecting ArgumentNullException. Expecting TcpClient dispose to be called.
/// </summary>
[Test]
[ExpectedException(typeof(ArgumentNullException))]
public void ConstructorDispose_LoggerNull_ArgumentNullException()
{
    mockTcpClient.Setup(x => x.Dispose());
    using (var popClient = new PopClient(null, mockTcpClient.Object))
    {

    }
    mockTcpClient.VerifyAll();
}

正如您所看到的，verifyAll 方法永远不会被调用，但是测试将会成功。那么...

• 解决这个问题的最佳方法是什么？
• 还有其他方法尝试捕获吗？

更新我暂时这样修复了它：

        mockTcpClient.Setup(x => x.Dispose());
        var correctExceptionThrown = false;
        try
        {
            using (var popClient = new PopClient(null, mockTcpClient.Object))
            {

            }
        }
        catch (ArgumentNullException)
        {
            correctExceptionThrown = true;
        }
        finally
        {
            mockTcpClient.VerifyAll();
        }
        Assert.That(correctExceptionThrown);

但是没有调用 dispose，似乎是 C# 规范。

Answer 1

mockTcpClient.Setup(x => x.Dispose());

try 
{
    using (var popClient = new PopClient(null, mockTcpClient.Object))
    {

    }
}
finally 
{
    mockTcpClient.VerifyAll();
}

Answer 2

这并不能回答您的问题（因为它已经解决了），但它是相关的并且无论如何都值得发布。

[ExpectedException] 是一种非常可靠的测试异常的方法。它可能容易出错，因为错误的行可能会触发预期类型的​​异常，从而导致错误的传递。我强烈建议您改为查看 Assert.Throws() :)

它更好用（您可以查询返回的异常），更具可读性，最重要的是，更安全。

您可以在此处找到示例

Answer 3

您已经发现，如果构造函数失败，确实不应该调用 dispose。但在其他情况下，您可能希望在引发预期异常后验证您的模拟。我只是在测试 TearDown 中这样做：

[SetUp]
public void SetUp()
{
   this.mockFactory = new MockFactory(MockBehavior.Loose);
}

[TearDown]
public void TearDown()
{
   this.mockFactory.VerifyAll();
}


[Test]
[ExpectedException(typeof(NoBananasException))]
public void EatThrowsIfNoBananasAvailable
{
   var bananaProvider = this.mockFactory.Create<IBananaProvider>();
   bananaProvider.SetUp(bp => bp.IsBananaAvailable).Returns(false).Verifiable();

   var monkey = new Monkey(bananaProvider.Object);
   monkey.Eat();
}

Answer 4

您似乎正在测试即使构造函数抛出异常，注入的mockTcpClient实例也会被释放，在这种情况下这应该有效：

mockTcpClient.Setup(x => x.Dispose());
try
{
    var popClient= new PopClient(null, mockTcpClient.Object);
}
finally
{
    mockTcpClient.VerifyAll();
}

编辑：实际上，try/finally会比捕获异常更干净。请注意，您不需要处置 popClient，因为没有创建实例。