fix Unsafe Tyehandling issue (#1165)

* fix #pragma warning disable CA2327 * Update test/Integration/test-csharp/Startup.cs Co-authored-by: Charles Gagnon <[email protected]> * remove TypeNameHandling --------- Co-authored-by: Charles Gagnon <[email protected]>
Azure · Feb 5, 2025 · 6697e7b · 6697e7b
1 parent 2c606fd
commit 6697e7b
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/test/Integration/test-csharp/Startup.cs b/test/Integration/test-csharp/Startup.cs
@@ -13,17 +13,15 @@ public class Startup : FunctionsStartup
     {
         public override void Configure(IFunctionsHostBuilder builder)
         {
-#pragma warning disable CA2327 // Test code, safe to ignore
             // Set default settings for JsonConvert to simulate a user doing the same in their function.
             // This will cause test failures if serialization/deserialization isn't done correctly
             // (using the helper methods in Utils.cs)
-            JsonConvert.DefaultSettings = () => new JsonSerializerSettings
+            var settings = new JsonSerializerSettings
             {
                 Formatting = Formatting.Indented,
-                TypeNameHandling = TypeNameHandling.Objects,
                 ContractResolver = new CamelCasePropertyNamesContractResolver()
             };
-#pragma warning restore CA2327
+            JsonConvert.DefaultSettings = () => settings;
         }
     }
 }