strip incoming xml (fixes #16)

CHANGELOG.md

@@ -19,6 +19,8 @@ Please make sure to read the updated README for how to use the new version.
 * Refactoring: Removed the `Nori::Parser` module methods. After refactoring the rest,
   there was only a single method left for this module and that was moved to `Nori`.
 
+* Fix: [#16](https://github.com/savonrb/nori/issues/16) strip XML passed to Nori.
+
 ## 1.1.3 (2012-07-12)
 
 * Fix: Merged [pull request 21](https://github.com/savonrb/nori/pull/21) to fix an

lib/nori.rb

@@ -19,8 +19,11 @@ def initialize(options = {})
   end
 
   def parse(xml)
+    cleaned_xml = xml.strip
+    return {} if cleaned_xml.empty?
+
     parser = load_parser @options[:parser]
-    parser.parse(xml, @options)
+    parser.parse(cleaned_xml, @options)
   end
 
   private

lib/nori/parser/nokogiri.rb

@@ -35,8 +35,6 @@ def characters(string)
       end
 
       def self.parse(xml, options)
-        return {} if xml.strip.empty?
-
         document = Document.new
         document.options = options
         parser = ::Nokogiri::XML::SAX::Parser.new document

spec/nori/api_spec.rb

@@ -75,6 +75,13 @@
       Nori::Parser::Nokogiri.should_receive(:parse).once
       nori.parse("<any>thing</any>")
     end
+
+    it "strips the XML" do
+      xml = double("xml")
+      xml.should_receive(:strip).and_return("<any>thing</any>")
+
+      nori.parse(xml).should == { "any" => "thing" }
+    end
   end
 
   context "#parse without :advanced_typecasting" do