3

我正在使用MoqMocking 框架。

我有一个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>()

我对起订量和一般的单元测试相当陌生。

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

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)); }
    }
}
4

1 回答 1

3

我对你的代码试图做什么感到困惑，设计似乎有点奇怪。

首先，你遇到问题的测试是引用一个PlaceManager你没有指定的 class ，因为这是测试的一部分，很难看出哪里出了问题。

此外，我不确定 PlaceManager 的用途。您的设计具有SocialWorker提供 的属性PlaceRepository，那么PlaceManagerfor 是什么，为什么要通过 a SocialWorker？SocialWorker自己不管理PlaceRepository？

你为什么不打电话_socialWorker.PlaceRepository.Add(place)，或者更直接一点_socialWorker.Add(place)？

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

此外，您的IRepository代码有Add()一个 void，但PlaceManager返回一个object. 为什么这不是强类型的Place？看来这就是您所期望的...并且您的PlaceManager.Add()方法返回 a Place，尽管该Repository Add()方法是void. （它被坚持到上下文之后是同一个地方吗？）你PlaceManager在做其他事情Place吗？

也许我有错误的一端？您能否进一步说明您的代码的意图是什么，而不是它的作用？抱歉有这么多问题，但我想知道你的设计应该做什么，然后再弄清楚你为什么要尝试验证非虚拟......

我认为具体问题的原因是，在您的验证中，您Find期望First()对虚拟的或抽象的。

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

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

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

于 2012-04-07T13:09:49.420 回答