#87 - AttributeArgument with null value, TypedConstant ToString throw…

…s NullReferenceException
NeVeSpl · Aug 1, 2023 · bf8573c · bf8573c
1 parent 23a40b9
commit bf8573c
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 3 deletions.
diff --git a/NTypewriter.CodeModel.Roslyn/TypedConstant.cs b/NTypewriter.CodeModel.Roslyn/TypedConstant.cs
@@ -32,7 +32,7 @@ public static object Create(Microsoft.CodeAnalysis.TypedConstant typedConstant)
 
         public override string ToString()
         {
-            string valueAsString = typedConstant.Type.GetDefaultConstantValueAsString(typedConstant.Value).ToString();
+            string valueAsString = typedConstant.Type.GetDefaultConstantValueAsString(typedConstant.Value)?.ToString();
 
             return valueAsString;
         }

diff --git a/NTypewriter.CodeModel.Tests/AttributeArgument/Value/expectedResult.txt b/NTypewriter.CodeModel.Tests/AttributeArgument/Value/expectedResult.txt
@@ -1,4 +1,10 @@
-AuctionContractCount44
+NullOne
+[StatisticsGroup]
+             name : object 
+             type : object
+             value : 
+             -------------
+AuctionContractCount44
 [StatisticsGroup]
              name : type 
              type : StatisticsGroupTypeEnum

diff --git a/NTypewriter.CodeModel.Tests/AttributeArgument/Value/inputCode.cs b/NTypewriter.CodeModel.Tests/AttributeArgument/Value/inputCode.cs
@@ -8,7 +8,10 @@
 namespace NTypewriter.Tests.CodeModel
 {
     public enum RegionStatTypeEnum
-    {        
+    {
+        [StatisticsGroup(null)]
+        NullOne,
+
         [StatisticsGroup(StatisticsGroupTypeEnum.Restriction, FederalLawEnum.FZ44)]
         AuctionContractCount44,
 
@@ -19,6 +22,11 @@ public enum RegionStatTypeEnum
     [AttributeUsage(AttributeTargets.Field, AllowMultiple = true)]
     public class StatisticsGroupAttribute : Attribute
     {
+        public StatisticsGroupAttribute(object @object)
+        {
+
+        }
+
         public StatisticsGroupAttribute(StatisticsGroupTypeEnum type, params object[] values)
         {