I'm fairly new to Unit Testing and Moq, but I'm attempting to setup unit tests for a generic repository using Moq and MSTest. I'm sure there's plenty of things I'm potentially doing incorrectly, but I could really use help with testing the expression based parameters of my generic repository. Specifically, I can't seem to get my mocked repository to return an expected List of objects Here's the code:
Context:
public class OrderContext : DbContext, IDisposable
{
public virtual DbSet<Order> Orders { get; set; }
public virtual void Commit()
{
base.SaveChanges();
}
}
Repository Interface / Repository
public interface IRepository<T> where T : class
{
IEnumerable<T> GetManyBySearch(Func<T, bool> where);
}
public class RepositoryBase<T> : IRepository<T> where T : class
{
private OrderContext _db;
private DbSet<T> _dbSet;
public RepositoryBase(OrderContext db)
{
_db = db;
_dbSet = _db.Set<T>();
}
// Get many records by search
public virtual IEnumerable<T> GetManyBySearch(Func<T, bool> where)
{
return _dbSet.Where(where).AsQueryable();
}
}
... And in my Unit Test, ideally I would be able to setup a test similar to this one, only I'd like to have my repository return a subset of in-memory object data based on the expression being passed to the GetManyBySearch() method. Currently, my mock repository is returning the entire data set, regardless of the expression being passed. Here's the example:
Unit Test
[TestMethod]
[TestCategory("Repository Tests")]
public void Order_Get_Many_By_Search()
{
Mock<OrderContext> mockDb = new Mock<OrderContext>();
Mock<RepositoryBase<Order>> mockRepo = new Mock<RepositoryBase<Order>>(mockDb.Object);
var data = new List<Order>()
{
new Order { OrderId = 1, OrderNumber = 111, OrderDate = DateTime.Now },
new Order { OrderId = 2, OrderNumber = 222, OrderDate = DateTime.Now },
new Order { OrderId = 3, OrderNumber = 222, OrderDate = DateTime.Now }
}.AsQueryable();
var expectedResults = data.Where(m => m.OrderNumber == 222);
var mockSet = new Mock<DbSet<Order>>();
mockSet.As<IQueryable<Order>>().Setup(m => m.Provider).Returns(data.Provider);
mockSet.As<IQueryable<Order>>().Setup(m => m.Expression).Returns(data.Expression);
mockSet.As<IQueryable<Order>>().Setup(m => m.ElementType).Returns(data.ElementType);
mockSet.As<IQueryable<Order>>().Setup(m => m.GetEnumerator()).Returns(data.GetEnumerator());
mockDb.Setup(m => m.Set<Order>()).Returns(mockSet.Object);
mockRepo.Setup(m => m.GetManyBySearch(It.IsAny<Func<Order, bool>>()))
.Returns(mockSet.Object.AsQueryable());
List<Order> results = mockRepo.Object.GetManyBySearch(m => m.OrderNumber == 222).ToList(); // This returns the entire "data" dataset
mockRepo.Verify(m => m.GetManyBySearch(It.IsAny<Func<Order, bool>>()), Times.Once());
Assert.AreEqual(2, results.Count()); // This fails :(
}
Any help would be greatly appreciated. Thanks in advance.
GetManyBySearch
on that. That just tests that Moq does what you tell it to... Also, there are 3 objects in the list and you're asserting 2! – Patrick Quirk Oct 9 '14 at 19:27mockRepo
with your actual implementation, that'd be a good test. – Patrick Quirk Oct 9 '14 at 19:49