Question

我正在使用Moq框架进行模拟。

我有一个SocialWorker来自abstract DataWorker类。

SocialWorker引用了几个Repositories作为virtual property

我正在尝试运行以下测试：

private Place _place;
private Mock<IRepository<Resources.Data.Place>> _placeRepositoryMock;
private Mock<SocialWorker> _socialWorkerMock;

[SetUp]
public void SetUp()
{
    _place = new Place {Name = "A"};

    _socialWorkerMock = new Mock<SocialWorker> {DefaultValue = DefaultValue.Mock};

    IRepository<Resources.Data.Place> placeRepository = _socialWorkerMock.Object.PlaceRepository;

    _placeRepositoryMock = Mock.Get(placeRepository);

    _placeRepositoryMock.Setup(
        repository =>
        repository.Find(It.IsAny<Expression<Func<Resources.Data.Place, bool>>>(), null, null))
        .Returns(new[] {new Resources.Data.Place()});
}

[Test]
public void AddShouldAddANewPlace()
{
    var placeManager = new PlaceManager(_socialWorkerMock.Object);

    object placeEntity = placeManager.Add(_place);

    placeEntity.GetType().Should().Equal(typeof (Resources.Data.Place));

    _socialWorkerMock.Verify(
        socialWorker => socialWorker.PlaceRepository.Add(It.IsAny<Resources.Data.Place>()), Times.Once());

    _placeRepositoryMock.Verify(
        placeRepository =>
        placeRepository.Find(p => p.Name.Equals(_place.Name), null, null).First(),
        Times.Once());
}

此测试的最后一次验证失败，并显示以下错误：

System.NotSupportedException : Invalid verify on a non-virtual (overridable in VB) member: placeRepository => placeRepository.Find(p => p.Name.Equals(._place.Name), null, null).First<Place>()

我对Moq和单元测试都很新。

以下是供您参考的相关代码：

IUnitOfWork

public interface IUnitOfWork : IDisposable
{
    void CommitChanges();
}

IRepository

public interface IRepository<T>
{
    void Add(T entity);

    void Delete(T entity);

    IEnumerable<T> Find(Expression<Func<T, bool>> filter = null,
                        Func<IQueryable<T>, IOrderedQueryable<T>> orderBy = null,
                        IList<string> includedProperties = null);

    T FindById(object id);

    void Update(T entity);
}

DataWorker

public abstract class DataWorker : IUnitOfWork
{
    protected ObjectContext ObjectContext;
    private bool _disposed;

    protected DataWorker()
    {
        ObjectContext = new ObjectContext(ConfigurationManager.ConnectionStrings["DataEntities"].ConnectionString);
    }

    ~DataWorker()
    {
        . . .
    }

    protected virtual void Dispose(bool disposing)
    {
        . . .
    }

    public virtual void CommitChanges()
    {
        ObjectContext.SaveChanges();
    }

    public void Dispose()
    {
        . . .
    }
}

DataRepository

public class DataRepository<T> : IRepository<T> where T : EntityObject
{
    private readonly ObjectSet<T> _objectSet;

    public DataRepository(ObjectContext objectContext)
    {
            _objectSet = objectContext.CreateObjectSet<T>();
    }

    public void Add(T entity)
    {
        . . .
    }

    public void Delete(T entity)
    {
        . . .
    }

    public virtual IEnumerable<T> Find(Expression<Func<T, bool>> filter = null,
                               Func<IQueryable<T>, IOrderedQueryable<T>> orderBy = null,
                               IList<string> includedProperties = null)
    {
        . . .
    }

    public T FindById(object id)
    {
        . . .
    }


    public void Update(T entity)
    {
        . . .
    }
}

SocialWorker

public class SocialWorker : DataWorker
{
    private IRepository<ContactRequest> _contactRequestRepository;
    private IRepository<Place> _placeRepository;
    private IRepository<User> _userRepository;

    public virtual IRepository<ContactRequest> ContactRequestRepository
    {
        get
        {
            return _contactRequestRepository ??
                   (_contactRequestRepository = new DataRepository<ContactRequest>(ObjectContext));
        }
    }

    public virtual IRepository<Place> PlaceRepository
    {
        get { return _placeRepository ?? (_placeRepository = new DataRepository<Place>(ObjectContext)); }
    }

    public virtual IRepository<User> UserRepository
    {
        get { return _userRepository ?? (_userRepository = new DataRepository<User>(ObjectContext)); }
    }
}

Answer 1

我对你的代码试图做的事感到困惑，设计看起来有点奇怪。

首先，你遇到问题的测试是引用一个你没有指定的类PlaceManager，而这是测试的一部分，很难看出它出错的地方。

此外，我不确定PlaceManager的用途。您的设计有SocialWorker，其中包含提供PlaceRepository的属性，那么PlaceManager是什么，为什么它会传递给SocialWorker？不SocialWorker管理自己的PlaceRepository？

为什么不打电话给_socialWorker.PlaceRepository.Add(place)，或者更整洁地_socialWorker.Add(place)？

我只是猜测PlaceManger访问SocialWorker.PlaceRepository，但我不明白为什么SocialWorker.PlaceRepository是公共财产，没有什么可以阻止任何人直接使用而忽略PlaceManager。如果PlaceManager也是Repository的某种Place，那就是这样。

此外，您的IRepository代码为Add()为空，但PlaceManager返回object。为什么Place没有强类型？这似乎是您所期待的......而PlaceManager.Add()方法返回Place，但Repository Add()方法为void。（它是否一直存在于上下文之后是相同的位置？）PlaceManager是否正在使用Place执行其他操作？

也许我有错误的结局？你能澄清一下代码的意图，它的作用是什么吗？很抱歉这么多问题，但我想知道你的设计应该做些什么，然后才弄清楚你为什么要尝试验证非虚拟......

我认为具体问题的原因是，在您的验证中，您希望对Find进行验证，但是，这包括对First()的调用，需要对其进行模拟，不能，因为它是静态扩展方法，不是虚拟或抽象的。

无论如何，我不确定你为什么要反对First()。您要验证的方法只是：

_placeRepositoryMock.Verify(placeRepository =>
   placeRepository.Find(p => p.Name.Equals(_place.Name), null, null),
   Times.Once());

这就是实际调用的内容，因为它是存储库中方法的签名。您只能检查是否调用了接口方法。

Moq，抽象类和虚拟属性

1 个答案: