装饰器模式的单元测试

Question

我最近使用装饰器模式解决了我的问题之一。一切工作正常，一切都足够解耦（或者我认为如此），我能够单独对每个有效字段进行单元测试。

我的问题是，NameValidator 和 AgeValidator 是否都通过了 Validate() 和 IsValid() （抽象）函数的测试。我是否还需要对我的 ValidationDecorator 类（尚未创建）进行单元测试？ ValidationDecorator 将负责用每个验证类装饰我的验证器。

public abstract class FieldValidator
{
    protected IMessage validateReturnType;

    public FieldValidator() { }

    public bool IsValid()
    {
        return (validateReturnType.GetType() == typeof(Success));
    }
}

public class NameValidator : FieldValidator, IValidator
{
    private string name;

    public NameValidator(string _name) { 
        name = _name;
    }

    public IMessage Validate()
    {
        if (name.Length < 5)
        {
            validateReturnType = new Error("Name error.");
        }
        else
        {
            validateReturnType = new Success("Name no errror.");
        }

        return validateReturnType;
    }
}

public class AgeValidator : FieldValidator, IValidator
{
    private int age;

    public AgeValidator(int _age)
    {
        age = _age;
    }

    public IMessage Validate()
    {
        if (age <= 18)
        {
            validateReturnType = new Error("Age error.");
        }
        else
        {
            validateReturnType = new Success("Age no errror.");
        }

        return validateReturnType;
    }
}

public interface IValidator
{
    IMessage Validate();
    bool IsValid();
}

这是我的单元测试。

[TestFixture]
public class ValidatorTest
{
    Type successType;
    Type errorType;

    Model m;

    [SetUp]
    public void SetUp()
    {
        successType = typeof(Success);
        errorType = typeof(Error);

        m = new Model();
        m.Name = "Mike Cameron";
        m.Age = 19;
        m.Height = 325;

        Validator v = new Validator();
        v.Validate(m);
    }

    [Test]
    public void ValidateNameTest()
    {
        IValidator im = new NameValidator(m.Name);

        IMessage returnObj = im.Validate();

        Assert.AreEqual(successType, returnObj.GetType());
    }

    [Test]
    public void IsValidNameTest()
    {
        IValidator im = new NameValidator(m.Name);

        IMessage returnObj = im.Validate();

        Assert.IsTrue(im.IsValid());
    }

    [Test]
    public void ValidateAgeTest()
    {
        IValidator im = new AgeValidator(m.Age);

        IMessage returnObj = im.Validate();

        Assert.AreEqual(successType, returnObj.GetType(), "Must be over 18");
    }

    [Test]
    public void IsValidAgeTest()
    {
        IValidator im = new AgeValidator(m.Age);

        IMessage returnObj = im.Validate();

        Assert.IsTrue(im.IsValid());
    }

谢谢。

Answer 1

经验法则是“测试所有可能损坏的东西”。现在，在现实生活中判断什么可能会破坏绝非易事。需要练习和经验才能做到正确。而且不可能提供一般性建议，尤其是在没有看到您的设计和代码的情况下。

因此，最终，只有您可以可靠地评估您是否有足够的信心认为 ValidationDecorator 是如此微不足道，永远不会崩溃。如果有疑问，最好宁可进行太多测试:-) 这意味着您要在可能不重要的任务上花费一些额外的时间。相反 - 未能编写所需的单元测试 - 意味着您可能会让错误漏网，这通常是一个更大的问题。

Answer 2

是的，您应该对任何可能包含错误的类进行单元测试。要对 ValidationDecorator 类进行单元测试，您应该使用 IValidator 的模拟或存根实现。