Использование Moq для проверки параметра типа List <>?

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

var mockSomeRepository = new Mock<SomeRepository>();
mockSomeRepository.Setup(m => m.Write(It.IsAny<List<SomeDTO>>())).Verifiable();

var mainClass = new MainClass(mockSomeRepository.Object);
List<SomeDTO> someList = GetListWith25Items();

mainClass.DoRepositoryWrite(someList); // calls SomeRepository.Write(someList);

mockSomeRepository.Verify(m => 
    m.Write(It.Is<List<SomeDTO>>(l => l.Count() == 25)), Times.Once());

Проверка assert выдает исключение, в котором говорится, что метод никогда не вызывается таким образом. Однако удаление ограничения и использование Is.Any<List<SomeDTO>>() вместо этого приводит к проходу. Я не уверен, что я правильно использую It.Is < > () здесь - это то, на что я интуитивно хочу, чтобы мой тест выглядел, но я не уверен, правильно ли я использую фреймворк. Как должным образом создать этот тест?

Ответ 1

Вы можете избавиться от вызова Setup и Verifiable по вашему макету. Просто используйте Verify.

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

using System;
using System.Collections.Generic;
using System.Linq;
using Moq;

namespace csharp
{
    public class SomeDTO { }

    public class SomeRepository { public virtual void Write(List<SomeDTO> list) { } }

    public class MainClass
    {
        private SomeRepository someRepository;

        public MainClass(SomeRepository someRepository) { this.someRepository = someRepository; }

        public void DoRepositoryWrite(List<SomeDTO> list) { this.someRepository.Write(list); }
    }

    class Program
    {
        static void Main(string[] args)
        {
            var mockSomeRepository = new Mock<SomeRepository>();
            var someList = Enumerable.Repeat(new SomeDTO(), 25).ToList();

            var mainClass = new MainClass(mockSomeRepository.Object);
            mainClass.DoRepositoryWrite(someList);

            mockSomeRepository.Verify(m => m.Write(It.IsAny<List<SomeDTO>>()), Times.Once(), "Write was not called");
            mockSomeRepository.Verify(m => m.Write(It.Is<List<SomeDTO>>(l => l.Count == 25)), Times.Once(), "Write was not called with a 25-element-list");
        }
    }
}

Ответ 2

Когда я разместил этот вопрос, мне не хватало нескольких важных деталей. Я хочу рассказать о том, что на самом деле происходит, если оно поможет кому-то в будущем. Мой тестируемый метод фактически очищал список, который передавался макету:

public class SomeDTO { }

public class SomeRepository
{
    public virtual void Write(IEnumerable<SomeDTO> list) { }
}

public class MainClass
{
    private readonly SomeRepository _someRepository;
    private readonly List<SomeDTO> _testList = new List<SomeDTO>(); 

    public MainClass(SomeRepository someRepository)
    {
        _someRepository = someRepository;
    }

    public void DoRepositoryWrite()
    {
        _testList.AddRange(Enumerable.Repeat(new SomeDTO(), 25));
        _someRepository.Write(_testList);
        _testList.Clear();
    }
}

class Program
{
    static void Main(string[] args)
    {
        var mockSomeRepository = new Mock<SomeRepository>();

        var mainClass = new MainClass(mockSomeRepository.Object);

        mainClass.DoRepositoryWrite();

        mockSomeRepository.Verify(m => m.Write(It.IsAny<IEnumerable<SomeDTO>>()), Times.Once(), "Write was not called");
        mockSomeRepository.Verify(m => m.Write(It.Is<IEnumerable<SomeDTO>>(l => l.Count() == 25)), Times.Once(), "Write was not called with a 25-element-list");
    }
}

Оглядываясь назад, кажется немного очевидным, но вынос здесь для меня заключается в том, что макет держится за ссылку на список, который он передал. Поэтому вам нужно быть осторожным с любыми побочными эффектами, которые изменяют этот список.

Чтобы правильно написать тест, мне нужно было сразу проверить свойства переданного списка, когда был вызван метод издевательства. Для этого я использовал обратный вызов:

int listCountAtTimeOfCall = 0;
mockSomeRepository.Setup(
    m => m.Write(It.IsAny<IEnumerable<SomeDTO>>())).Callback
        <IEnumerable<SomeDTO>>(list => listCountAtTimeOfCall = list.Count());

... do the work ...

Assert.AreEqual(listCountAtTimeOfCall, 25);