fixes enums being skipped when calling MapFromAttributes()

Author: Mpdreamz

src/Nest/Resolvers/Writers/TypeMappingWriter.cs

@@ -136,7 +136,7 @@ internal void WriteProperties(JsonWriter jsonWriter)
 
 				var propertyName = this.Infer.PropertyName(p);
 				var type = GetElasticSearchType(att, p);
-				
+
 				if (type == null) //could not get type from attribute or infer from CLR type.
 					continue;
 
@@ -234,8 +234,8 @@ private string GetElasticSearchTypeFromFieldType(FieldType? fieldType)
 					return "boolean";
 				case FieldType.Completion:
 					return "completion";
-        case FieldType.Nested:
-          return "nested";
+				case FieldType.Nested:
+					return "nested";
 				case FieldType.Object:
 					return "object";
 				default:
@@ -255,6 +255,9 @@ private string GetElasticSearchTypeFromFieldType(FieldType? fieldType)
 			if (propertyType == typeof(string))
 				return FieldType.String;
 
+			if (propertyType.IsEnum)
+				return FieldType.Integer;
+
 			if (propertyType.IsValueType)
 			{
 				switch (propertyType.Name)
@@ -284,7 +287,7 @@ private static Type GetUnderlyingType(Type type)
 			if (type.IsArray)
 				return type.GetElementType();
 
-            if (type.IsGenericType && type.GetGenericArguments().Length == 1 && (type.GetInterface("IEnumerable") != null || Nullable.GetUnderlyingType(type) != null))
+			if (type.IsGenericType && type.GetGenericArguments().Length == 1 && (type.GetInterface("IEnumerable") != null || Nullable.GetUnderlyingType(type) != null))
 				return type.GetGenericArguments()[0];
 
 			return type;

src/Tests/Nest.Tests.Unit/Core/Map/Enums/EnumCanBeOverriddenAfterMapFromAttributes.json

@@ -0,0 +1,10 @@
+{
+  "myclass": {
+    "properties": {
+      "myEnum": {
+        "index": "not_analyzed",
+        "type": "string"
+      }
+    }
+  }
+}

src/Tests/Nest.Tests.Unit/Core/Map/Enums/EnumMappingTests.cs

@@ -0,0 +1,49 @@
+using NUnit.Framework;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Reflection;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace Nest.Tests.Unit.Core.Map.Enums
+{
+	[TestFixture]
+	public class EnumMappingTests : BaseJsonTests
+	{
+		private class MyClass
+		{
+			public MyEnum MyEnum { get; set; }
+		}
+
+		private enum MyEnum
+		{
+			Value1,
+			Value2
+		}
+
+		[Test]
+		public void EnumShouldMapToIntByDefault()
+		{
+			var result = this._client.Map<MyClass>(m => m.MapFromAttributes());
+			this.JsonEquals(result.ConnectionStatus.Request, MethodBase.GetCurrentMethod());
+		}
+
+		[Test]
+		public void EnumCanBeOverriddenAfterMapFromAttributes()
+		{
+			var result = this._client.Map<MyClass>(m => m
+				.MapFromAttributes()
+				.Properties(props=>props
+					.String(s=>s
+						.Name(p=>p.MyEnum)
+						.Index(FieldIndexOption.NotAnalyzed)
+					)
+				)
+			);
+			this.JsonEquals(result.ConnectionStatus.Request, MethodBase.GetCurrentMethod());
+		}
+
+	}
+
+}

src/Tests/Nest.Tests.Unit/Core/Map/Enums/EnumShouldMapToIntByDefault.json

@@ -0,0 +1,9 @@
+{
+  "myclass": {
+    "properties": {
+      "myEnum": {
+        "type": "integer"
+      }
+    }
+  }
+}

src/Tests/Nest.Tests.Unit/Nest.Tests.Unit.csproj

@@ -107,6 +107,7 @@
     <Compile Include="Core\Indices\Analysis\TokenFilters\TokenFilterTests.cs" />
     <Compile Include="Core\Indices\Similarity\SimilarityTests.cs" />
     <Compile Include="Core\Map\CustomMapping\ImagePluginMappingTests.cs" />
+    <Compile Include="Core\Map\Enums\EnumMappingTests.cs" />
     <Compile Include="Core\Map\GenericTypes\GenericTypeMappingTests.cs" />
     <Compile Include="Core\Map\GeoShape\GeoShapeMappingTests.cs" />
     <Compile Include="Core\Map\GetMappingSerializationTests.cs" />
@@ -176,6 +177,12 @@
     <None Include="Core\Map\CustomMapping\RepresentUnknowImageMappingPlugin.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
+    <None Include="Core\Map\Enums\EnumCanBeOverriddenAfterMapFromAttributes.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
+    <None Include="Core\Map\Enums\EnumShouldMapToIntByDefault.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
     <None Include="Core\Map\GenericTypes\GenericTypeMapping.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
@@ -392,7 +399,7 @@
     <Compile Include="Reproduce\Reproduce1146Tests.cs" />
     <Compile Include="Reproduce\Reproduce629Tests.cs" />
     <Compile Include="Reproduce\Reproduce1187Tests.cs" />
-    <Compile Include="Reproduce\Reproduce991Tests.cs" />
+    <Compile Include="Reproduce\Reproduce901Tests.cs" />
     <Compile Include="Reproduce\Reproduce990Tests.cs" />
     <Compile Include="Reproduce\Reproduce974Tests.cs" />
     <Compile Include="Reproduce\Reproduce928Tests.cs" />
@@ -819,6 +826,9 @@
     <None Include="Reproduce\Issue1199Mapping.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
+    <None Include="Reproduce\EnumQueryDefaultsToInt.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
     <None Include="Reproduce\Issue928.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>

src/Tests/Nest.Tests.Unit/Reproduce/Reproduce991Tests.cs renamed to src/Tests/Nest.Tests.Unit/Reproduce/Reproduce901Tests.cs

@@ -12,7 +12,7 @@ namespace Nest.Tests.Unit.Reproduce
 	/// tests to reproduce reported errors
 	/// </summary>
 	[TestFixture]
-	public class Reproduce991Tests : BaseJsonTests
+	public class Reproduce901Tests : BaseJsonTests
 	{
 		private class MyClass
 		{
@@ -32,5 +32,6 @@ public void EnumQueryDefaultsToInt()
 				.Query(q => q.Term(p => p.MyEnum, MyEnum.Value2));
 			this.JsonEquals(query, MethodBase.GetCurrentMethod());
 		}
+
 	}
 }