Add new tests for nullable types and fix GetFullTypeName

Author: fabien.menager

src/EntityFrameworkCore.Projectables/Services/ProjectionExpressionResolver.cs

@@ -128,6 +128,13 @@ static string GetFullTypeName(Type type)
                     return type.Name;
                 }
 
+                // Handle nullable value types (e.g., int? -> int?)
+                var underlyingType = Nullable.GetUnderlyingType(type);
+                if (underlyingType != null)
+                {
+                    return $"{GetFullTypeName(underlyingType)}?";
+                }
+                
                 // Handle array types
                 if (type.IsArray)
                 {

tests/EntityFrameworkCore.Projectables.FunctionalTests/ComplexArgumentsTests.NullableValueTypes.DotNet10_0.verified.txt

@@ -0,0 +1,3 @@
+SELECT [t].[Id]
+FROM [TestEntity] AS [t]
+WHERE 1 > [t].[Id] AND NEWID() <> '00000000-0000-0000-0000-000000000000'

tests/EntityFrameworkCore.Projectables.FunctionalTests/ComplexArgumentsTests.NullableValueTypes.DotNet9_0.verified.txt

@@ -0,0 +1,3 @@
+SELECT [t].[Id]
+FROM [TestEntity] AS [t]
+WHERE 1 > [t].[Id] AND NEWID() <> '00000000-0000-0000-0000-000000000000'

tests/EntityFrameworkCore.Projectables.FunctionalTests/ComplexArgumentsTests.NullableValueTypes.verified.txt

@@ -0,0 +1,3 @@
+SELECT [t].[Id]
+FROM [TestEntity] AS [t]
+WHERE 1 > [t].[Id] AND NEWID() <> '00000000-0000-0000-0000-000000000000'

tests/EntityFrameworkCore.Projectables.FunctionalTests/ComplexArgumentsTests.cs

@@ -25,6 +25,9 @@ public class TestEntity
 
             [Projectable]
             public bool IsValid3(params int[] validIds) => validIds.Contains(Id);
+
+            [Projectable]
+            public bool IsValid4(int? validId, Guid? guid) => validId > Id && guid != Guid.Parse("00000000-0000-0000-0000-000000000000");
         }
 
         [Fact]
@@ -52,8 +55,7 @@ public Task ArrayOfPrimitivesArguments()
 
             return Verifier.Verify(query.ToQueryString());
         }
-
-
+        
         [Fact]
         public Task ParamsOfPrimitivesArguments()
         {
@@ -64,5 +66,16 @@ public Task ParamsOfPrimitivesArguments()
 
             return Verifier.Verify(query.ToQueryString());
         }
+        
+        [Fact]
+        public Task NullableValueTypes()
+        {
+            using var dbContext = new SampleDbContext<TestEntity>();
+
+            var query = dbContext.Set<TestEntity>()
+                .Where(x => x.IsValid4(1, Guid.NewGuid()));
+
+            return Verifier.Verify(query.ToQueryString());
+        }
     }
 }

tests/EntityFrameworkCore.Projectables.FunctionalTests/ComplexModelTests.ProjectOverGenericType.DotNet10_0.verified.txt

@@ -0,0 +1,2 @@
+SELECT [g].[Id]
+FROM [GenericObject<int>] AS [g]

tests/EntityFrameworkCore.Projectables.FunctionalTests/ComplexModelTests.ProjectOverGenericType.DotNet9_0.verified.txt

@@ -0,0 +1,2 @@
+SELECT [g].[Id]
+FROM [GenericObject<int>] AS [g]

tests/EntityFrameworkCore.Projectables.FunctionalTests/ComplexModelTests.ProjectOverGenericType.verified.txt

@@ -0,0 +1,2 @@
+SELECT [g].[Id]
+FROM [GenericObject<int>] AS [g]

tests/EntityFrameworkCore.Projectables.FunctionalTests/ComplexModelTests.cs

@@ -52,6 +52,11 @@ public class Order
 
             public DateTime RecordDate { get; set; }
         } 
+        
+        public class GenericObject<T>
+        {
+            public T Id { get; set; }
+        } 
 
         [Fact]
         public Task ProjectOverNavigationProperty()
@@ -98,5 +103,16 @@ public Task ProjectQueryFilters()
 
             return Verifier.Verify(query.ToQueryString());
         }
+
+        [Fact]
+        public Task ProjectOverGenericType()
+        {
+            using var dbContext = new SampleDbContext<GenericObject<int>>();
+            
+            var query = dbContext.Set<GenericObject<int>>()
+                .Select(x => x.Id);
+            
+            return Verifier.Verify(query.ToQueryString());
+        }
     }
 }

tests/EntityFrameworkCore.Projectables.FunctionalTests/MethodOverloadsTests.MethodOverload_WithIntParameter.DotNet10_0.verified.txt

@@ -0,0 +1,2 @@
+SELECT [e].[Id], [e].[Id] + 10 AS [Result]
+FROM [Entity] AS [e]