如何在 tdd 中重构测试？

Question

我正在执行此 TDD 型练习：http://osherove.com/tdd-kata-1

我生成了以下代码（本练习中的第 1 点到第 5 点 - 我对其进行了单元测试）：

public class StringCalculator
{
    private readonly string[] _defaultSeparators = { ",", "\n" };

    public int Add(string numbers)
    {
        // Parser section (string to list of ints)
        var separators = _defaultSeparators;

        var isSeparatorDefinitionSpecified = numbers.StartsWith("//");
        if (isSeparatorDefinitionSpecified)
        {
            var endOfSeparatorDefinition = numbers.IndexOf('\n');

            var separator = numbers.Substring(2, endOfSeparatorDefinition - 2);

            numbers = numbers.Substring(endOfSeparatorDefinition);
            separators = new[] { separator };
        }

        var numbersArray = numbers.Split(separators, StringSplitOptions.RemoveEmptyEntries);
        var numbersArrayAsInts = numbersArray.Select(int.Parse).ToArray();

        // Validator section
        var negativeNumbers = numbersArrayAsInts.Where(c => c < 0).ToArray();
        if (negativeNumbers.Any())
        {
            throw new Exception(string.Format("negatives not allowed ({0})", string.Join(", ", negativeNumbers)));
        }

        return numbersArrayAsInts.Sum();
    }
}

现在我想把代码重构成这样：

public int Add(string numbers)
{
    var numbersAsInts = CalculatorNumbersParser.Parse(numbers);

    CalculatorNumbersValidator.Validate(numbersAsInts);

    return numbersAsInts.Sum();
}

我应该如何规划重构以正确重构我的代码和单元测试？

我认为我应该将部分测试移至新创建的实现类测试（CalculatorNumbersParserTests 和 CalculatorNumbersValidatorTests），更改一些现有测试并添加用于 Parse 和 Validate 方法执行的测试。

但是在不破坏测试的情况下执行此操作的正确方法是什么？

Answer 1

我会警告不要移动测试，因为如果您这样做，那么您的测试将与实现相关联，这意味着它们非常脆弱，因此每次您想要更改您的实现时都必须更改您的测试。当您拥有庞大的代码库时，这很快就会变得昂贵，并且可能成为进行更改的阻碍因素。

您现有的测试应该指定您的字符串计算器的行为，这样您就可以将您的实现重构为任何东西，只要您保持所需的行为。

我倾向于将一个单元视为 'unit of behaviour'，可能需要几个类才能实现。

如果将一些类放在不同的程序集中，情况可能会发生变化，此时您可能希望在新程序集旁边进行一些新测试，以确保这些组件的行为是正确的没有意外改变，但在这种情况下，我怀疑你会这样做。

如果您开始在多个地方重用类，情况也可能会发生变化，此时您可能需要单独的测试来指定类的行为，而与它们在这些地方的使用无关.

Answer 2

我认为@Sam Holder 已经涵盖了大部分内容。我要补充的一件事是，当您重构代码时，您应该将您创建的任何类标记为 internal（我假设你使用的是 .net)，这样它们在它们所在的程序集之外是不可见的。

我倾向于认为 public 类应该单独进行测试，因为它们可以通过引用您的程序集的其他代码轻松实例化。另一方面，internal 类可以被认为是实现细节，通常可以通过程序集 public 接口进行测试。当然也有例外，这取决于你在做什么/代码的复杂性等。但这是我的一般规则。

如何在 tdd 中重构测试？

How to refactor tests in tdd?

tdd

unit-testing