Question

我有一个Transfer类，简化它看起来像这样：

public class Transfer
{
    public virtual IFileConnection source { get; set; }
    public virtual IFileConnection destination { get; set; }

    public virtual void GetFile(IFileConnection connection, 
        string remoteFilename, string localFilename)
    {
        connection.Get(remoteFilename, localFilename);
    }

    public virtual void PutFile(IFileConnection connection, 
        string localFilename, string remoteFilename)
    {
        connection.Get(remoteFilename, localFilename);
    }

    public virtual void TransferFiles(string sourceName, string destName)
    {
        source = internalConfig.GetFileConnection("source");
        destination = internalConfig.GetFileConnection("destination");
        var tempName = Path.GetTempFileName();
        GetFile(source, sourceName, tempName);
        PutFile(destination, tempName, destName);
    }
}

IFileConnection界面的简化版本如下所示：

public interface IFileConnection
{
    void Get(string remoteFileName, string localFileName);
    void Put(string localFileName, string remoteFileName);
}

真正的类应该处理当System.IO.IOException具体类失去与远程连接，发送电子邮件和不发送电子邮件时抛出的IFileConnection。

我想使用Moq创建一个Transfer类，并将其用作所有属性和方法中的具体Transfer类，除非调用GetFile方法 - 然后我希望它抛出System.IO.IOException并确保Transfer类正确处理它。

我使用合适的工具吗？我是以正确的方式来做这件事的吗？我如何编写NUnit的单元测试设置？

Answer 1

以下是模仿FileConnection

的方法

Mock<IFileConnection> fileConnection = new Mock<IFileConnection>(
                                                           MockBehavior.Strict);
fileConnection.Setup(item => item.Get(It.IsAny<string>,It.IsAny<string>))
              .Throws(new IOException());

然后实例化您的Transfer类并在方法调用中使用mock

Transfer transfer = new Transfer();
transfer.GetFile(fileConnection.Object, someRemoteFilename, someLocalFileName);

<强>更新

首先，您必须仅模拟您的依赖项，而不是您正在测试的类（在本例中为Transfer类）。在构造函数中声明这些依赖项，可以轻松查看您的类需要使用哪些服务。在编写单元测试时，它还可以用假货替换它们。目前，用假货替换这些属性是不可能的。

由于您使用其他依赖项设置这些属性，我会这样写：

public class Transfer
{
    public Transfer(IInternalConfig internalConfig)
    {
        source = internalConfig.GetFileConnection("source");
        destination = internalConfig.GetFileConnection("destination");
    }

    //you should consider making these private or protected fields
    public virtual IFileConnection source { get; set; }
    public virtual IFileConnection destination { get; set; }

    public virtual void GetFile(IFileConnection connection, 
        string remoteFilename, string localFilename)
    {
        connection.Get(remoteFilename, localFilename);
    }

    public virtual void PutFile(IFileConnection connection, 
        string localFilename, string remoteFilename)
    {
        connection.Get(remoteFilename, localFilename);
    }

    public virtual void TransferFiles(string sourceName, string destName)
    {
        var tempName = Path.GetTempFileName();
        GetFile(source, sourceName, tempName);
        PutFile(destination, tempName, destName);
    }
}

通过这种方式，您可以模拟internalConfig并使其返回执行您想要的IFileConnection模拟。

Answer 2

这就是我设法做我想做的事情：

[Test]
public void TransferHandlesDisconnect()
{
    // ... set up config here
    var methodTester = new Mock<Transfer>(configInfo);
    methodTester.CallBase = true;
    methodTester
        .Setup(m => 
            m.GetFile(
                It.IsAny<IFileConnection>(), 
                It.IsAny<string>(), 
                It.IsAny<string>()
            ))
        .Throws<System.IO.IOException>();

    methodTester.Object.TransferFiles("foo1", "foo2");
    Assert.IsTrue(methodTester.Object.Status == TransferStatus.TransferInterrupted);
}

如果这种方法有问题，我想知道;其他答案表明我做错了，但这正是我想要做的。

Answer 3

我认为这就是你想要的，我已经测试了这段代码并且正常工作

使用的工具是:(所有这些工具都可以作为Nuget包下载）

http://fluentassertions.codeplex.com/

http://autofixture.codeplex.com/

http://code.google.com/p/moq/

https://nuget.org/packages/AutoFixture.AutoMoq

var fixture = new Fixture().Customize(new AutoMoqCustomization());
var myInterface = fixture.Freeze<Mock<IFileConnection>>();

var sut = fixture.CreateAnonymous<Transfer>();

myInterface.Setup(x => x.Get(It.IsAny<string>(), It.IsAny<string>()))
        .Throws<System.IO.IOException>();

sut.Invoking(x => 
        x.TransferFiles(
            myInterface.Object, 
            It.IsAny<string>(), 
            It.IsAny<string>()
        ))
        .ShouldThrow<System.IO.IOException>();

<强>编辑：

让我解释一下：

当您编写测试时，您必须确切地知道要测试的内容，这称为：“受测试对象（SUT）”，如果我的理解是正确的，在这种情况下，您的SUT是：Transfer

所以考虑到这一点，你不应该嘲笑你的SUT，如果你替换你的SUT，那么你就不会真正测试真正的代码

当您的SUT具有外部依赖性（非常常见）时，您需要替换它们以便在隔离中测试您的SUT。当我说替代时，我指的是根据你的需要使用模拟，虚拟，模拟等

在这种情况下，您的外部依赖项是IFileConnection，因此您需要为此依赖项创建模拟并将其配置为抛出异常，然后只需调用您的SUT实方法并断言您的方法按预期处理异常

var fixture = new Fixture().Customize(new AutoMoqCustomization());：这个linie初始化一个新的Fixture对象（Autofixture库），这个对象用于创建SUT，而不必显式地担心构造函数参数，因为它们是自动创建的或者是模拟的，在这种情况下使用Moq
var myInterface = fixture.Freeze<Mock<IFileConnection>>();：这会冻结IFileConnection依赖关系。冻结意味着Autofixture在被问到时会始终使用这种依赖关系，就像单身一样简单。但有趣的是我们正在创建一个这种依赖的模拟，你可以使用所有的Moq方法，因为这是一个简单的Moq对象
var sut = fixture.CreateAnonymous<Transfer>();：此处AutoFixture正在为我们创建SUT
myInterface.Setup(x => x.Get(It.IsAny<string>(), It.IsAny<string>())).Throws<System.IO.IOException>();这里您要配置依赖项，以便在调用Get方法时抛出异常，此接口中的其余方法未配置，因此如果您尝试访问它们您将收到意外的异常
sut.Invoking(x => x.TransferFiles(myInterface.Object, It.IsAny<string>(), It.IsAny<string>())).ShouldThrow<System.IO.IOException>();：最后，测试你的SUT的时间，这一行使用FluenAssertions库，它只是调用来自SUT的TransferFiles 真实方法并且作为参数，它接收模拟的IFileConnection，因此每当您在SUT IFileConnection.Get方法的正常流程中调用TransferFiles时，模拟对象将调用抛出已配置的异常，这是断言你的SUT正确处理异常的时候，在这种情况下，我只是确保使用ShouldThrow<System.IO.IOException>()（来自FluentAssertions库）抛出异常

建议参考：

http://martinfowler.com/articles/mocksArentStubs.html

http://misko.hevery.com/code-reviewers-guide/

http://misko.hevery.com/presentations/

http://www.youtube.com/watch?v=wEhu57pih5w&feature=player_embedded

http://www.youtube.com/watch?v=RlfLCWKxHJ0&feature=player_embedded

模拟一个抛出异常（moq）的方法，但其他方式就像模拟对象一样？

3 个答案: