Moq

Question

我有一个单元测试，基本上是测试 EF Core 的行为。我正在尝试测试的 class 看起来像这样：

namespace MusicPortal.Repository.Repository
{
    public class ArtistRepository : IArtistRepository
    {
        private readonly MusicPortalDbContext _context;

        public ArtistRepository(MusicPortalDbContext context)
        {
            _context = context;
        }

        public async Task<MusicPortalDatabaseResponse<bool>> AddNewArtist(Artist artist)
        {
            try
            {
                await _context.Artists.AddAsync(new Artist
                {
                    ArtistType = ArtistTypes.Band,
                    City = artist.City,
                    Country = artist.Country,
                    Genre = artist.Genre,
                    Name = artist.Name,
                    ProfileImageUrl = artist.ProfileImageUrl
                });

                _context.SaveChanges();
                return new MusicPortalDatabaseResponse<bool>
                {
                    HasError = false,
                    Exception = null,
                    Response = true
                };
            }
            catch (Exception e)
            {
                return new MusicPortalDatabaseResponse<bool>
                {
                    HasError = true,
                    Exception = e,
                    Response = false
                };
            }
        }
    }
}

我使用 Moq

对其进行了以下单元测试

namespace MusicPortal.Tests.Repository.ArtistRepository.AddNewArtist
{
    [TestFixture]
    public class GivenAddingANewArtistToADatabaseFails
    {
        private Mock<DbSet<Artist>> _mockArtistDbSet;
        private Mock<MusicPortalDbContext> _mockContext;

        private IArtistRepository _artistRepository;
        private MusicPortalDatabaseResponse<bool> _addArtistToDbResponse;

        [OneTimeSetUp]
        public async Task Setup()
        {
            _mockArtistDbSet = new Mock<DbSet<Artist>>();
            _mockContext = new Mock<MusicPortalDbContext>();

            _mockArtistDbSet
                .Setup(x => x.AddAsync(It.IsAny<Artist>(), It.IsAny<CancellationToken>()))
                .Callback((Artist artist, CancellationToken token) => { })
                .ReturnsAsync(It.IsAny<EntityEntry<Artist>>());
            _mockContext
                .Setup(x => x.SaveChanges())
                .Throws(new Exception("Cannot save new Artist to Database"));

            _artistRepository = new MusicPortal.Repository.Repository.ArtistRepository(_mockContext.Object);
            _addArtistToDbResponse = await _artistRepository.AddNewArtist(It.IsAny<Artist>());
        }

        [Test]
        public void ThenANegativeResultIsReturned() // pass
        {
            Assert.IsFalse(_addArtistToDbResponse.Response);
            Assert.IsTrue(_addArtistToDbResponse.HasError);
            Assert.IsInstanceOf<Exception>(_addArtistToDbResponse.Exception);
        }

        [Test]
        public void ThenTheArtistContextAddMethodIsCalledOnce() //fail
        {
            _mockArtistDbSet.Verify(x => x.AddAsync(It.IsAny<Artist>(), It.IsAny<CancellationToken>()), Times.Once);
        }

        [Test]
        public void ThenTheArtistsContextSaveMethodIsNeverCalled() //pass
        {
            _mockContext.Verify(x => x.SaveChanges(), Times.Never);
        }
    }
}

第一个和最后一个断言通过但 ThenTheArtistContextAddMethodIsCalledOnce() 由于以下错误而失败：

MusicPortal.Tests.Repository.ArtistRepository.AddNewArtist.GivenAddingANewArtistToADatabaseFails.ThenTheArtistContextAddMethodIsCalledOnce

Moq.MockException : Expected invocation on the mock once, but was 0 times: x => x.AddAsync(It.IsAny(), It.IsAny())

Performed invocations:

Mock:1> (x): No invocations performed.

at Moq.Mock.Verify(Mock mock, LambdaExpression expression, Times times, String failMessage) at Moq.Mock1.Verify[TResult](Expression1 expression, Func`1 times) at MusicPortal.Tests.Repository.ArtistRepository.AddNewArtist.GivenAddingANewArtistToADatabaseFails.ThenTheArtistContextAddMethodIsCalledOnce() in MusicPortal.Tests\Repository\ArtistRepository\AddNewArtist\GivenAddingANewArtistToADatabaseFails.cs:line 53

我了解到问题代码是 c# _mockArtistDbSet .Setup(x => x.AddAsync(It.IsAny<Artist>(), It.IsAny<CancellationToken>())) .Callback((Artist artist, CancellationToken token) => { }) .ReturnsAsync(It.IsAny<EntityEntry<Artist>>());

而且我知道问题很可能是由于异步问题引起的，但我不知道为什么，也不知道实际问题是什么。有什么建议和解决方案吗？

Answer 1

您声明并设置了 _mockArtistDbSet，但没有 use/attach 它到 _mockContext。我认为您需要添加如下内容：

_mockContext.Setup(m => m.Artist).Returns(_mockArtistDbSet.Object);

Answer 2

所以看起来 EF Core 并没有那么容易地使用 SaveChangesAsync 和 AddAsync 等异步任务进行测试。最后，我遵循了用于测试 EF 核心的 MS 指南并创建了一个模拟上下文。唯一的缺点是我只能测试快乐的道路。尽管错误路径是由使用存储库的服务测试的，但我希望在存储库层上有更多的测试覆盖率。

无论如何，这是规范

using System.Linq;
using System.Threading.Tasks;
using Microsoft.EntityFrameworkCore;
using MusicPortal.Core.Context;
using MusicPortal.Core.Repository;
using MusicPortal.Tests.Repository.ArtistRepository.TestHelpers;
using MusicPortal.Tests.Repository.ArtistRepository.TestHelpers.MockDB;
using NUnit.Framework;
using MockArtistRepository = MusicPortal.Repository.Repository.ArtistRepository;

namespace MusicPortal.Tests.Repository.ArtistRepository.AddNewArtist 
{
    [TestFixture]
    public class GivenANewArtistToInsertIntoTheDb
    {
        private DbContextOptions<MusicPortalDbContext> _options;
        private MusicPortalDatabaseResponse<bool> _mockResponse;

        [OneTimeSetUp]
        public async Task Setup()
        {
            _options = new MockDbFactory("MusicPortalDB").Options;

            using (var context = new MusicPortalDbContext(_options))
            {
                var artistRepository = new MockArtistRepository(context);
                _mockResponse = await artistRepository.AddNewArtist(MockRepositoryData.Artist);
            }
        }

        [Test]
        public void AndThenAPositiveResultIsReturned()
        {
            Assert.Null(_mockResponse.Exception);
            Assert.IsTrue(_mockResponse.Response);
            Assert.IsFalse(_mockResponse.HasError);
        }

        [Test]
        public void ThenTheArtistShouldBeSavedWithNoProblem()
        {
            using (var context = new MusicPortalDbContext(_options))
            {
                Assert.AreEqual(1, context.Artists.Count());
            }
        } 
    }
}

和模拟数据库：

using System;
using Microsoft.EntityFrameworkCore;
using MusicPortal.Core.Context;
using MusicPortal.Core.DBModels;

namespace MusicPortal.Tests.Repository.ArtistRepository.TestHelpers.MockDB
{
    public sealed class MockDbFactory
    {
        public DbContextOptions<MusicPortalDbContext> Options { get; }

        public MockDbFactory(string dbName)
        {
            Options = new DbContextOptionsBuilder<MusicPortalDbContext>()
                .UseInMemoryDatabase(dbName)
                .Options;
        }

        public void AddArtistsToContext()
        {
            using (var context = new MusicPortalDbContext(Options))
            {
                context.Artists.Add(new Artist
                {
                    City = "Orange County",
                    Country = "USA",
                    Events = null,
                    Genre = "Pop Punk",
                    Id = Guid.Parse("8a07504b-8152-4d8b-8e21-74bf64322ebc"),
                    Merchandise = null,
                    Name = "A Day To Remember",
                    ArtistType = "Band",
                    ProfileImageUrl = "https://placehold.it/30x30"
                });

                context.SaveChanges();
            }
        }    
    }
}

我希望这可以帮助任何人看同样的问题。吸取的教训是，除非万不得已，否则不要使用 Async。

Moq - Mock DbSet<T>.AddAsync 抛出未执行的调用

Moq - Mock DbSet<T>.AddAsync throws no invocations performed

c#

unit-testing

ef-core-2.2