用于存储过程方法的C＃单元测试

Question

我使用实体库代替实体框架来执行数据操作。

以下是我的示例BL方法，该方法用于更新人的文化。我不知道如何为这种方法编写单元测试。

public static Culture UpdatePersonCulture(Guid userId, Culture culture)
{
    try
    {
        if (userId == Guid.Empty)
            throw new exception("User id should not be empty");

        DatabaseProviderFactory factory = new DatabaseProviderFactory();
        Database database = factory.CreateDefault();

        using (DbConnection dbConnection = database.CreateConnection())
        {
            if (dbConnection.State == ConnectionState.Closed)
                dbConnection.Open();

            DbCommand dbCommand = database.GetStoredProcCommand("usp_UpdatePersonCulture");
            database.DiscoverParameters(dbCommand);
            dbCommand.Parameters["@userId"].Value = userId;
            dbCommand.Parameters["@cultureId"].Value = culture.Id;

            DataSet dataSet = database.ExecuteDataSet(dbCommand);

            return BLDataPopulation.PopulateCultures(dataSet.Tables[0]).FirstOrDefault();
        }
    }
    catch (Exception exception)
    {
        throw BLLogger.Log(exception);
    }
}

Answer 1

将您的班级更改为可单元测试的

单元测试意味着在给定输入的情况下，我的代码将以已知的行为运行，并产生预先已知的输出。它应该快速执行，因为它经常执行（甚至可以在代码更改时执行，就像NCrunch一样）。单元测试范围应该是一个类。

集成测试是您在更大范围内进行测试的测试。您可以测试几个类，部分程序甚至不同的程序是否可以很好地协同工作。但是，这些测试的稳定性（因为它可能依赖于可能会不时出现故障的外部组件），“工作正常”的定义（最好保持模糊）以及此类测试的维护成本，使得这些测试的附加值更高。集成测试真的很低。实际上，由于维护成本，我经常看到没有任何项目甚至没有进行集成测试的项目。

但是，您可以将不可测试的代码限制为最小：

• 将接口与实现分开：在接口后面隔离外部资源（例如文件I / O或数据库）访问，从而隐藏实现复杂性。

• 避免使用静态代码，因为无法对静态代码进行单元测试。实现是使用静态类绑定的。如果静态代码调用外部资源，那么您将无法控制输入和输出，因此无法测试结果。

• 学习重构：如果您掌握相关技能，则可以轻松处理旧版代码。隔离一小段代码，为其编写单元测试，然后用功能更强大的代码替换。

• 首先进行测试或测试驱动的开发，TDD可以使新编写的代码从一开始就可以进行测试。该过程涉及在编写类的实现之前编写对给定类的期望。

• 行为驱动的开发或BDD：有时候，当您需要的只是客户需求时，就无法轻松地达到单元测试所需的最低级别。您可以使用 Specflow之类的工具来构建需求语法并将其转换为测试。

为了成功编写单元测试并重构您的遗留不可测代码，您将需要IOC工具（因为构造函数签名会随着这些技术的发展而发展）和Mock框架（如Moq），以模拟在不绑定的情况下添加的接口他们任何实现。

例如，在您的情况下，我重构了包含要测试的方法的类：

public class ClassToTest : IInterfaceToTest
{
    private readonly IBLLogger _logger;
    private readonly IBLDataPopulation _blDataPopulation;
    private readonly IDatabase _database;

    public ClassToTest(IBLLogger logger, IBLDataPopulation blDataPopulation, IDatabase database)
    {
        _logger = logger;
        _blDataPopulation = blDataPopulation;
        _database = database;
    }

    // this cannot be static if you want
    // to unit test the classes depending on this one.
    public Culture UpdatePersonCulture(Guid userId, Culture culture)
    {
        try
        {
            if (userId == Guid.Empty)
                throw new Exception("User id should not be empty");

            var dataSet = _database.ProvideCultureMappings(userId, culture); 

            return _blDataPopulation.PopulateCultures(dataSet.Tables[0]).FirstOrDefault();

        }
        catch (Exception exception)
        {
            throw _logger.Log(exception);
        }
    }
}

我添加的接口：

public interface IDatabase
{
    DataSet ProvideCultureMappings(Guid userId, Culture culture);
}

public interface IBLLogger
{
    Exception Log(Exception exception);
}

public interface IBLDataPopulation
{
    IEnumerable<Culture> PopulateCultures(DataTable dataTable);
}

结果：

[TestFixture]
public class DatabaseUnitTest
{
    [Test]
    public void Test()
    {
        var logger = new Mock<IBLLogger>(MockBehavior.Strict);
        var blDataPopulation = new Mock<IBLDataPopulation>(MockBehavior.Strict);
        var database = new Mock<IDatabase>(MockBehavior.Strict);
        var toTest = new ClassToTest(logger.Object, blDataPopulation.Object, database.Object);
        var userId = Guid.NewGuid();
        var culture = new Culture{Id = "MyId"};
        var dataSet = new DataSet();
        var table = new DataTable();
        dataSet.Tables.Add(table);
        database.Setup(x => x.ProvideCultureMappings(userId, culture)).Returns(dataSet);
        var cultureList = new List<Culture> {culture, new Culture {Id = "AnotherCulture"}};
        blDataPopulation.Setup(x => x.PopulateCultures(table)).Returns(cultureList);
        var result = toTest.UpdatePersonCulture(userId, culture);
        Assert.AreEqual(result.Id, culture.Id);
    }
}

作为奖励，不可测类：

public class Database : IDatabase
{
    public DataSet ProvideCultureMappings(Guid userId, Culture culture)
    {

        DatabaseProviderFactory factory = new DatabaseProviderFactory();
        Database database = factory.CreateDefault();

        using (DbConnection dbConnection = database.CreateConnection())
        {
            if (dbConnection.State == ConnectionState.Closed)
                dbConnection.Open();

            DbCommand dbCommand = database.GetStoredProcCommand("usp_UpdatePersonCulture");
            database.DiscoverParameters(dbCommand);
            dbCommand.Parameters["@userId"].Value = userId;
            dbCommand.Parameters["@cultureId"].Value = culture.Id;

            DataSet dataSet = database.ExecuteDataSet(dbCommand);
            return dataSet;
        }
    }

    private DataSet ExecuteDataSet(DbCommand dbCommand)
    {
        throw new NotImplementedException();
    }

    private void DiscoverParameters(DbCommand dbCommand)
    {
        throw new NotImplementedException();
    }

    private DbCommand GetStoredProcCommand(string uspUpdatepersonculture)
    {
        throw new NotImplementedException();
    }

    private DbConnection CreateConnection()
    {
        throw new NotImplementedException();
    }
}