8

次のようなメソッドの単体テストを作成しようとしています。

public int Save(IEnumerable<int> addedIds, IEnumerable<int> removedIds)
{
    var existingIds = repository.Get();
    IEnumerable<int> ids = existingIds.Except(removedIds).Union(addedIds));
    return repository.Create(ids);
}

Moq でのテストは次のようになります。

repository.Setup(r => r.Get()).Returns(CreateList());
service.Save(addedIds, removedIds);
repository.Verify(r => r.Create(It.Is<IEnumerable<int>>(l => VerifyList(l))));

これは次のエラーで失敗し、VerifyList()呼び出されません。

モックで少なくとも 1 回の呼び出しが期待されていましたが、実行されませんでした:

r => r.Create(It.Is<IEnumerable'1>(list => VerifyList(list)))

実行された呼び出し:

IRepo.Create(System.Linq.Enumerable+<UnionIterator>d__88'1[System.Int32])

呼び出された型は is ではなくIEnumerable<int>、実際には is であるSystem.Linq.Enumerable+<UnionIterator>d__88'1[System.Int32])ため、テストは失敗します。(テストをステップ実行すると、すべてが正しく行われ、結果は期待どおりです)

ids.ToList()テスト対象のメソッドを呼び出すと、次の結果が得られます。

モックで少なくとも 1 回の呼び出しが期待されていましたが、実行されませんでした:

r => r.Create(It.Is<List'1>(l => VerifyList(l)))

実行された呼び出し: IRepo.Create(System.Collections.Generic.List'1[System.Int32])

これを回避する方法はありますか？それとも私は何か間違ったことをしていますか？

編集：VerifyListメソッドに誤りがあったことが判明したため、falseが返されましたが、Moqはその情報を提供していませんでした。種類の違いはアカニシンです。

4

3 に答える 3

11

これはうまくいくようです。ただし、いくつかの仮定を行いました。VerifyList メソッドの方が優れていると思います。=)

[Test]
    public void Test()
    {
        // SETUP
        Mock<IRepository> repository = new Mock<IRepository>();
        Service service = new Service(repository.Object);
        repository.Setup(r => r.Get()).Returns(CreateList());

        IEnumerable<int> addedIds = new[]{1,2};
        IEnumerable<int> removedIds = new[]{3,4};
        service.Save(addedIds, removedIds);

        repository.Verify(r => r.Create(It.Is<IEnumerable<int>>(l => VerifyList(l))));
    }

    private static bool VerifyList(IEnumerable<int> enumerable)
    {
        return enumerable.Contains(1) && enumerable.Contains(2) && enumerable.Contains(5);
    }

    private IEnumerable<int> CreateList()
    {
        return new[] { 3, 4, 5 };
    }

    public interface IRepository
    {
        IEnumerable<int> Get();
        int Create(IEnumerable<int> id);
    }
    public class Service
    {
        public Service(IRepository repository)
        {
            this.repository = repository;
        }

        private IRepository repository;

        public int Save(IEnumerable<int> addedIds, IEnumerable<int> removedIds)
    {
        var existingIds = repository.Get();
            IEnumerable<int> ids = existingIds.Except(removedIds).Union(addedIds);

        return repository.Create(ids);
    }
于 2013-01-24T12:36:20.757 に答える
2

素早く汚いもの -

public interface IBlah
{
    void Sum(IEnumerable<int> baz);
}

class Blah : IBlah
{
    public void Sum(IEnumerable<int> baz)
    {
        return;
    }
}

public class Baz
{
    private readonly IBlah blah;

    public Baz(IBlah blah)
    {
        this.blah = blah;
    }

    public void Sum(IEnumerable<int> baz)
    {
        blah.Sum(baz);
    }
}

そして、次のようにテストします-

[Test]
public void foo()
{
    var mock = new Mock<IBlah>();
    var enumerable = Enumerable.Range(1, 10);

    var baz = new Baz(mock.Object);
    baz.Sum(enumerable.Where(x => x%2 == 0));

    mock.Verify(p => p.Sum(It.Is<IEnumerable<int>>(z => z.All(x => x%2==0))));
}
于 2013-01-24T12:34:48.353 に答える