Question

我在控制器中有这个方法：

[HttpPatch]
[AllowAdminOnly]
public JsonResult EditUser(User _user)
{
    try
    {
        if (ModelState.IsValid)
        {
            _user.Edit();
        }
        else
        {
            string error_messages = "";
            foreach (var e in ModelState.Select(x => x.Value.Errors).Where(y => y.Count > 0).ToList())
            {
                error_messages += e[0].ErrorMessage + "\n";
            }
            throw new Exception(error_messages);
        }
        return MessageHelper(@Resources.Global.success, @Resources.Users.success_editing_user, "success");
    }
    catch (Exception err)
    {
        return ErrorHelper(@Resources.Global.error, @Resources.Users.error_editing_user, err.Message);
    }
}

为了对这种方法进行单元测试，我发现引用（...）表示我应该＆＃34;模拟＆＃34; _user.Edit();。看起来很好，它将避免将数据保存到数据库，并使测试更快。

然后我的测试（对于有效用户）变为：

[TestMethod]
public void UserController_EditUser_Valid()
{
    // Arrange
    FundController controller = new UserController();
    controller.ControllerContext = TestModelHelper.AdminControllerContext();

    var _user = new Mock<Fund>();
    _user.SetupGet(f => f.id).Returns(1);
    _user.SetupGet(f => f.name).Returns("User name");
    _user.SetupGet(f => f.nickname).Returns("User nickname");
    _user.SetupGet(f => f.active).Returns(true);
    _user.Setup(f => f.Edit()).Callback(() => {}).Verifiable();

    // Act
    var result = (JsonResult)controller.EditUser(_user.Object);
    SimpleMessage resultMessage = m_serializer.Deserialize<SimpleMessage>(m_serializer.Serialize(result.Data));

    // Assert
    Assert.IsNotNull(resultMessage, "Result must not be null");
    Assert.IsTrue(resultMessage.status.Equals("success"), "status must be 'success'");
}

但是当我这样做时，我收到以下错误：

Test Name:  UserController_EditUser_Valid
Test FullName:  Pmanager.Tests.Controllers.UserControllerTest.UserController_EditUser_Valid
Test Source:    ...\Pmanager\Pmanager.Tests\Controllers\UserControllerTest.cs : line 95
Test Outcome:   Failed
Test Duration:  0:00:00,0179908

Result StackTrace:  
em Moq.Mock.ThrowIfCantOverride(Expression setup, MethodInfo method)
   em Moq.Mock.<>c__DisplayClass66_0`2.<SetupGet>b__0()
   em Moq.PexProtector.Invoke[T](Func`1 function)
   em Moq.Mock.SetupGet[T,TProperty](Mock`1 mock, Expression`1 expression, Condition condition)
   em Moq.Mock`1.SetupGet[TProperty](Expression`1 expression)
   em Pmanager.Tests.Controllers.UserControllerTest.UserController_EditUser_Valid() na ...\Pmanager\Pmanager.Tests\Controllers\UserControllerTest.cs:linha 100
Result Message: 
Test method Pmanager.Tests.Controllers.UserControllerTest.UserController_EditUser_Valid threw exception: 
System.NotSupportedException: Valid setup on a non-virtual (overridable in VB) member: f => f.id

我找到了一些关于（...）的文档说我在创建＆＃34; mock＆＃34; 时应该使用界面而不是类。

所以，我创建了一个界面：

public interface IUser
{
    int id { get; set; }
    string name { get; set; }
    string nickname { get; set; }
    bool active { get; set; }
    void Edit();
}

然后我更改了控制器方法签名IUser的所有内容：

public JsonResult EditUser(IUser _user);

测试＆＃34; mock＆＃34; 声明：

var _user = new Mock<IUser>();

等等。

现在测试工作正常，但实际控制器编辑用户的方法不行！

如何在不破坏控制器功能的情况下将所有这些东西放在一起？

Answer 1

更新控制器以使用依赖关系来更新用户模型。从模型本身中删除该功能。模型应该是POCO / DTO。

<uses-permission android:name="android.permission.INTERNET" />

其中public class UserController : Controller { readonly IUserService userService; public UserController(IUSerService userService) { this.userService = userService; } [HttpPatch] [AllowAdminOnly] public JsonResult EditUser(User _user) { try { if (ModelState.IsValid) { userService.Edit(user); } else { string error_messages = ""; foreach (var e in ModelState.Select(x => x.Value.Errors).Where(y => y.Count > 0).ToList()) { error_messages += e[0].ErrorMessage + "\n"; } throw new Exception(error_messages); } return MessageHelper(@Resources.Global.success, @Resources.Users.success_editing_user, "success"); } catch (Exception err) { return ErrorHelper(@Resources.Global.error, @Resources.Users.error_editing_user, err.Message); } } }类似于

IUserService

及其生产实施将执行所需的操作。请记住使用您正在使用的DI注册抽象和实现。

然后，测试将模拟它独立运行所需的依赖关系。

public interface IUserService {
    void Edit(User user);
}

Answer 2

好吧，我仍然怀疑这种做法的正确方法。

但我想办法让两者兼得。它是：

保留控制器的方法签名：public JsonResult EditUser(User _user);
在将模型传递给控制器的方法时，在模拟中进行解析：var result = (JsonResult)controller.EditUser(_user.Object as User);

虽然它有效但我不确定整个东西是否和谐...... :(

希望有人能给出更好的答案。

C＃MVC模拟模型通过接口来破坏控制器

2 个答案: