Unidad de prueba de un controlador en ASP.NET MVC 2 con RedirectToAction

Question

Tengo un controlador que implementa una simple operación de Agregar de una entidad y redirige a la página de detalles:

[HttpPost] 
public ActionResult Add(Thing thing) 
{ 
    // ... do validation, db stuff ... 
    return this.RedirectToAction<c => c.Details(thing.Id)); 
} 

Esto funciona muy bien (con el RedirectToAction de el ensamblaje MvcContrib).

Cuando estoy probando unitariamente este método, quiero acceder al ViewData que se devuelve desde la acción Detalles (para poder obtener la clave primaria de la cosa recién insertada y probar que ahora está en la base de datos).

La prueba tiene:

var result = controller.Add(thing); 

Pero resultado aquí es de tipo: System.Web.Mvc.RedirectToRouteResult (que es un System.Web.Mvc.ActionResult). Todavía no ha ejecutado el método Detalles.

He intentado llamar ExecuteResult en el objeto devuelto que pasa en un ControllerContext imitado pero el marco no estaba contento con la falta de detalles en el objeto burlado.

Podría intentar completar los detalles, etc., pero luego mi código de prueba es mucho más largo que el código que estoy probando y creo que necesito pruebas unitarias para las pruebas de la unidad.

¿Me falta algo en la filosofía de las pruebas? ¿Cómo pruebo esta acción cuando no puedo obtener su estado devuelto?

Answer 1

Parece que está haciendo demasiado para una prueba de unidad. La validación y el acceso a los datos normalmente los realizarían los servicios a los que llamas desde la acción del controlador. Se burla de esos servicios y solo prueba que fueron llamados correctamente.

Algo como esto (utilizando la sintaxis aproximado para Rhino.Mocks & NUnit):

[Test] 
public void Add_SavesThingToDB() 
{ 
    var dbMock = MockRepository.GenerateMock<DBService>(); 
    dbMock.Expect(x => x.Save(thing)).Repeat.Once(); 

    var controller = new MyController(dbMock); 
    controller.Add(new Thing()); 

    dbMock.VerifyAllExpectations(); 
} 

[Test] 
public void Add_RedirectsAfterSave() 
{ 
    var dbMock = MockRepository.GenerateMock<DBService>(); 

    var controller = new MyController(dbMock); 
    var result = (RedirectToRouteResult)controller.Add(new Thing()); 

    Assert.That(result.Url, Is.EqualTo("/mynew/url")); 
} 

Answer 2

Hay MVC Contrib TestHelper que son fantásticos para probar la mayor parte del ActionResult

Puede obtenerlo aquí: http://mvccontrib.codeplex.com/wikipage?title=TestHelper

Aquí es un ejemplo de la sintaxis:

var controller = new TestController(); 

controller.Add(thing) 
      .AssertActionRedirect() 
      .ToAction<TestController>(x => x.Index()); 

Para comprobar si los datos se han conservado correctamente, debería preguntar directamente a su base de datos, no sé si está utilizando un ORM o algo, pero debe hacer algo para obtener el último elemento insertado en su base de datos, luego compare con el valor que proporcionó a su Add ActionResult y vea si esto está bien.

No creo que probar su Details ActionResult para ver si sus datos se mantienen es el enfoque correcto. Eso no sería una prueba unitaria, más una prueba funcional.

Pero también debe probar su unidad Detalles método para asegurarse de que su objeto viewdata se llena con los datos correctos procedentes de su base de datos.

Answer 3

Estoy usando MVC2 RC2 en este momento y la respuesta de rmacfie no me funcionó, pero me puso en el camino correcto.

razón o sin ella me las arreglé para hacer esto en mi prueba en su lugar:

var actionResult = (RedirectToRouteResult)logonController.ForgotUsername(model); 

actionResult.RouteValues["action"].should_be_equal_to("Index"); 
actionResult.RouteValues["controller"].should_be_equal_to("Logon"); 

No estoy seguro si esto va a ayudar a alguien, pero podría ahorrar 10 minutos.

Answer 4

Tengo un método de ayuda estático que prueba la redirección.

public static class UnitTestHelpers 
{ 
    public static void ShouldEqual<T>(this T actualValue, T expectedValue) 
    { 
     Assert.AreEqual(expectedValue, actualValue); 
    } 

    public static void ShouldBeRedirectionTo(this ActionResult actionResult, object expectedRouteValues) 
    { 
     RouteValueDictionary actualValues = ((RedirectToRouteResult)actionResult).RouteValues; 
     var expectedValues = new RouteValueDictionary(expectedRouteValues); 

     foreach (string key in expectedValues.Keys) 
     { 
      Assert.AreEqual(expectedValues[key], actualValues[key]); 
     } 
    } 
} 

Luego, crear una prueba de redirección es muy fácil.

[Test] 
public void ResirectionTest() 
{ 
    var result = controller.Action(); 

    result.ShouldBeRedirectionTo(
     new 
     { 
      controller = "ControllerName", 
      action = "Index" 
     } 
    ); 
}