React to aspnet/Mvc#4834

- JSON now serialized with camel case
- another part of fix for aspnet/Mvc#4283

Author: dougbu

samples/MusicStore.Spa/Infrastructure/SmartJsonResult.cs

@@ -1,6 +1,6 @@
-using System;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Mvc.Formatters;
 using Newtonsoft.Json;
 
 namespace Microsoft.AspNetCore.Mvc
@@ -9,7 +9,6 @@ public class SmartJsonResult : ActionResult
     {
         public SmartJsonResult() : base()
         {
-            
         }
 
         public JsonSerializerSettings Settings { get; set; }
@@ -29,8 +28,8 @@ public override Task ExecuteResultAsync(ActionContext context)
             //    context.HttpContext.Response.ContentType = "application/json";
             //    context.HttpContext.Response.ContentEncoding = Encoding.UTF8;
             //}
-            
-            return context.HttpContext.Response.WriteAsync(JsonConvert.SerializeObject(Data, Settings ?? new JsonSerializerSettings()));
+            var settings = Settings ?? JsonSerializerSettingsProvider.CreateSerializerSettings();
+            return context.HttpContext.Response.WriteAsync(JsonConvert.SerializeObject(Data, settings));
         }
     }
 }

test/FunctionalTests/MvcTests/FormatFilterSampleTest.cs

@@ -30,7 +30,7 @@ public async Task FormatFilter_NoExtensionInRequest()
 
             // Assert
             Assert.Equal(HttpStatusCode.OK, response.StatusCode);
-            Assert.Equal(@"{""SampleInt"":5}", await response.Content.ReadAsStringAsync());
+            Assert.Equal(@"{""sampleInt"":5}", await response.Content.ReadAsStringAsync());
         }
 
         [Fact]
@@ -41,7 +41,7 @@ public async Task FormatFilter_ExtensionInRequest_Default()
 
             // Assert
             Assert.Equal(HttpStatusCode.OK, response.StatusCode);
-            Assert.Equal(@"{""SampleInt"":5}", await response.Content.ReadAsStringAsync());
+            Assert.Equal(@"{""sampleInt"":5}", await response.Content.ReadAsStringAsync());
         }
 
         [Fact]
@@ -56,7 +56,7 @@ public async Task FormatFilter_ExtensionInRequest_Optional()
 
             // Assert
             Assert.Equal(HttpStatusCode.OK, response.StatusCode);
-            Assert.Equal(@"{""SampleInt"":0}", await response.Content.ReadAsStringAsync());
+            Assert.Equal(@"{""sampleInt"":0}", await response.Content.ReadAsStringAsync());
         }
 
         [Fact]
@@ -107,7 +107,7 @@ public async Task FormatFilter_And_ProducesFilter_Match()
 
             // Assert
             Assert.Equal(HttpStatusCode.OK, response.StatusCode);
-            Assert.Equal(@"{""SampleInt"":5}", await response.Content.ReadAsStringAsync());
+            Assert.Equal(@"{""sampleInt"":5}", await response.Content.ReadAsStringAsync());
         }
 
         [Fact]

test/FunctionalTests/MvcTests/JsonPatchSampleTest.cs

@@ -301,7 +301,7 @@ public async Task JsonPatch_JsonConverterOnProperty_Success()
             // Assert
             var body = await response.Content.ReadAsStringAsync();
             dynamic d = JObject.Parse(body);
-            Assert.Equal("OrderTypeSetInConverter", (string)d.Orders[2].OrderType);
+            Assert.Equal("OrderTypeSetInConverter", (string)d.orders[2].orderType);
         }
 
         [Fact]
@@ -324,7 +324,7 @@ public async Task JsonPatch_JsonConverterOnClass_Success()
             // Assert
             var body = await response.Content.ReadAsStringAsync();
             dynamic d = JObject.Parse(body);
-            Assert.Equal("CategorySetInConverter", (string)d.ProductCategory.CategoryName);
+            Assert.Equal("CategorySetInConverter", (string)d.productCategory.CategoryName);
 
         }
     }