Fix Type.GetDefaultValue() for nullable value types

skrysmanski · Jun 25, 2024 · 6252b87 · 6252b87
1 parent c69a3f7
commit 6252b87
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/src/AppMotor.Core/Extensions/TypeExtensions.cs b/src/AppMotor.Core/Extensions/TypeExtensions.cs
@@ -43,6 +43,11 @@ public static string GetCSharpName(this Type type, bool includeNamespaces = fals
     {
         if (type.IsValueType)
         {
+            if (type.IsNullableValueType())
+            {
+                return null;
+            }
+
             return RuntimeHelpers.GetUninitializedObject(type);
         }
         else

diff --git a/tests/AppMotor.Core.Tests/Tests/Extensions/TypeExtensionsTests.cs b/tests/AppMotor.Core.Tests/Tests/Extensions/TypeExtensionsTests.cs
@@ -37,6 +37,18 @@ public void Test_GetDefaultValue()
         typeof(bool).GetDefaultValue().ShouldBe(false);
     }
 
+    [Fact]
+    public void Test_GetDefaultValue_NullableValueTypes()
+    {
+        // Verify assumptions
+        default(bool?).ShouldBe(null);
+        default(int?).ShouldBe(null);
+
+        // Tests
+        typeof(bool?).GetDefaultValue().ShouldBe(null);
+        typeof(int?).GetDefaultValue().ShouldBe(null);
+    }
+
     /// <summary>
     /// Tests that mutating the default value of a mutable struct doesn't mutate the default value itself.
     /// This version works with regular C# (where such a thing isn't possible - see <see cref="Test_GetDefaultValue_MutableStruct_Reflection"/>).