与上调用SaveChanges

问题描述:

我找到一份像样的文章,让我开始单元测试使用起订量我的基于框架的实体应用程序时出现验证惩戒EF6:https://msdn.microsoft.com/en-us/data/dn314429.aspx与上调用SaveChanges

我遇到这个问题是的的SaveChanges方法模拟似乎没有像通常那样触发ValidateEntity方法。我在EntityTypeConfiguration中配置的验证设置都不是DbEntityValidationException

例如,我的AddRoles_Fails_For_Empty_Name测试以确保服务不能添加名称为空的角色。 IsRequired()配置未被应用,或者ValidateEntity方法未被调用。我应该提到,如果我在Web应用程序中使用实际的上下文,它可以正常工作。

我在下面列出了一些相关的单元测试,DbContext和服务代码。

我做错了什么吗?是否有任何已知问题或解决方法?

角色DB地图

public class RoleMap : EntityTypeConfiguration<Role> 
{ 
    public RoleMap() 
    { 
     ToTable("bm_Roles"); 
     HasKey(r => r.Id); 
     Property(r => r.Name).IsRequired().HasMaxLength(100).HasIndex(new IndexAttribute("UX_Role_Name") { IsUnique = true }); 
     Property(r => r.Description).HasMaxLength(500); 
    } 
} 

的DbContext

public class BlueMoonContext : DbContext, IBlueMoonContext 
{ 
    public BlueMoonContext() : base("name=BlueMoon") 
    { 

    } 

    public DbSet<Role> Roles { get; set; } 
    public DbSet<User> Users { get; set; } 

    protected override void OnModelCreating(DbModelBuilder modelBuilder) 
    { 
     base.OnModelCreating(modelBuilder); 
     modelBuilder.Configurations.AddFromAssembly(typeof(BlueMoonContext).Assembly); 
    } 

    public void MarkAsModified<T>(T entity) where T : class 
    { 
     entity.ThrowIfNull("entity"); 
     Entry<T>(entity).State = EntityState.Modified; 
    } 

    protected override DbEntityValidationResult ValidateEntity(DbEntityEntry entityEntry, IDictionary<object, object> items) 
    { 
     var result = base.ValidateEntity(entityEntry, items); 

     if (entityEntry.State == EntityState.Added || entityEntry.State == EntityState.Modified) 
     { 
      // Perform validations that require database lookups 
      if (entityEntry.Entity is Role) 
      { 
       ValidateRole((Role)entityEntry.Entity, result); 
      } 
      else if (entityEntry.Entity is User) 
      { 
       ValidateUser((User)entityEntry.Entity, result);      
      } 
     } 

     return result; 
    } 

    private void ValidateRole(Role role, DbEntityValidationResult result) 
    { 
     if (role.Name.HasValue() && !Roles.NameAvailable(role.Name, role.Id)) 
     { 
      result.ValidationErrors.Add(new DbValidationError("Name", "Already in use")); 
     } 
    } 

    private void ValidateUser(User user, DbEntityValidationResult result) 
    { 
     if (user.UserName.HasValue() && !Users.UserNameAvailable(user.UserName, user.Id)) 
     { 
      result.ValidationErrors.Add(new DbValidationError("UserName", "Already in use")); 
     } 
     if (user.Email.HasValue() && !Users.UserNameAvailable(user.UserName, user.Id)) 
     { 
      result.ValidationErrors.Add(new DbValidationError("Email", "Already in use")); 
     } 
    } 
} 

账户服务

public class AccountService : BaseService, IAccountService 
{ 
    private IPasswordHasher _passwordHasher; 

    public AccountService(IBlueMoonContext context, IPasswordHasher passwordHasher) : base(context) 
    { 
     _passwordHasher = passwordHasher; 
    } 

    public ServiceResult CreateRole(Role role) 
    { 
     role.ThrowIfNull("role"); 
     Context.Roles.Add(role); 
     return Save(); 
    } 

    // Copied from base service class 
    protected ServiceResult Save() 
    { 
     var result = new ServiceResult(); 
     try 
     { 
      Context.SaveChanges(); 
     } 
     catch (DbEntityValidationException validationException) 
     { 

      foreach (var validationError in validationException.EntityValidationErrors) 
      { 
       foreach (var error in validationError.ValidationErrors) 
       { 
        result.AddError(error.ErrorMessage, error.PropertyName); 
       } 
      } 
     } 
     return result; 
    } 
} 

单元测试

[TestFixture] 
public class AccountServiceTests : BaseTest 
{ 
    protected Mock<MockBlueMoonContext> _context; 
    private IAccountService _accountService; 

    [TestFixtureSetUp] 
    public void Setup() 
    { 
     _context = new Mock<BlueMoonContext>(); 

     var data = new List<Role> 
     { 
      new Role { Id = 1, Name = "Super Admin" }, 
      new Role { Id = 2, Name = "Catalog Admin" }, 
      new Role { Id = 3, Name = "Order Admin" } 
     }.AsQueryable(); 

     var roleSet = CreateMockSet<Role>(data); 
     roleSet.Setup(m => m.Find(It.IsAny<object[]>())).Returns<object[]>(ids => data.FirstOrDefault(d => d.Id == (int)ids[0])); 

     _context.Setup(m => m.Roles).Returns(roleSet.Object); 
     // _context.Setup(m => m.SaveChanges()).Returns(0); 
     _accountService = new AccountService(_context.Object, new CryptoPasswordHasher()); 
    } 

    [Test] 
    public void AddRole_Fails_For_Empty_Name() 
    { 
     var role = new Role { Id = 4, Name = "" }; 

     var result = _accountService.CreateRole(role); 
     Assert.False(result.Success); 
    } 
} 

SaveChangesvirtual方法,这意味着你调用一个方法假....

您可以创建模拟CallBase = true,但它不是一个好主意(它错过了主意UT的):

_context = new Mock<BlueMoonContext>(){ CallBase = true }; 

上面的代码将使用实际实施BlueMoonContext任何方法/属性这是不明确地设置。

RoleMap负责你的数据库结构,你应该测试它作为集成测试(使用数据库)的一部分。

在我看来,你应该创建一个集成测试,以验证完整性(例如,覆盖RoleMap)与您的数据库,并采用Throw建立覆盖闭锁段创建UT(这是你的单元的一部分):

_contest.Setup(x => x.SaveChanges()) 
      .Throws(new DbEntityValidationException()); 

编辑回答问题OP在评论

不,你不必分开内置的验证,你必须创建另一个测试(集成测试)。在此测试中,您将验证验证行为:插入一个非法实体,期望异常会提升(使用ExpectedExceptionAttribute),然后验证数据库是否为空...应用此行为使用这种模式:

try 
{ 
    \\... 
    \\try to commit 
} 
catch(DbEntityValidationException ex) 
{ 
    \\do some validation, then: 
    throw;\\for ExpectedExceptionAttribute 
} 

我看着的EntityTypeConfiguration的API,我没有看到任何接触它允许UT规则(除非你使用的工具,如MsFakesTypeMock Isolator没有办法验证ToTable/HasKey/Property被调用)。该课程正在EntityFramework(这是BCL的一部分)中使用,您不必验证EntityFramework是否正常工作,您将验证您的自定义规则是否已经集成并按预期工作(在this answer你可以阅读不测试BCL类的原因)。因此在AccountService的UT中使用Moq。为BlueMoonContextRoleMap(不含Moq)创建集成测试。

顺便说@LadislavMrnka提供一个interesting way to test(integration test)EntityTypeConfiguration

+0

虽然我同意RoleMap确实是负责数据库的结构,这也让我像强制执行'Required'和'StringLength'简单的验证约束。为了“正确”进行单元测试,是否需要剥离内置的验证并将其放入其自己的层?或者我会不要使用Moq,只是创建另一个'IBlueMoonContext'的实现,它不会触及数据库进行测试? – Sam

+0

@Sam的答案太长以至于无法发表评论,所以我编辑了答案...... –

+0

所以,我最终做的是将验证与SaveChanges()方法分开。我通过在构造函数中设置了'Configuration.ValidateOnSaveEnabled = false;'来实现这一点。然后我创建了一个轻量级的实体验证器类,它封装了DataAnnotations'Validator'类并将其注入到我的服务中。 – Sam