[MODEL] Fixed incorrect delegation to @Result Hash in Response::Result

Author: karmi

elasticsearch-model/lib/elasticsearch/model/response/result.rb

@@ -22,9 +22,9 @@ def initialize(attributes={})
         def method_missing(method_name, *arguments)
           case
           when @result.respond_to?(method_name.to_sym)
-            @result[method_name.to_sym]
+            @result.__send__ method_name.to_sym, *arguments
           when @result._source && @result._source.respond_to?(method_name.to_sym)
-            @result._source[method_name.to_sym]
+            @result._source.__send__ method_name.to_sym, *arguments
           else
             super
           end
@@ -33,8 +33,8 @@ def method_missing(method_name, *arguments)
         # Respond to methods from `@result` or `@result._source`
         #
         def respond_to?(method_name, include_private = false)
-          @result.has_key?(method_name.to_sym) || \
-          @result._source && @result._source.has_key?(method_name.to_sym) || \
+          @result.respond_to?(method_name.to_sym) || \
+          @result._source && @result._source.respond_to?(method_name.to_sym) || \
           super
         end
 

elasticsearch-model/test/unit/response_result_test.rb

@@ -27,5 +27,18 @@ class Elasticsearch::Model::ResultTest < Test::Unit::TestCase
       assert_equal 'baz', result.bar
     end
 
+    should "delegate methods to @result" do
+      result = Elasticsearch::Model::Response::Result.new foo: 'bar'
+
+      assert_equal 'bar', result.foo
+      assert_equal 'bar', result.fetch('foo')
+      assert_equal 'moo', result.fetch('NOT_EXIST', 'moo')
+
+      assert_respond_to result, :to_hash
+      assert_equal({'foo' => 'bar'}, result.to_hash)
+
+      assert_raise(NoMethodError) { result.does_not_exist }
+    end
+
   end
 end