毫无意义的单元测试

Question

我有一个 baseService 类，我的大多数服务都继承自该类，如下所示。

public abstract class BaseService<T> : IBaseService<T>
    where T : class, IBaseEntity
{
    protected IDataContext _context;
    protected IValidator<T> _validator = null;

    protected BaseService(IDataContext context)
    {
        _context = context;
    }

    protected BaseService(IDataContext context, IValidator<T> validator)
        : this(context)
    {
        _validator = validator;
    }

    public virtual async Task<ICollection<T>> GetAllAsync()
    {
        return await _context.Set<T>().ToListAsync();
    }

    public virtual Task<T> GetAsync(long id)
    {
        return _context.Set<T>().Where(e => e.Id == id).FirstOrDefaultAsync();
    }

    public virtual Task<ValidationResult> ValidateAsync(T t)
    {
        if (_validator == null) throw new MissingFieldException("Validator does not exist for class " + t.GetType().ToString() + ". override method if no validation needed");
        return _validator.ValidateAsync(t);
    }

    public virtual async Task<int> AddAsync(T t)
    {
        var results = await ValidateAsync(t);

        if (!results.IsValid) {
            throw new ValidationException(results.Errors);
        }

        if (_context.GetState(t) == EntityState.Detached)
        {
            _context.Set<T>().Add(t);
            _context.SetState(t, EntityState.Added);
        }

        return await _context.SaveChangesAsync();
    }

    public virtual async Task<int> UpdateAsync(T updated)
    {
        var results = await ValidateAsync(updated);

        if (!results.IsValid)
        {
            throw new ValidationException(results.Errors);
        }

        if (_context.GetState(updated) == EntityState.Detached)
        {
            _context.SetState(updated, EntityState.Modified);
        }

        return await _context.SaveChangesAsync();
    }

    public virtual Task<int> DeleteAsync(T t)
    {
        _context.SetState(t, EntityState.Deleted);

        return _context.SaveChangesAsync();
    }
}

我是否认为在实现此服务的每个类中对它进行单元测试是没有意义的？但是，在我的集成测试中测试每个测试的功能？

Answer 1

如果您选择composition over inheritance，则可以避免该决定，也就是说，您的服务不是从 BaseService 继承，而是将它们创建为具有 IBaseService 作为依赖项，例如

public class MyService 
{
  private readonly IBaseService<SomeBaseIdentity> _service;

  public MyService(IBaseService<SomeBaseIdentity> service)
  {
    _service = service;
  }

  //.... methods that use _service
}

这样您就可以使用隔离框架（Rhino Mocks、Moq 等）测试 MyService，而无需担心 IBaseService 的实际实现。

关于选择组合而不是继承的主题已经写了很多。其中绝大多数意味着它是一种优越的方法（主要是因为它更灵活，即如果您使用组合，您可以交换实现，而如果您使用继承，则所有派生类都与基类相关联，因为发生了变化到基类影响所有派生类），Java 语言创建者说如果可以，他会disallow it in Java。