Fix Response#to_hash bug lovehandle#1

lib/agent_cooper.rb

@@ -11,7 +11,7 @@ def self.configure(&block)
 
 require 'agent_cooper/config'
 require 'agent_cooper/response'
-require 'agent_cooper/builder'
+require 'agent_cooper/utils/nokogiri_hash'
 
 require 'agent_cooper/request'
 require 'agent_cooper/requests/finder'

lib/agent_cooper/response.rb

@@ -12,7 +12,7 @@ def body
 
     # @api public
     def to_hash
-      AgentCooper::Builder.from_xml(xml)
+      xml.to_hash
     end
 
     # @api public

lib/agent_cooper/utils/nokogiri_hash.rb

@@ -0,0 +1,33 @@
+module Nokogiri::XML
+
+  class Document
+
+    def to_hash
+      self.root.element? ? self.root.to_hash : {self.root.name => self.root.text}
+    end
+
+  end
+
+  class Element
+
+    def to_hash
+      { self.name => self.collect_attributes }
+    end
+
+    def collect_attributes
+      ({}).tap do |hash|
+        self.children.each do |child|
+          next if child.text?
+          if child.element?
+            if child.children.length > 1
+              hash[child.name] = child.collect_attributes
+            else
+              hash[child.name] = child.text.strip
+            end
+          end
+        end
+      end
+    end
+
+  end
+end

spec/agent_cooper/response_spec.rb

@@ -12,7 +12,7 @@
   its(:body)    { should eql(body) }
   its(:code)    { should eql(code) }
 
-  its(:to_hash) { should eql({"foo" => "bar"}) }
+  its(:to_hash) { should eql({"root" => {"foo" => "bar"}}) }
 
   its(:xml)     { should be_a(Nokogiri::XML::Document) }
   its(:valid?)  { should be_true }
@@ -22,4 +22,4 @@
     its(:valid?) { should be_false }
   end
 
-end
+end