Merge pull request #171 from valit-stack/develop

v2.0.0

Author: GooRiOn

README.md

@@ -15,12 +15,12 @@ Valit is available on [NuGet](https://www.nuget.org/packages/Valit/).
 
 ### Package manager
 ```bash
-Install-Package Valit -Version 1.0.0
+Install-Package Valit -Version 2.0.0
 ```
 
 ### .NET CLI
 ```bash
-dotnet add package Valit --version 1.0.0
+dotnet add package Valit --version 2.0.0
 ```
 
 # Getting started

src/Valit/Rules/ValitRule.cs

@@ -81,13 +81,18 @@ public IValitResult Validate(TObject @object)
             var property = _propertySelector.Compile().Invoke(@object);
             var hasAllConditionsFulfilled = true;
 
-            foreach(var condition in _conditions)
+            foreach (var condition in _conditions)
                 hasAllConditionsFulfilled &= condition(@object);
 
-            var isSatisfied = _predicate?.Invoke(property) != false;
-            var errors = _errors.Where(e => !e.IsDefault).Any() ? _errors.Where(e => !e.IsDefault) : _errors;
+            if (!hasAllConditionsFulfilled)
+            {
+                return ValitResult.Success;
+            }
+
+            var isSatisfied = _predicate?.Invoke(property) ?? false;
+            var errors = _errors.Any(e => !e.IsDefault) ? _errors.Where(e => !e.IsDefault) : _errors;
 
-            return !hasAllConditionsFulfilled || isSatisfied ? ValitResult.Success : ValitResult.Fail(errors.ToArray());
+            return isSatisfied ? ValitResult.Success : ValitResult.Fail(errors.ToArray());
         }
     }
 }

src/Valit/Valit.csproj

@@ -10,7 +10,7 @@
     <PackageProjectUrl>https://github.com/valit-stack/Valit</PackageProjectUrl>
     <PackageLicenseUrl>https://github.com/valit-stack/Valit/blob/master/LICENSE</PackageLicenseUrl>
     <PackageIconUrl>https://user-images.githubusercontent.com/7096476/32700023-298ffb9e-c7bf-11e7-9d83-82690b7e260b.png</PackageIconUrl>
-    <VersionPrefix>1.0.0</VersionPrefix>
+    <VersionPrefix>2.0.0</VersionPrefix>
     <TargetFramework>netstandard2.0</TargetFramework>
   </PropertyGroup>
 

src/Valit/ValitRuleStringExtensions.cs

@@ -8,21 +8,21 @@ public static class ValitRuleStringExtensions
     {
         private static string _emailRegularExpression => @"\A(?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?)\Z";
         public static IValitRule<TObject, string> IsEqualTo<TObject>(this IValitRule<TObject, string> rule, string value) where TObject : class
-            => rule.Satisfies(p => !String.IsNullOrEmpty(p) && !String.IsNullOrEmpty(value) && p == value).WithDefaultMessage(ErrorMessages.IsEqualTo, value);
+            => rule.Satisfies(p => p != null && value != null && p == value).WithDefaultMessage(ErrorMessages.IsEqualTo, value);
 
         public static IValitRule<TObject, string> MinLength<TObject>(this IValitRule<TObject, string> rule, int length) where TObject : class
-            => rule.Satisfies(p => !String.IsNullOrEmpty(p) && p.Length >= length).WithDefaultMessage(ErrorMessages.MinLength, length);
+            => rule.Satisfies(p => p != null && p.Length >= length).WithDefaultMessage(ErrorMessages.MinLength, length);
 
         public static IValitRule<TObject, string> MaxLength<TObject>(this IValitRule<TObject, string> rule, int length) where TObject : class
-            => rule.Satisfies(p => !String.IsNullOrEmpty(p) && p.Length <= length).WithDefaultMessage(ErrorMessages.MaxLength, length);
+            => rule.Satisfies(p => p != null && p.Length <= length).WithDefaultMessage(ErrorMessages.MaxLength, length);
 
         public static IValitRule<TObject, string> Matches<TObject>(this IValitRule<TObject, string> rule, string regularExpression) where TObject : class
-            => rule.Satisfies(p => !String.IsNullOrEmpty(p) && !String.IsNullOrEmpty(regularExpression) && Regex.IsMatch(p, regularExpression)).WithDefaultMessage(ErrorMessages.Matches, regularExpression);
+            => rule.Satisfies(p => p != null && !String.IsNullOrEmpty(regularExpression) && Regex.IsMatch(p, regularExpression)).WithDefaultMessage(ErrorMessages.Matches, regularExpression);
 
         public static IValitRule<TObject, string> Email<TObject>(this IValitRule<TObject, string> rule) where TObject : class
-            => rule.Satisfies(p => !String.IsNullOrEmpty(p) && !String.IsNullOrEmpty(_emailRegularExpression) && Regex.IsMatch(p, _emailRegularExpression)).WithDefaultMessage(ErrorMessages.Email);
+            => rule.Satisfies(p => p != null && !String.IsNullOrEmpty(_emailRegularExpression) && Regex.IsMatch(p, _emailRegularExpression)).WithDefaultMessage(ErrorMessages.Email);
 
         public static IValitRule<TObject, string> Required<TObject>(this IValitRule<TObject, string> rule) where TObject : class
-            => rule.Satisfies(p => !string.IsNullOrEmpty(p)).WithDefaultMessage(ErrorMessages.Required);
+            => rule.Satisfies(p => p != null).WithDefaultMessage(ErrorMessages.Required);
     }
 }

tests/Valit.Tests/Strategies/Complete.cs

@@ -36,7 +36,7 @@ public void Complete_Strategy_Checks_All_Ensure()
                 .Validate();
 
             result.Succeeded.ShouldBe(false);
-            result.ErrorMessages.Count().ShouldBe(6);
+            result.ErrorMessages.Count().ShouldBe(5);
             result.ErrorMessages.ShouldContain(M1);
             result.ErrorMessages.ShouldContain(M3);
             result.ErrorMessages.ShouldContain(M4);

tests/Valit.Tests/Strategies/FailFast.cs

@@ -68,7 +68,7 @@ public void FailFast_Strategy_Checks_Until_First_Error_While_Error_Is_Not_First(
 
             result.Succeeded.ShouldBe(false);
             result.ErrorMessages.Count().ShouldBe(1);
-            result.ErrorMessages.ShouldContain(M2);
+            result.ErrorMessages.ShouldContain(M3);
         }
 
 #region ARRANGE

tests/Valit.Tests/String/String_IsEqualTo_Tests.cs

@@ -36,7 +36,7 @@ public void String_IsEqualTo_Returns_Proper_Result_For_Left_Value(string value,
         [Theory]
         [InlineData("text", false)]
         [InlineData("other", false)]
-        [InlineData("", false)]
+        [InlineData("", true)]
         [InlineData(null, false)]
         public void String_IsEqualTo_Returns_Proper_Result_For_Left_Empty_Value(string value, bool expected)
         {

tests/Valit.Tests/String/String_MaxLength_Tests.cs

@@ -33,9 +33,9 @@ public void String_MaxLength_Returns_Proper_Result_For_Left_Value(int value, boo
         }
 
         [Theory]
-        [InlineData(0, false)]
-        [InlineData(4, false)]
-        [InlineData(8, false)]
+        [InlineData(0, true)]
+        [InlineData(4, true)]
+        [InlineData(8, true)]
         public void String_MaxLength_Returns_Proper_Result_For_Left_Empty_Value(int value, bool expected)
         {
             IValitResult result = ValitRules<Model>

tests/Valit.Tests/String/String_MinLength_Tests.cs

@@ -33,7 +33,7 @@ public void String_MinLength_Returns_Proper_Result_For_Left_Value(int value, boo
         }
 
         [Theory]
-        [InlineData(0, false)]
+        [InlineData(0, true)]
         [InlineData(4, false)]
         [InlineData(8, false)]
         public void String_MinLength_Returns_Proper_Result_For_Left_Empty_Value(int value, bool expected)

tests/Valit.Tests/String/String_Required_Tests.cs

@@ -33,7 +33,7 @@ public void String_Required_Fails_For_Null_Value()
         }
 
         [Fact]
-        public void String_Required_Fails_For_Empty_Value()
+        public void String_Required_Succeeds_For_Empty_Value()
         {
             IValitResult result = ValitRules<Model>
                 .Create()
@@ -42,7 +42,7 @@ public void String_Required_Fails_For_Empty_Value()
                 .For(_model)
                 .Validate();
 
-            result.Succeeded.ShouldBeFalse();
+            result.Succeeded.ShouldBeTrue();
         }
 
         [Fact]