我有一个非常有趣的行为。我有以下两个测试用例:
[Fact]
public void Ctor_WhenNeverIsTrueAndAfterOcurrenceIsNotNull_HasError()
{
// arrange
var reccurenceEnd = new RecurrenceEnd()
{
IsNever = true,
AfterOccurence = 1
};
// act
var validator = GetValidator();
// assert
validator.ShouldHaveValidationErrorFor(p => p.AfterOccurence, reccurenceEnd);
}
[Fact]
public void Ctor_WhenNeverIsFalseAndAfterOccurenceIsNullAndByDateIsNull_HasError()
{
// arrange
var reccurenceEnd = new RecurrenceEnd()
{
IsNever = false,
AfterOccurence = null,
ByDate = null
};
// act
var validator = GetValidator();
// assert
validator.ShouldHaveValidationErrorFor(p => p.AfterOccurence, reccurenceEnd);
}
在我的验证器上,如果我有以下内容,则第一次测试失败,第二次测试失败。如果我更改规则的顺序,则第一次测试通过,第二次测试失败。
RuleFor(dto => dto.AfterOccurence)
.Cascade(CascadeMode.StopOnFirstFailure)
.Null()
.When(p => p.IsNever == true)
.NotEmpty()
.When(p => p.IsNever == false && p.ByDate == null);
如果我将验证器更改为以下内容,则两个测试都通过。
RuleFor(dto => dto.AfterOccurence)
.Null()
.When(p => p.IsNever);
RuleFor(dto => dto.AfterOccurence)
.NotEmpty()
.When(p => p.IsNever == false && p.ByDate == null);
我错了吗?
好的,从FluentValidation的github-issues得到了答案:
嗨,这与你使用When条件有关。默认情况下,When条件适用于同一RuleFor调用中的所有先前验证器,因此当您使用对RuleFor的单个调用时,您实际上是在加倍条件。您应该使用对RuleFor的单独调用(如第二个示例中所示),或者使用其他重载时允许您指定ApplyConditionTo.CurrentValidator
第二个解决方案是:
RuleFor(dto => dto.AfterOccurence)
.Cascade(CascadeMode.StopOnFirstFailure)
.Null()
.When(p => p.IsNever == true)
.NotEmpty()
.When(p => p.IsNever == false && p.ByDate == null, ApplyConditionTo.CurrentValidator);