c# - 在测试实现时，我采用的方法是通用的还是我做错了？

Question

好的，我有下面的模块，它从表中返回一个用户 ID 列表，其中 ID 与正则表达式匹配:

public sealed class UserIdListRetriever : IUserIdListRetriever
{
    private readonly EntityFrameworkClass _databaseConnection;

    public UserIdListRetriever(EntityFrameworkClass databaseConnection)
    {
        _databaseConnection = databaseConnection;
    }

    public IEnumerable<string> Retrieve()
    {
        var salesAgents = _databaseConnection
            .tblAccounts
            .Select(account => account.UserId)
            .Distinct();

        var regex = new Regex(@"(?<=\\)(.*?)(?=\:)");

        return (from agent in salesAgents
                .AsEnumerable()
                select regex.Match(agent)
                into match
                where match.Success
                select match.Value.ToUpper())
                .OrderBy(match => match);
    }
}

这是界面:

public interface IUserIdListRetriever
{
    IEnumerable<string> Retrieve();
}

我一直读到我应该测试行为，而不是实现，但我在这里关心的是我的类是否返回准确的用户 ID 列表。

我可以创建一个 IUserIdListRetriever 的模拟实现，并可能在我的单元测试中断言我得到一个不为空的字符串的 IEnumerable，但这不会测试我的 LINQ 是否正确，或者是否或者我的 Regex 是否正确，感觉没那么有用。

我觉得好像这是这里重要的两件事(无论我的 LINQ 是否正确，以及我的 Regex 是否正确)，我最终得到这样的测试类:

using System.Linq;
using Microsoft.VisualStudio.TestTools.UnitTesting;
etc etc

namespace myNamespaceTests
{
    [TestClass]
    public class UserIdListRetrieverTests
    {
        [TestMethod]
        public void UserIdListRetrieverReturnsAccurateInformation()
        {
            var databaseConnection = 
                new EntityFrameworkClass("connection;string;");

            var userIdListRetriever = new UserIdListRetriever(databaseConnection );

            var userIds = userIdListRetriever.Retrieve();

            /*
             * I put a breakpoint here, 
             * and run the same query in SQL Management studio 
             * to make sure I have the same results
            */
             Assert.IsTrue(userIds.Any());
        }
    }
}

这感觉很不对，但从我的角度来看，我发现这是最有用的，因为它仍然允许我快速测试(尽管不是那么快)该模块正在执行我想要它执行的操作。

我有很多这样的模块，我的代码仍然是模块化的和可测试的，但只有当我花一点时间手动运行单元测试、逐步执行每个测试并对数据库运行查询时，我才发现这些测试有用手动验证我的数据检索模块返回给我的信息是否符合我的预期。在此之后，我可以自信地说，我的代码库中的每个模块都在做我想要它做的事情。

我不认识其他人以这种方式工作，这通常是一个不好的迹象(是我错了，还是其他人都错了？)。知识渊博的人可以解释我在这里出错的地方并解释他们如何测试上面的类，以一种他们可以快速运行测试并且这些测试是自动化的方式，但他们可以自信地说他们的每个模块都有预期的行为？

谢谢

Answer 1

我一般认为，如果涉及到数据库，就不是单元测试了。话虽如此，多年来我一直在研究同一个问题，但我无法为您提供更优雅的解决方案来测试您的数据检索语句。

我会注意到此代码不遵守单一职责原则 - 它从 EF 源检索数据，然后进一步过滤它。您可以做的是将这段代码分成两部分:一部分用于检索列表，另一部分用于检查列表中是否有与您的正则表达式匹配的字符串。然后，您可以轻松设置单元测试以验证正则表达式是否按预期工作。