单元测试装饰器模式

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是微不足道的，它永远不会破裂。如果有疑问，最好宁可进行太多测试:-) 这意味着您要在可能不重要的任务上花费一些额外的时间。相反 - 未能编写所需的单元测试 - 意味着您可能会让错误漏网，这通常是一个更大的问题。