Updates to #31

I just verified that the applied fix worked and I also cleaned up a unit
test that is a todo.. We should get that scenario working (the commented
out test).

@ejsmith I don't see any problems with this pull request. I think we
just need todo some testing.. Do you remember why we weren't doing this
before (setting the contract resolver) or was this just an oversight?

Author: niemyjski

Source/Tests/Serializer/SerializerTests.cs

@@ -3,7 +3,6 @@
 using System.Collections.ObjectModel;
 using Exceptionless.Extensions;
 using Exceptionless.Json;
-using Exceptionless.Json.Serialization;
 using Exceptionless.Models;
 using Exceptionless.Serializer;
 using Xunit;
@@ -80,13 +79,17 @@ public void WillIgnoreEmptyCollections() {
         //[Fact]
         public void CanDeserializeDataWithoutUnderscores() {
             const string json = @"{""BlahId"":""Hello""}";
-            var settings = new JsonSerializerSettings();
-            settings.ContractResolver = new LowerCaseUnderscorePropertyNamesContractResolver();
+            const string jsonWithUnderScore = @"{""blah_id"":""Hello""}";
 
-            var m = JsonConvert.DeserializeObject<Blah>(json, settings);
-            Assert.Equal("Hello", m.BlahId);
+            IJsonSerializer serializer = GetSerializer();
+            var value = serializer.Deserialize<Blah>(json);
+            Assert.Equal("Hello", value.BlahId);
+
+            value = serializer.Deserialize<Blah>(jsonWithUnderScore);
+            Assert.Equal("Hello", value.BlahId);
 
-            string newJson = JsonConvert.SerializeObject(m, settings);
+            string serialized = serializer.Serialize(value);
+            Assert.Equal(jsonWithUnderScore, serialized);
         }
 
         [Fact]
@@ -111,21 +114,4 @@ public class SampleModel {
         public ICollection<string> Collection { get; set; } 
         public SampleModel Nested { get; set; }
     }
-
-    public class LowerCaseUnderscorePropertyNamesContractResolver : DefaultContractResolver {
-        public LowerCaseUnderscorePropertyNamesContractResolver() : base(true) { }
-
-        protected override JsonDictionaryContract CreateDictionaryContract(Type objectType) {
-            if (objectType != typeof(DataDictionary) && objectType != typeof(SettingsDictionary))
-                return base.CreateDictionaryContract(objectType);
-
-            JsonDictionaryContract contract = base.CreateDictionaryContract(objectType);
-            contract.PropertyNameResolver = propertyName => propertyName;
-            return contract;
-        }
-
-        protected internal override string ResolvePropertyName(string propertyName) {
-            return propertyName.ToLowerUnderscoredWords();
-        }
-    }
 }