[MODEL] Changed, that TYPES_WITH_EMBEDDED_PROPERTIES is evaluated against a string version of the passed field type

Related: elastic#480

Author: karmi

elasticsearch-model/lib/elasticsearch/model/indexing.rb

@@ -52,7 +52,7 @@ def indexes(name, options={}, &block)
 
           if block_given?
             @mapping[name][:type] ||= 'object'
-            properties = TYPES_WITH_EMBEDDED_PROPERTIES.include?(@mapping[name][:type]) ? :properties : :fields
+            properties = TYPES_WITH_EMBEDDED_PROPERTIES.include?(@mapping[name][:type].to_s) ? :properties : :fields
 
             @mapping[name][properties] ||= {}
 

elasticsearch-model/test/unit/indexing_test.rb

@@ -127,6 +127,10 @@ class NotFound < Exception; end
           indexes :bar
         end
 
+        mappings.indexes :foo_nested_as_symbol, type: :nested do
+          indexes :bar
+        end
+
         # Object is the default when `type` is missing and there's a block passed
         #
         assert_equal 'object', mappings.to_hash[:mytype][:properties][:foo][:type]
@@ -140,6 +144,10 @@ class NotFound < Exception; end
         assert_equal 'nested', mappings.to_hash[:mytype][:properties][:foo_nested][:type]
         assert_equal 'string', mappings.to_hash[:mytype][:properties][:foo_nested][:properties][:bar][:type]
         assert_nil             mappings.to_hash[:mytype][:properties][:foo_nested][:fields]
+
+        assert_equal :nested, mappings.to_hash[:mytype][:properties][:foo_nested_as_symbol][:type]
+        assert_not_nil        mappings.to_hash[:mytype][:properties][:foo_nested_as_symbol][:properties]
+        assert_nil            mappings.to_hash[:mytype][:properties][:foo_nested_as_symbol][:fields]
       end
     end