Fix #192

CHANGELOG.md

@@ -1,4 +1,10 @@
-## `master` (upcoming 1.7)
+## Master (upcoming 1.8)
+
+## Version 1.7.1
+
+- [**BUGFIX**] Support camel case serialization for nested properties in attributes and meta hashes
+
+## Version 1.7
 
 - [**BUGFIX**] Serialise complex objects correctly (#149 by @laurence79)
 - [**BUGFIX**] Do not require Accept header (#151 by @bjornharrtell)

Saule/Serialization/JsonApiContractResolver.cs

@@ -7,13 +7,20 @@ namespace Saule.Serialization
 {
     internal class JsonApiContractResolver : DefaultContractResolver
     {
+        private readonly IPropertyNameConverter _nameConverter;
+
+        public JsonApiContractResolver(IPropertyNameConverter nameConverter)
+        {
+            _nameConverter = nameConverter;
+        }
+
         protected override IList<JsonProperty> CreateProperties(Type type, MemberSerialization memberSerialization)
         {
             var properties = base.CreateProperties(type, memberSerialization);
 
             foreach (var property in properties)
             {
-                property.PropertyName = property.PropertyName.ToDashed();
+                property.PropertyName = _nameConverter.ToJsonPropertyName(property.PropertyName);
             }
 
             return properties;

Saule/Serialization/ResourceSerializer.cs

@@ -47,7 +47,7 @@ public JObject Serialize()
 
         public JObject Serialize(JsonSerializer serializer)
         {
-            serializer.ContractResolver = new JsonApiContractResolver();
+            serializer.ContractResolver = new JsonApiContractResolver(_propertyNameConverter);
             _serializer = serializer;
 
             if (_value == null)

Tests/Integration/PropertyNameConverterTests.cs

@@ -24,5 +24,43 @@ public async Task CamelCaseConstructor()
                 Assert.NotNull(result["data"]["attributes"]["firstName"]);
             }
         }
+
+        [Fact(DisplayName = "Nested properties in attributes are converted correctly")]
+        public async Task CamelCaseNestedAttrs()
+        {
+            using (var server = new NewSetupJsonApiServer(new JsonApiConfiguration
+            {
+                PropertyNameConverter = new CamelCasePropertyNameConverter()
+            }))
+            {
+                var client = server.GetClient();
+
+                var result = await client.GetJsonResponseAsync("api/people/123");
+
+                var address = result["data"]["attributes"]["address"];
+
+                Assert.NotNull(address["streetName"]);
+                Assert.NotNull(address["zipCode"]);
+            }
+        }
+
+        [Fact(DisplayName = "Meta hash properties are converted correctly")]
+        public async Task CamelCaseMeta()
+        {
+            using (var server = new NewSetupJsonApiServer(new JsonApiConfiguration
+            {
+                PropertyNameConverter = new CamelCasePropertyNameConverter()
+            }))
+            {
+                var client = server.GetClient();
+
+                var result = await client.GetJsonResponseAsync("api/people/123");
+
+                var meta = result["meta"];
+
+                Assert.NotNull(meta["numberOfFriends"]);
+                Assert.NotNull(meta["numberOfFamilyMembers"]);
+            }
+        }
     }
 }

appveyor.yml

@@ -1,4 +1,4 @@
-version: 1.7.0.{build}
+version: 1.7.1.{build}
 configuration: Release
 skip_tags: true
 assembly_info: