Как я могу имитировать фабрику асинхронного общего кэша с помощью Moq?

Я пытаюсь высмеять следующий метод службы кэширования, используя Moq:

Task<TItem> GetOrCreateAsync<TItem>(string key, Func<Task<TItem>> factory)

Этот метод обычно проверяет кэш на наличие записи и возвращает ее, а если не найден, создает новый экземпляр, вызывая фабрику. Я хочу создать макет, который просто вызывает фабрику и возвращает результат вызывающей стороне.

Вот минимальный воспроизводимый пример (LINQPad Instant Share):

var mock = new Mock<ICache>();
mock.Setup(cache => cache.GetOrCreateAsync(It.IsAny<string>(), It.IsAny<Func<Task<It.IsAnyType>>>()).Result)
  .Returns((string b, Func<Task<object>> c) => c().Result);

mock.Object.GetOrCreateAsync("test", () => Task.FromResult(123));

public interface ICache
{
  Task<TItem> GetOrCreateAsync<TItem>(string key, Func<Task<TItem>> factory);   
}

Когда я запускаю вышеуказанное, я получаю InvalidCastException: Unable to cast object of type 'System.Threading.Tasks.Task'1[Moq.It+IsAnyType]' to type 'System.Threading.Tasks.Task'1[System.Int32]'.

Есть идеи?

Какая подпись у GetOrCreateAsync?

DavidG 15.04.2024 03:02

Не могу воспроизвести. Пожалуйста, опубликуйте минимальный воспроизводимый пример.

Mark Seemann 15.04.2024 09:00

Добавлен минимальный воспроизводимый пример.

Mike 15.04.2024 18:46
Стоит ли изучать PHP в 2026-2027 годах?
Стоит ли изучать PHP в 2026-2027 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
1
3
64
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Для насмешки возвращаемого значения асинхронной функции следует использовать ReturnsAsync, а не Returns. Как только вы это исправите, я также буду использовать более конкретные объекты в макете на основе конкретного теста.

Согласно github.com/devlooped/moq/wiki/Quickstart#async-methods ReturnsAsync больше не требуется.

Mike 15.04.2024 02:53

Узнавайте что-то новое каждый день!

Samuel Barnes 15.04.2024 03:10

Мне любопытно, почему вы не вернули объект вместо лямбда-функции. Похоже, ему не нравится разница между тем, что вы заявили, что это необходимо в качестве входных данных в макете, и внутренними возвратами лямбды.

Samuel Barnes 15.04.2024 03:15
Ответ принят как подходящий

Сообщение об ошибке уже объясняет, в чем проблема. Task<It+AnyType> несовместим с Task<int>. Самый простой способ решить эту проблему — использовать аргументы соответствующего типа:

[Fact]
public async Task Repro()
{
    var mock = new Mock<ICache>();
    mock.Setup(cache => cache.GetOrCreateAsync(
            It.IsAny<string>(),
            It.IsAny<Func<Task<int>>>()))
      .Returns((string b, Func<Task<int>> c) => c());

    await mock.Object.GetOrCreateAsync("test", () => Task.FromResult(123));
}

Этот тест проходит без каких-либо исключений.

Если вы хотите иметь какую-то многоразовую настройку, используйте дженерики так, как они были задуманы. Возможно, проще всего сначала настроить вспомогательный метод:

private static void SetupCache<T>(Mock<ICache> mock, T returnValue)
{
    mock.Setup(cache => cache.GetOrCreateAsync(
            It.IsAny<string>(),
            It.IsAny<Func<Task<T>>>()))
        .ReturnsAsync(returnValue);
}

Обратите внимание, что этот метод является универсальным в T, а это означает, что вы можете вызывать его с аргументами разных типов:

[Fact]
public async Task CacheInt()
{
    var mock = new Mock<ICache>();
    SetupCache(mock, 42);
    
    var actual =
        await mock.Object.GetOrCreateAsync("test", () => Task.FromResult(123));
    
    Assert.Equal(42, actual);
}

[Fact]
public async Task CacheString()
{
    var mock = new Mock<ICache>();
    SetupCache(mock, "foo");
    
    var actual =
        await mock.Object.GetOrCreateAsync("test", () => Task.FromResult("bar"));
    
    Assert.Equal("foo", actual);
}

Оба эти теста пройдены.

Спасибо, Марк. Я надеялся на более общее решение, использующее функцию It.IsAnyType, но вместо этого ваше решение является наиболее простым.

Mike 16.04.2024 17:20

Другие вопросы по теме