如果您以后验证方法已被调用,则在回调中进行断言是否可以接受?这是确保我的模拟程序获得期望的参数的首选方法,还是应该在回调中设置局部变量并对该实例执行断言?
我遇到一种情况,在Presenter类中有一些逻辑,该逻辑可根据输入派生值并将它们传递给Creator类。为了测试Presenter类中的逻辑,我想验证在调用Creator时是否遵守了正确的派生值。我想出了下面的示例,但是我不确定我是否喜欢这种方法:
[TestFixture]
public class WidgetCreatorPresenterTester
{
[Test]
public void Properly_Generates_DerivedName()
{
var widgetCreator = new Mock<IWidgetCreator>();
widgetCreator.Setup(a => a.Create(It.IsAny<Widget>()))
.Callback((Widget widget) =>
Assert.AreEqual("Derived.Name", widget.DerivedName));
var presenter = new WidgetCreatorPresenter(widgetCreator.Object);
presenter.Save("Name");
widgetCreator.Verify(a => a.Create(It.IsAny<Widget>()), Times.Once());
}
}
我担心,因为如果没有最后的
Verify
调用,则无法保证回调中的断言将被调用。另一种方法是在回调中设置局部变量:[Test]
public void Properly_Generates_DerivedName()
{
var widgetCreator = new Mock<IWidgetCreator>();
Widget localWidget = null;
widgetCreator.Setup(a => a.Create(It.IsAny<Widget>()))
.Callback((Widget widget) => localWidget = widget);
var presenter = new WidgetCreatorPresenter(widgetCreator.Object);
presenter.Save("Name");
widgetCreator.Verify(a => a.Create(It.IsAny<Widget>()), Times.Once());
Assert.IsNotNull(localWidget);
Assert.AreEqual("Derived.Name", localWidget.DerivedName);
}
我觉得这种方法更容易出错,因为它更明确,而且更容易看到
Assert
语句将被调用。一种方法比另一种更好吗?有没有更简单的方法来测试传递给我所缺少的模拟的输入参数?如果有帮助,下面是此示例的其余代码:
public class Widget
{
public string Name { get; set; }
public string DerivedName { get; set; }
}
public class WidgetCreatorPresenter
{
private readonly IWidgetCreator _creator;
public WidgetCreatorPresenter(IWidgetCreator creator)
{
_creator = creator;
}
public void Save(string name)
{
_creator.Create(
new Widget { Name = name, DerivedName = GetDerivedName(name) });
}
//This is the method I want to test
private static string GetDerivedName(string name)
{
return string.Format("Derived.{0}", name);
}
}
public interface IWidgetCreator
{
void Create(Widget widget);
}
编辑
我更新了代码,以使在问题中概述的第二种方法更易于使用。我将在Setup/Verify中使用的表达式的创建拉到一个单独的变量中,因此只需要定义一次即可。我觉得这种方法是我最喜欢的方法,它易于设置,并且会因错误消息而失败。
[Test]
public void Properly_Generates_DerivedName()
{
var widgetCreator = new Mock<IWidgetCreator>();
Widget localWidget = null;
Expression<Action<IWidgetCreator>> expressionCreate =
(w => w.Create(It.IsAny<Widget>()));
widgetCreator.Setup(expressionCreate)
.Callback((Widget widget) => localWidget = widget);
var presenter = new WidgetCreatorPresenter(widgetCreator.Object);
presenter.Save("Name");
widgetCreator.Verify(expressionCreate, Times.Once());
Assert.IsNotNull(localWidget);
Assert.AreEqual("Derived.Name", localWidget.DerivedName);
}
最佳答案
由于代码的结构方式,您不得不在一个单元测试中测试两件事。您正在测试A)演示者正在调用注入(inject)的WidgetCreator的create方法,以及B)在新的Widget上设置了正确的名称。如果可能的话,最好以某种方式使这两项成为两个单独的测试,但这会更好,但是在这种情况下,我真的看不到这样做的方法。
考虑到所有这些,我认为第二种方法更清洁。它对您的期望更加明确,如果失败,则完全可以理解失败的原因和原因。
关于c# - 验证传递给Mock的参数的正确方法已按预期设置,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/5709058/