我对单元测试相对较新,而且对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);
}
答案 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);
}
这简化了需要完成/测试以演示方法的内容。注入依赖关系时需要注意的一个问题是,最终创建抽象抽象,将实际逻辑更深入地深入到代码中,只是发现你只是推动了它更深层次的问题,你仍然不知道如何测试一个特定的逻辑,因为从本质上说它没有改变。