在单元测试期间模拟特定方法

时间:2015-05-26 13:06:51

标签: c# unit-testing nunit moq

我对单元测试相对较新,而且对Moq来说是全新的。我的任务是为一些预先存在的代码编写一些单元测试。

我正在努力使用以下方法,因为它调用model.importing.RunJob,正如其名称所示,开始工作。

显然,我不希望它真正完成工作,我只是想知道它被称为方法。

我将如何实现这一目标(理想情况下避免任何代码更改):

public ActionResult ImportsAndLoads(ImportsAndLoadsViewModel importsAndLoadsModel)
{
    Initialise(importsAndLoadsModel);
    LoadData(importsAndLoadsModel);

    if (importsAndLoadsModel.ActionToPerform == "RunJob")
    {
        using (var model = new Model())
        {
            List<Message> lstMessage;
            model.importing.RunJob((Jobs)Enum.ToObject(typeof(Jobs), importsAndLoadsModel.SelectedJob), out lstMessage);
            importsAndLoadsModel.lstMessages = lstMessage;
        }
    }
    return View(importsAndLoadsModel);
}

1 个答案:

答案 0 :(得分:1)

正如您对问题的评论中所讨论的那样,您无法使用Moq以当前形式单独测试您的ImportsAndLoads方法。如果您使用的是足够高版本的Visual Studio,那么您可以使用Shims测试该方法,但是如果您尚未在项目中使用此方法,则可能不是正确的方法。

目前,您的代码有点令人困惑。有一个名为Model的类,您正在创建它,以便访问属性以调用RunJob方法,这有点奇怪。如果这确实是代码,那么您可能希望鼓励团队重新访问Model类。

其中一个建议是您注入Model依赖项而不是创建它(或者注入工厂并调用工厂进行创建)。这将是最好的方式,但这不是一个微不足道的改变方法,你的团队真的需要购买它作为一种方法,而不是你作为一次性的变化。

如果您还没有使用IOC容器(AutoFac,CastleWindsor,Ninject),那么您可能需要考虑使用一个。他们将更容易切换到依赖注入。如果你想手工完成,那么你可以这样做,但它更难。

如果不了解更多课程的结构,很难给出一个完整的例子,但有一种方法可能如下:

// Create an importer interface
public interface IImporter {
    void RunJob(Jobs job, out List<Message> listMessages);
}

// Implement it in a concrete class
public class Importer : IImporter{

    public void RunJob(Jobs job, out List<Message> listMessages) {
        // Do whatever it is it does.
    }
}

// Modify the constructor of your controller to allow the IImporter
// interface to be injected.  Default to null if not supplied, so that
// it can still be crated without parameters by the default MVC plumbing
public class ImportController : Controller
{
    private IImporter _importer;
    public ImportController(IImporter importer = null) {
        // If importer not injected, created a concrete instance
        if (null == importer) {
            importer = new Importer();
        }
        // Save dependency for use in actions
        _importer = importer;
    }

    // Use the injected importer in your action, rather than creating a model
    public ActionResult ImportsAndLoads(ImportsAndLoadsViewModel importsAndLoadsViewModel)
    {
        List<Message> listMsgs;
        _importer.RunJob(Jobs.One, out listMsgs);
        importsAndLoadsViewModel.lstMessages = listMsgs;
        return View(importsAndLoadsViewModel);
    }
}

这将允许您编写测试以验证importsAndLoadsViewModel已按预期更新,使用如下测试:

[Test]
public void TestModelMessagesAreUpdatedFromJobRunner() {
    var mockImporter = new Mock<IImporter>();
    List<Message> expectedMessages = new List<Message>();

    mockImporter.Setup(x=>x.RunJob(It.IsAny<Jobs>(), out expectedMessages));

    var model = new ImportsAndLoadsViewModel();

    // Inject the mocked importer while constructing your controller
    var sut = new ImportController(mockImporter.Object);

    // call the action you're testing on your controller
    ViewResult response = (ViewResult)sut.ImportsAndLoads(model);

    // Validate the the model has been updated to have the messages
    // returned by the mocked importer.
    Assert.AreEqual(expectedMessages, 
                    ((ImportsAndLoadsViewModel)response.Model).lstMessages);
}

这简化了需要完成/测试以演示方法的内容。注入依赖关系时需要注意的一个问题是,最终创建抽象抽象,将实际逻辑更深入地深入到代码中,只是发现你只是推动了它更深层次的问题,你仍然不知道如何测试一个特定的逻辑,因为从本质上说它没有改变。