NSubstitute не соответствует выражению Linq

Я реализую класс запроса шаблона репозитория и тестирую его с помощью NSubstitute.

Интерфейс репозитория:

public interface IMyRepository
{
    IQueryable<T> Query<T>(Expression<Func<T, bool>> filter) where T : class;
}

Интерфейс DateTimeProvider:

public interface IMyDateTimeProvider
{
    DateTime GetDateNow();
}

Интерфейс приложения:

public interface IMyApplication
{
    List<Thing> GetThingsByQuery(int status);
}

Реализация приложения:

public class MyApplication : IMyApplication
{
    private readonly IMyRepository myRepository;

    private readonly IMyDateTimeProvider myDateTimeProvider;

    public MyApplication(IMyRepository myRepository, IMyDateTimeProvider myDateTimeProvider)
    {
        this.myRepository = myRepository;
        this.myDateTimeProvider = myDateTimeProvider;
    }

    public List<Thing> GetThingsByQuery(int status)
    {
        var createdDate = this.myDateTimeProvider.GetDateNow();

        return this.myRepository.Query<Thing>(t => t.CreatedDate == createdDate && t.Status == status).ToList();
    }
}

Тестовое задание:

[TestClass]
public class ApplicationTest
{
    private IMyApplication myApplication;

    private IMyDateTimeProvider myDateTimeProvider;

    private IMyRepository myRepository;

    [TestMethod]
    public void QueriesRepository()
    {
        // Arrange
        var createdDate = new DateTime(2014, 1, 1);

        this.myDateTimeProvider.GetDateNow().Returns(createdDate);

        const int Status = 1;

        // Act
        this.myApplication.GetThingsByQuery(Status);

        // Assert
        this.myRepository.Received().Query<Thing>(t => t.CreatedDate == createdDate && t.Status == Status);
    }

    [TestInitialize]
    public void TestInitialize()
    {
        this.myRepository = Substitute.For<IMyRepository>();

        this.myDateTimeProvider = Substitute.For<IMyDateTimeProvider>();

        this.myApplication = new MyApplication(this.myRepository, this.myDateTimeProvider);
    }
}

Но тест не проходит со следующим сообщением:

NSubstitute.Exceptions.ReceivedCallsException: Expected to receive a call matching:
    Query<Thing>(t => ((t.CreatedDate == value(MySolution.Test.ApplicationTest+<>c__DisplayClass0).createdDate) AndAlso (t.Status == 1)))
Actually received no matching calls.
Received 1 non-matching call (non-matching arguments indicated with '*' characters):
    Query<Thing>(*t => ((t.CreatedDate == value(MySolution.Application.MyApplication+<>c__DisplayClass0).createdDate) AndAlso (t.Status == value(MySolution.Application.MyApplication+<>c__DisplayClass0).status))*)

DateTime и Status анализируются в value(), которые различаются между Application и Test.

Почему это? Как я могу это исправить?


repository-pattern linq nsubstitute linq-expressions
person Shevek    schedule 20.10.2014    source источник

Ответы (2)


arrow_upward
1
arrow_downward

Для сложных выражений, если часто проще утверждать захваченные аргументы с помощью обратных вызовов, чем с Received() . (неполный) пример:

Expression<Func<Thing, bool>> receivedFilter receivedFilter = null;
myRepository.When(x => x.Query<Thing>(Arg.Any<...>))
  .Do(x => receivedQuery = x.Arg<Expression<Func<Thing, bool>>>());

Затем утвердите захваченное выражение фильтра. На самом деле может быть проще просто выполнить функцию фильтра выражения (см., например, здесь)

Func<Thing, bool> predicate = receivedFilter.Compile();
var matchingThing = new Thing 
  { CreatedDate = createdData, Status = Status };
// assert matching
predicate(matchingThing).Should().BeTrue();

// assert non.matching
predicate(nonMatchingThing).Should().BeFalse();

Этот подход, кажется, делает тест немного более закрытым, но в целом это неплохо.

person mkoertgen    schedule 15.07.2015

arrow_upward
0
arrow_downward

Используется компаратор равенства по умолчанию для выражения (ссылочное равенство):

например, выражение (t => t.CreatedDate == createdDate && t.Status == Status``) в:

this.myRepository.Received().Query<Thing>(t => t.CreatedDate == createdDate  
                                            && t.Status == Status          );

Является другим экземпляром выражения в:

return this.myRepository.Query<Thing>(t => t.CreatedDate == createdDate  
                                        && t.Status == status          ).ToList();

Чтобы исправить проверку этого метода, проверьте сопоставители аргументов в NSubstitute.

Но как пример:

Func<Expression<Thing, bool>, bool> validator = 
    // TODO this needs to be written properly, based on the expression, 
    // not its string representation
    e => e.Body.ToString() == "t.CreatedDate == createdDate  
      && t.Status == Status"; 
this.myRepository.Received().Query<Thing>(Arg.Is<Expression<Thing, bool>>(validator));
person Rich O'Kelly    schedule 20.10.2014
comment
Я не уверен, как это реализовать. Я пробовал this.myRepository.Received().Query<Thing>(t => t.CreatedDate == Arg.Any<DateTime>() && t.Status == Arg.Any<int>()); и this.myRepository.Received().Query<Thing>(t => t.CreatedDate == Arg.Is(createdDate) && t.Status == Arg.Is(Status));, но оба все еще используют value() и терпят неудачу - person Shevek; 20.10.2014
comment
Я не смог заставить ваш пример скомпилироваться. Это компилируется, но тест завершается с той же ошибкой: Expression<Func<Thing, bool>> validator = t => t.CreatedDate == createdDate && t.Status == Status; и this.myRepository.Received().Query<Thing>(Arg.Is<Expression<Func<Thing, bool>>>(validator)); - person Shevek; 20.10.2014
comment
На самом деле, ReSharper затем заявляет, что объявления не требуются, и .Received становится this.myRepository.Received().Query(Arg.Is(validator));, но все равно не проходит тест. - person Shevek; 20.10.2014
comment
Мой пример был задуман как пример, вы можете изменить валидатор на: Func<Expression<Thing, bool>, bool> validator = e => true, который позволит пройти тест и проверит, что метод был вызван, но не с правильным выражением... - person Rich O'Kelly; 20.10.2014
comment
Я могу проверить, что метод был вызван с использованием this.myRepository.Received().Query(Arg.Any<Expression<Func<Thing, bool>>>());, но на самом деле я хочу проверить, что используется правильный запрос. - person Shevek; 20.10.2014