我想用一个方法设置一个模拟对象,该方法应该为每个输入返回不同的东西。当我尝试类似下面给出的代码时,我得到了关于访问修改后的闭包的警告(即访问i
)。如何正确设置模拟对象的FooMethod(i)
以返回i*3
?
var ret = new Mock<FooClass>();
for(var i = 0; i < 10; i++){
var bar = i*3;
ret.Setup(x => x.FooMethod(i)).Returns(bar);
}
奥利维尔在评论中已经说过了。由于for
变量的闭包语义,Moq对您的代码的看法是:
ret.Setup(x => x.FooMethod(10)).Returns(0);
ret.Setup(x => x.FooMethod(10)).Returns(3);
ret.Setup(x => x.FooMethod(10)).Returns(6);
ret.Setup(x => x.FooMethod(10)).Returns(9);
ret.Setup(x => x.FooMethod(10)).Returns(12);
ret.Setup(x => x.FooMethod(10)).Returns(15);
ret.Setup(x => x.FooMethod(10)).Returns(18);
ret.Setup(x => x.FooMethod(10)).Returns(21);
ret.Setup(x => x.FooMethod(10)).Returns(24);
ret.Setup(x => x.FooMethod(10)).Returns(27);
当然,这不是你想要的。 (Moq只会设置为值10
,并且它将使用最新的Setup
,因此返回27
。)
不要关闭(捕获)for
变量。一个修复是:
for (var i = 0; i < 10; i++) {
var iCopy = i;
ret.Setup(x => x.FooMethod(iCopy)).Returns(3 * iCopy);
}
在较新版本的C#中,foreach
循环是不同的。因此,这有效:
foreach (var i in Enumerable.Range(0, 10)) {
ret.Setup(x => x.FooMethod(i)).Returns(3 * i);
}
Moq提供了另一种不需要循环的方法。它是这样的:
ret.Setup(x => x.FooMethod(It.IsAny<int>())).Returns((int i) => 3 * i);
当然,这适用于所有i
。如果您不想这样,可以更改为:
ret.Setup(x => x.FooMethod(It.Is((int i) => 0 <= i && i < 10))).Returns((int i) => 3 * i);