diff --git a/src/FluentValidation.Tests/ExclusiveBetweenValidatorTests.cs b/src/FluentValidation.Tests/ExclusiveBetweenValidatorTests.cs index 75fd95d..a4ebfa4 100644 --- a/src/FluentValidation.Tests/ExclusiveBetweenValidatorTests.cs +++ b/src/FluentValidation.Tests/ExclusiveBetweenValidatorTests.cs @@ -127,12 +127,6 @@ public class ExclusiveBetweenValidatorTests { Assert.Throws(() => RangeValidatorFactory.CreateExclusiveBetween("ccc", "aaa")); } - [Fact] - public void When_the_to_is_smaller_than_the_from_then_the_validator_should_throw_for_chars() { - // char.CompareTo returns the difference ('a' vs 'c' is -2), not just -1. - Assert.Throws(() => RangeValidatorFactory.CreateExclusiveBetween('c', 'a')); - } - [Fact] public void When_the_validator_fails_the_error_message_should_be_set_for_strings() { var validator = new TestValidator(v => v.RuleFor(x => x.Surname).ExclusiveBetween("bbb", "zzz")); diff --git a/src/FluentValidation.Tests/InclusiveBetweenValidatorTests.cs b/src/FluentValidation.Tests/InclusiveBetweenValidatorTests.cs index c10a982..8341b18 100644 --- a/src/FluentValidation.Tests/InclusiveBetweenValidatorTests.cs +++ b/src/FluentValidation.Tests/InclusiveBetweenValidatorTests.cs @@ -122,12 +122,6 @@ public class InclusiveBetweenValidatorTests { Assert.Throws(() => RangeValidatorFactory.CreateInclusiveBetween("ccc", "aaa")); } - [Fact] - public void When_the_to_is_smaller_than_the_from_then_the_validator_should_throw_for_chars() { - // char.CompareTo returns the difference ('a' vs 'c' is -2), not just -1. - Assert.Throws(() => RangeValidatorFactory.CreateInclusiveBetween('c', 'a')); - } - [Fact] public void When_the_validator_fails_the_error_message_should_be_set_for_strings() { var validator = new TestValidator(v => v.RuleFor(x => x.Surname).InclusiveBetween("bbb", "zzz")); diff --git a/src/FluentValidation/Validators/RangeValidator.cs b/src/FluentValidation/Validators/RangeValidator.cs index 65b00e4..eafe717 100644 --- a/src/FluentValidation/Validators/RangeValidator.cs +++ b/src/FluentValidation/Validators/RangeValidator.cs @@ -34,8 +34,7 @@ public abstract class RangeValidator : PropertyValidator only guarantees the sign, not the magnitude. - if (comparer.Compare(to, from) < 0) { + if (comparer.Compare(to, from) == -1) { throw new ArgumentOutOfRangeException(nameof(to), "To should be larger than from."); } }