fix: serialization of primitives present in additionalData

CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.15.1] - 2024-11-13
+
+### Added
+
+- Fixes serialization collections of primitives present in additional data. [microsoftgraph/msgraph-sdk-dotnet#2729](https://github.com/microsoftgraph/msgraph-sdk-dotnet/issues/2729)
+
 ## [1.15.0] - 2024-11-13
 
 ### Added

Directory.Build.props

@@ -1,7 +1,7 @@
 <Project>
   <!-- Common default project properties for ALL projects-->
   <PropertyGroup>
-    <VersionPrefix>1.15.0</VersionPrefix>
+    <VersionPrefix>1.15.1</VersionPrefix>
     <VersionSuffix></VersionSuffix>
     <!-- This is overidden in test projects by setting to true-->
     <IsTestProject>false</IsTestProject>

src/serialization/json/JsonSerializationWriter.cs

@@ -1,4 +1,4 @@
-// ------------------------------------------------------------------------------
+// ------------------------------------------------------------------------------
 //  Copyright (c) Microsoft Corporation.  All Rights Reserved.  Licensed under the MIT License.  See License in the project root for license information.
 // ------------------------------------------------------------------------------
 
@@ -23,7 +23,7 @@
     /// <summary>
     /// The <see cref="ISerializationWriter"/> implementation for json content types.
     /// </summary>
     public class JsonSerializationWriter : ISerializationWriter, IDisposable
     {
         private readonly MemoryStream _stream = new MemoryStream();
         private readonly KiotaJsonSerializationContext _kiotaJsonSerializationContext;
@@ -273,7 +273,7 @@
         /// <param name="key">The key of the json node</param>
         /// <param name="value">The enumeration value</param>
 #if NET5_0_OR_GREATER
         public void WriteEnumValue<[DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicFields)] T>(string? key, T? value) where T : struct, Enum
 #else
         public void WriteEnumValue<T>(string? key, T? value) where T : struct, Enum
 #endif
@@ -311,7 +311,9 @@
         /// </summary>
         /// <param name="key">The key of the json node</param>
         /// <param name="values">The primitive collection</param>
-        public void WriteCollectionOfPrimitiveValues<T>(string? key, IEnumerable<T>? values)
+        public void WriteCollectionOfPrimitiveValues<T>(string? key, IEnumerable<T>? values) => WriteCollectionOfPrimitiveValuesInternal(key, values);
+
+        private void WriteCollectionOfPrimitiveValuesInternal(string? key, IEnumerable? values)
         {
             if(values != null)
             { //empty array is meaningful
@@ -374,7 +376,7 @@
         private void WriteDictionaryValue<T>(string? key, T values) where T : IDictionary
 #endif
         {
             if(values != null)
             {
                 if(!string.IsNullOrEmpty(key))
                     writer.WritePropertyName(key!);
@@ -412,16 +414,16 @@
         /// <param name="key">The key of the json node</param>
         /// <param name="value">The object instance to write</param>
         /// <param name="additionalValuesToMerge">The additional values to merge to the main value when serializing an intersection wrapper.</param>
         public void WriteObjectValue<T>(string? key, T? value, params IParsable?[] additionalValuesToMerge) where T : IParsable
         {
             var filteredAdditionalValuesToMerge = (IParsable[])Array.FindAll(additionalValuesToMerge, static x => x is not null);
             if(value != null || filteredAdditionalValuesToMerge.Length > 0)
             {
                 // until interface exposes WriteUntypedValue()
                 var serializingUntypedNode = value is UntypedNode;
                 if(!serializingUntypedNode && !string.IsNullOrEmpty(key))
                     writer.WritePropertyName(key!);
                 if(value != null)
                     OnBeforeObjectSerialization?.Invoke(value);
 
                 if(serializingUntypedNode)
@@ -436,7 +438,7 @@
                     var serializingScalarValue = value is IComposedTypeWrapper;
                     if(!serializingScalarValue)
                         writer.WriteStartObject();
                     if(value != null)
                     {
                         OnStartObjectSerialization?.Invoke(value, this);
                         value.Serialize(this);
@@ -451,7 +453,7 @@
                     if(!serializingScalarValue)
                         writer.WriteEndObject();
                 }
                 if(value != null) OnAfterObjectSerialization?.Invoke(value);
             }
         }
 
@@ -477,7 +479,7 @@
             if(!string.IsNullOrEmpty(key))
                 writer.WritePropertyName(key!);
             writer.WriteStartObject();
             if(value == null)
                 writer.WriteNullValue();
             else
                 foreach(var oProp in value.GetType().GetProperties())
@@ -525,9 +527,6 @@
                 case TimeSpan timeSpan:
                     WriteTimeSpanValue(key, timeSpan);
                     break;
-                case IEnumerable<object> coll:
-                    WriteCollectionOfPrimitiveValues(key, coll);
-                    break;
                 case UntypedNode node:
                     WriteUntypedValue(key, node);
                     break;
@@ -551,6 +550,9 @@
                 case IDictionary dictionary:
                     WriteDictionaryValue(key, dictionary);
                     break;
+                case IEnumerable coll:
+                    WriteCollectionOfPrimitiveValuesInternal(key, coll);
+                    break;
                 case object o:
                     WriteNonParsableObjectValue(key, o);
                     break;

tests/serialization/json/JsonSerializationWriterTests.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Globalization;
 using System.IO;
@@ -40,6 +40,7 @@ public void WritesSampleObjectValue()
                     {"createdDateTime", DateTimeOffset.MinValue}, // write date value
                     {"weightInKgs", 51.80m}, // write weigth
                     {"businessPhones", new List<string>() {"+1 412 555 0109"}}, // write collection of primitives value
+                    {"dates",new List<DateTimeOffset> { DateTimeOffset.MaxValue , DateTimeOffset.MinValue }},
                     {"endDateTime", new DateTime(2023,03,14,0,0,0,DateTimeKind.Utc) }, // ensure the DateTime doesn't crash
                     {"manager", new TestEntity{Id = "48d31887-5fad-4d73-a9f5-3c356e68a038"}}, // write nested object value
                     {"anonymousObject", new {Value1 = true, Value2 = "", Value3 = new List<string>{ "Value3.1", "Value3.2"}}}, // write nested object value
@@ -69,6 +70,7 @@ public void WritesSampleObjectValue()
                                  "\"createdDateTime\":\"0001-01-01T00:00:00+00:00\"," +
                                  "\"weightInKgs\":51.80," +
                                  "\"businessPhones\":[\"\\u002B1 412 555 0109\"]," +
+                                 "\"dates\":[\"9999-12-31T23:59:59.9999999+00:00\",\"0001-01-01T00:00:00+00:00\"]," +
                                  "\"endDateTime\":\"2023-03-14T00:00:00+00:00\"," +
                                  "\"manager\":{\"id\":\"48d31887-5fad-4d73-a9f5-3c356e68a038\"}," +
                                  "\"anonymousObject\":{\"Value1\":true,\"Value2\":\"\",\"Value3\":[\"Value3.1\",\"Value3.2\"]}," +
@@ -292,6 +294,36 @@ public void ForwardsOptionsToWriterFromSerializationContext()
             Assert.Equal(expectedString, serializedJsonString.Replace("\r", string.Empty)); // string is indented and not escaped
         }
 
+        [Fact]
+        public void WritesPrimitiveCollectionsInAdditionalData()
+        {
+            // Arrange
+            var dates = new List<DateTimeOffset>
+            {
+                DateTimeOffset.MaxValue, DateTimeOffset.MinValue
+            };
+            var testEntity = new TestEntity
+            {
+                Id = "testId",
+                AdditionalData = new Dictionary<string, object>()
+                {
+                    {"dates", dates}
+                }
+            };
+            using var jsonSerializerWriter = new JsonSerializationWriter();
+            // Act
+            jsonSerializerWriter.WriteObjectValue(string.Empty, testEntity);
+            var serializedStream = jsonSerializerWriter.GetSerializedContent();
+            using var reader = new StreamReader(serializedStream, Encoding.UTF8);
+            var serializedJsonString = reader.ReadToEnd();
+
+            // Assert
+            Assert.Contains("\"id\":\"testId\"", serializedJsonString);
+            Assert.Contains("\"dates\":[\"", serializedJsonString);
+            Assert.Contains(JsonSerializer.Serialize(DateTimeOffset.MinValue), serializedJsonString);
+            Assert.Contains(JsonSerializer.Serialize(DateTimeOffset.MaxValue), serializedJsonString);
+        }
+
         [Fact]
         public void UsesDefaultOptionsToWriterFromSerializationContext()
         {