Add OrderBy NullPropagation tests

zzzprojects · Dec 24, 2024 · 3e61d5f · 3e61d5f
1 parent bf213e6
commit 3e61d5f
Show file tree

Hide file tree

Showing 3 changed files with 224 additions and 134 deletions.
diff --git a/test/System.Linq.Dynamic.Core.Tests/EntitiesTests.OrderBy.cs b/test/System.Linq.Dynamic.Core.Tests/EntitiesTests.OrderBy.cs
@@ -51,4 +51,26 @@ public void Entities_OrderBy_RestrictOrderByIsTrue_InvalidExpressionShouldThrow(
         // Assert
         action.Should().Throw<ParseException>().WithMessage("No property or field 'IIF' exists in type 'Blog'");
     }
+
+    [Fact]
+    public void Entities_OrderBy_NullPropagation_Int()
+    {
+        // Arrange
+        var resultBlogs = _context.Blogs.OrderBy(b => b.NullableInt ?? -1).ToArray();
+        var dynamicResultBlogs = _context.Blogs.OrderBy("np(NullableInt, -1)").ToArray();
+
+        // Assert
+        Assert.Equal(resultBlogs, dynamicResultBlogs);
+    }
+
+    [Fact]
+    public void Entities_OrderBy_NullPropagation_String()
+    {
+        // Arrange
+        var resultBlogs = _context.Blogs.OrderBy(b => b.X ?? "_").ToArray();
+        var dynamicResultBlogs = _context.Blogs.OrderBy("np(X, \"_\")").ToArray();
+
+        // Assert
+        Assert.Equal(resultBlogs, dynamicResultBlogs);
+    }
 }
diff --git a/test/System.Linq.Dynamic.Core.Tests/Helpers/Models/User.cs b/test/System.Linq.Dynamic.Core.Tests/Helpers/Models/User.cs
@@ -24,6 +24,8 @@ public class User
 
     public int? NullableInt { get; set; }
 
+    public string? NullableString { get; set; }
+
     public int Income { get; set; }
 
     public char C { get; set; }