c# - 最小起订量，抽象类和虚拟属性

我正在使用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)); }
    }
}

最佳答案

我对您的代码要做什么感到困惑，设计似乎有些奇怪。

首先，您遇到问题的测试引用了一个尚未指定的类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());

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

关于c# - 最小起订量，抽象类和虚拟属性，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/10050168/