测试MVC控制器失败,出现NULL引用异常

时间:2015-08-27 08:47:26

标签: asp.net-mvc unit-testing moq

以下是我要测试的设置。

控制器:

public ActionResult UpsertStudent(StudentModel studentModel)
{
    try
    {

        if (!CheckStudentUpdateForEdit(studentModel))
        {

            return Json(new { result = STUDENT_EXISTS });

        }
    // remaining code removed for brevity 
}

private bool CheckStudentUpdateForEdit(StudentModel studentModel)
{
    var returnVal = true;

    var existingStudent = _updateStudentManager.GetStudentInfo(studentModel.Id);

    if (existingStudent.StudentType == "Day Scholar")
    {
        returnVal = true;
    }
    else
    {
        returnVal = false;
    }

    return returnVal;
}

测试方法:

public void AllowStudentUpdates_Success()
{
    var studentModel = new StudentModel()
    {
        StudentName = "Joe",
        Id = "123",
        StudentType = "Day Scholar"
    };

    var studentToAdd = new Student()
    {
        Id = "123",
        Name = "Joe",
        StartDate = DateTime.UtcNow.ToShortDateString(),
        StudentType = "Day Scholar",
        EndDate = "08/10/2016"
    };

    _studentRulesHelper.Setup(x => x.GetStudentRule(studentModel, true)).Returns(studentToAdd);
    _productRulesHelper.Setup(x => x.ReturnStudentRule(studentModel, true)).Returns(studentToAdd);

    var res = _controller.UpsertStudent(studentModel) as JsonResult;
    if (res != null) Assert.AreEqual("{ result = True }", res.Data.ToString());
}

当它到达UpsertDoc调用时,它转到控制器中的实际调用并尝试执行CheckStudentUpdateForEdit() GetStudentInfo()尝试从数据库中获取对象并返回null对象,因为没有学生具有从测试方法传递的id。 然后测试失败,出现Null Reference异常。

现在被测系统不应该命中db。我不知道为什么会这样做!

编写此测试的任何其他人也将尝试传递一个虚拟对象,该对象在GetStudentInfo()处以现在设置测试的方式失败。

我该怎么做才能使这项工作?

1 个答案:

答案 0 :(得分:0)

我不确定我是否正确理解了您的问题,但是查看提供的代码片段,测试将会进入数据库,因为模拟对象及其期望未定义。

我会实现这样的解决方案 -

我假设您的_updateStudentManager对象适用于为Student进行数据库交互的类。我称之为StudentRepository。并且允许你模仿我将使它Interface驱动的行为。 所以通常我的设置看起来像这样 -

//Interface
public interface IStudentrepository
{
    StudentModel GetStudentInfo(int studentId);
}

//Class implementing IStudentrepository
public class StudentRepository : IStudentrepository
{
    public StudentModel GetStudentInfo(int studentId)
    {
        //Implementation goes here
    }
}

现在在我的控制器中,我将有一个IStudentrepository实例,可以通过构造函数注入。

public class StudentController
{
    private readonly IStudentrepository updateStudentManager;
    public StudentController(IStudentrepository updateStudentManager)
    {
        this.updateStudentManager = updateStudentManager;
    }
}
//Rest of the code for controller....

现在在编写Test时,我将创建一个IStudentrepository的模拟对象,定义模拟对象的期望,并在创建控制器对象时注入它。像这样的东西。

    [TestMethod]
    public void TestMethod1()
    {
        //--Arrange--
        //Define a mock object for student repository
        var mock = new Mock<IStudentrepository>();

        //Define the expectations of the mock object
        mock.Setup(s => s.GetStudentInfo(It.IsAny<int>()))
            .Returns(new StudentModel {/*return the required object */ });

        //Instantiate controller and inject the mock object
        StudentController _controller = new StudentController(mock.Object);

        //--Act--
        var res = _controller.UpsertStudent(studentModel) as JsonResult;


        //--Assert--
        if (res != null) Assert.AreEqual("{ result = True }", res.Data.ToString());

    }

现在,当您的测试方法调用GetStudentInfo方法时,它将返回模拟对象中设置的值,而不是命中数据库。

这只是一个高级实现,当然您可以根据您的设计进行修改。希望它有所帮助