fix failing initializer tests since it does not automap, eliminated unreachable code in FieldValues

Author: Mpdreamz

src/Nest/CommonAbstractions/Fields/FieldValues.cs

@@ -75,8 +75,6 @@ private K FieldArray<K>(string field)
 						using (var sr = new StringReader(s))
 						using (var jr = new JsonTextReader(sr) { DateParseHandling = DateParseHandling.None })
 							return ForceNoDateInferrence.Deserialize<K>(jr);
-
-						return array.Root.ToObject<K>(ForceNoDateInferrence);
 					}
 					return array.ToObject<K>();
 				}

src/Tests/Indices/MappingManagement/PutMapping/PutMappingApiTest.cs

@@ -187,7 +187,6 @@ protected override LazyResponses ClientUsage() => Calls(
 		{
 			Properties = new Properties<Project>
 			{
-
 				{ p => p.CuratedTags, new ObjectProperty
 						{
 							Properties = new Properties<Tag>
@@ -227,6 +226,7 @@ protected override LazyResponses ClientUsage() => Calls(
 				{ p => p.Name, new StringProperty { Index = FieldIndexOption.NotAnalyzed }  },
 				{ p => p.NumberOfCommits, new NumberProperty(NumberType.Integer) { Index = NonStringIndexOption.NotAnalyzed } },
 				{ p => p.StartedOn, new DateProperty { Index = NonStringIndexOption.No } },
+				{ p => p.DateString, new StringProperty { } },
 				{ p => p.State, new NumberProperty(NumberType.Integer) },
 				{ p => p.Suggest, new CompletionProperty() },
 				{ p => p.Tags, new ObjectProperty