Merge pull request #17 from mezuro/generalize_nested_object_hash_conv…

…ersion

Generalize hash conversion

CHANGELOG.rdoc

@@ -4,6 +4,8 @@ Likeno is a library for remote data model access over HTTP
 
 == Unreleased
 
+* Generalize Hash conversion for non Likeno::Entity instances
+
 == v1.0.1 - 23/03/2016
 
 * Fix hash conversion

lib/likeno/helpers/hash_converters.rb

@@ -25,7 +25,7 @@ def date_time_to_s(date)
     def convert_to_hash(value)
       return value if value.nil?
       return value.collect { |element| convert_to_hash(element) } if value.is_a? Array
-      return value.to_hash if value.is_a?(Likeno::Entity)
+      return value.to_hash if value.respond_to?(:to_hash)
       return date_time_to_s(value) if value.is_a? DateTime
       return 'INF' if value.is_a?(Float) && value.infinite? == 1
       return '-INF' if value.is_a?(Float) && value.infinite? == -1

spec/likeno/helpers/hash_converters_spec.rb

@@ -18,6 +18,14 @@
 
 include Likeno::HashConverters
 
+class Another
+  attr_accessor :attr
+
+  def to_hash
+    {'attr' => self.attr}
+  end
+end
+
 describe Likeno::HashConverters do
   describe 'date_time_to_s' do
     context 'with 21/12/1995 (first Ruby publication)' do
@@ -74,6 +82,18 @@
         expect(convert_to_hash(-1.0 / 0.0)).to eq('-INF')
       end
     end
+
+    context 'without a base class that responds to to_hash' do
+      let(:obj) { Another.new }
+
+      before do
+        obj.attr = "attribute"
+      end
+
+      it 'is expected to do the conversion' do
+        expect(convert_to_hash(obj)).to eq({'attr' => obj.attr})
+      end
+    end
   end
 
   describe 'field_to_hash' do