После использования Automapper для сопоставления ViewModel, как и что я должен проверить?

Question

Я пытаюсь проверить действие Index контроллера. Действие использует AutoMapper для сопоставления объекта домена Customer с моделью представления TestCustomerForm. Хотя это работает, меня беспокоит лучший способ проверить результаты, которые я получаю от действия Index.

Индексное действие контроллера выглядит так:

public ActionResult Index()
{
    TestCustomerForm cust = Mapper.Map<Customer,
        TestCustomerForm>(_repository.GetCustomerByLogin(CurrentUserLoginName));

    return View(cust);
}

И TestMethod выглядит так:

[TestMethod]
public void IndexShouldReturnCustomerWithMachines()
{
    // arrange
    var customer = SetupCustomerForRepository(); // gets a boiler plate customer
    var testController = CreateTestController();

    // act
    ViewResult result = testController.Index() as ViewResult;

    // assert
    Assert.AreEqual(customer.MachineList.Count(),
        (result.ViewData.Model as TestCustomerForm).MachineList.Count());
}

В методе CreateTestController я использую Rhino.Mocks, чтобы смоделировать хранилище клиента и настроить его для возврата клиента из SetupCustomerForRepository. Таким образом, я знаю, что хранилище вернет предполагаемого клиента, когда действие Index вызовет _repository.GetCustomerByLogin(CurrentUserLoginName). Поэтому я считаю, что равного количества достаточно для удовлетворения IndexShouldReturnCustomerWithMachines.

Все это говорит о том, что я обеспокоен тем, что мне следует проверять.

1. Кажется самонадеянным разыграть result.ViewData.Model as TestCustomerForm. Это действительно проблема? Это касается меня, потому что в данном случае я не занимаюсь разработкой, управляемой тестами, и мне кажется, что я рассчитываю на конкретную реализацию, чтобы выполнить тест.
2. Существуют ли более подходящие тесты для обеспечения правильного отображения?
3. Должен ли я проверять каждое сопоставленное свойство из TestCustomerForm?
4. Есть ли более общие тестовые действия контроллера, которые я должен делать?

Answer 1

Это одна из причин, по которой мы перемещаем AutoMapper в пользовательский ActionResult или ActionFilter. В какой-то момент вы действительно хотите проверить, что вы сопоставили Foo с FooDto, но не обязательно проверять фактическое сопоставление. Переместив AutoMapper в границы слоя (например, между контроллером и видом), вы можете просто проверить, что вы говорите AutoMapper.

Это похоже на тестирование ViewResult. Вы не проверяете на контроллере, что представление было отображено, но скорее говорите MVC отображать то-то и то-то. Результатом нашего действия становится:

public class AutoMapViewResult : ActionResult
{
    public Type SourceType { get; private set; }
    public Type DestinationType { get; private set; }
    public ViewResult View { get; private set; }

    public AutoMapViewResult(Type sourceType, Type destinationType, ViewResult view)
    {
        SourceType = sourceType;
        DestinationType = destinationType;
        View = view;
    }

    public override void ExecuteResult(ControllerContext context)
    {
        var model = Mapper.Map(View.ViewData.Model, SourceType, DestinationType);

        View.ViewData.Model = model;

        View.ExecuteResult(context);
    }
}

С помощью вспомогательного метода в базовом классе контроллера:

protected AutoMapViewResult AutoMapView<TDestination>(ViewResult viewResult)
{
    return new AutoMapViewResult(viewResult.ViewData.Model.GetType(), typeof(TDestination), viewResult);
}

Что затем заставляет контроллер теперь только указывать, что отображать в / из, а не выполнять фактическое отображение:

public ActionResult Index(int minSessions = 0)
{
    var list = from conf in _repository.Query()
                where conf.SessionCount >= minSessions
                select conf;

    return AutoMapView<EventListModel[]>(View(list));
}

На данный момент мне нужно только проверить, "убедитесь, что вы отображаете этот объект Foo в этот тип FooDto назначения", без необходимости фактически выполнять сопоставление.

EDIT:

Вот пример тестового фрагмента:

var actionResult = controller.Index();

actionResult.ShouldBeInstanceOf<AutoMapViewResult>();

var autoMapViewResult = (AutoMapViewResult) actionResult;

autoMapViewResult.DestinationType.ShouldEqual(typeof(EventListModel[]));
autoMapViewResult.View.ViewData.Model.ShouldEqual(queryResult);
autoMapViewResult.View.ViewName.ShouldEqual(string.Empty);

Answer 2

Я бы, вероятно, отделил связь между AutoMapper и контроллером, введя абстракцию:

public interface IMapper<TSource, TDest>
{
    TDest Map(TSource source);
}

public CustomerToTestCustomerFormMapper: IMapper<Customer, TestCustomerForm>
{
    static CustomerToTestCustomerFormMapper()
    {
        // TODO: Configure the mapping rules here
    }

    public TestCustomerForm Map(Customer source)
    {
        return Mapper.Map<Customer, TestCustomerForm>(source);
    }
}

Затем вы передадите это в контроллер:

public HomeController: Controller
{
    private readonly IMapper<Customer, TestCustomerForm> _customerMapper;
    public HomeController(IMapper<Customer, TestCustomerForm> customerMapper)
    {
        _customerMapper = customerMapper;
    }

    public ActionResult Index()
    {
        TestCustomerForm cust = _customerMapper.Map(
            _repository.GetCustomerByLogin(CurrentUserLoginName)
        );
        return View(cust);
    }
}

И в вашеммодульный тест, вы бы использовали ваш любимый макет заглушить этот маппер.