9

エンティティかどうかを確認するいくつかのテストを含むテストクラスがありますIsValid。独自のカスタム検証から使用IValidatableObjectするようになりましたが、正しい検証手法に固執しています。

これは私のテストクラスです：

[TestFixture]
public class StudentTests {
    private static Student GetContactWithContactInfo()
    {
        return new Student(new TestableContactRepository())
                            {
                                Phone = "7275551111"
                            };
    }

    private static Student GetContactWithoutContactInfo()
    {
        return new Student(new TestableContactRepository());
    }

    [Test]
    public void Student_Saving_StudentHasInfo_IsValid ()
    {
        // Arrange
        Student student = GetContactWithContactInfo();
        // Act
        student.Save();
        // Assert
        Assert.IsTrue(student.IsValid);
    }

    [Test]
    public void Student_Saving_StudentDoesNotHaveInfo_IsNotValid ()
    {
        // Arrange
        Student student = GetContactWithoutContactInfo();
        // Act
        student.Save();
        // Assert
        Assert.IsFalse(student.IsValid);
    }
}

これは私の実体です：

public class Student : IValidatableObject
{
    private readonly IContactRepository contactRepository;

    public Student(IContactRepository _contactRepository)
    {
        contactRepository = _contactRepository;
        Contacts = new List<Student>();
    }

    [Required]
    public int Id { get; private set; }

    [StringLength(10, MinimumLength = 10)]
    public string Phone { get; set; }


    public List<Student> Contacts { get; private set; }

    public bool IsValid { get; private set; }

    public void Save()
    {
        if (IsValidForPersistance())
        {
            IsValid = true;
            Id = contactRepository.Save();
        }
    }

    private bool IsValidForPersistance()
    {
        return Validator.TryValidateObject(this, new ValidationContext(this), null, true);
    }

    public IEnumerable<ValidationResult> Validate(ValidationContext validationContext)
    {
        if (string.IsNullOrEmpty(Phone) && Contacts.All(c => string.IsNullOrEmpty(c.Phone)))
            yield return new ValidationResult("The student or at least one contact must have a phone number entered", new[] { "Phone Number" });
    }
}

ご覧のとおり、IsValidを呼び出すことでテストがテストされIsValidForPersistanceます。Validate最終的にはより多くの検証が行われます。

上記のテストはすべてこのメソッドを使用して合格しますが、以下のこのテストも合格しますが、合格しないはずです。

[Test]
public void Student_Saving_HasContactInfoWithInvalidLength_IsNotValid()
{
    // Arrange
    Contact student = GetContactWithoutContactInfo();
    student.Phone = "string";

    // Act
    student.Save();

    // Assert
    Assert.IsFalse(student.IsValid);
}

ここでPhoneは、無効な長さの文字列の独自の値を設定しています。StringLength注釈が最小および最大10文字に設定されているため、検証が失敗することを期待しています。

なぜこれが通過するのですか？

更新 カスタム検証に問題があり、変更を加えてコードを更新しました。privateプロパティに修飾子がないことについてのnemesvからの提案に加えて、Phone現在は機能しています。すべてのコードを動作するように更新しました。

4

1 に答える 1

18

Validator.TryValidateObject は、デフォルトでs（およびタイプレベルの属性や実装などの他のもの）のみをチェックします。RequiredAttributeIValidatableObject

などのようなすべての属性を検証する必要がある場合は、メソッドのパラメータを次のようStringLengthに設定する必要があります。validateAllPropertiestrue

private bool IsValidForPersistance() {
    return Validator.TryValidateObject(this, 
                                       new ValidationContext(this), 
                                       null,
                                       true /* validateAllProperties */);
}
于 2012-09-01T17:07:12.193 に答える