jeff 0.3.0 → 0.3.1

data/lib/jeff/document.rb

@@ -3,7 +3,10 @@ require 'nokogiri'
 module Jeff
   class Document < Nokogiri::XML::SAX::Document
     def characters(val)
-      (node['__content__'] ||= '') << val
+      val.strip!
+      unless val.empty?
+        (node['__content__'] ||= '') << val
+      end
     end
 
     def end_element(key)

data/lib/jeff/streamer.rb

@@ -7,7 +7,7 @@ module Jeff
     end
 
     def call(chunk, remaining_bytes, total_bytes)
-      @parser << chunk.sub(/\n/, '')
+      @parser << chunk.sub(/^\n/, '')
       @parser.finish if remaining_bytes == 0
     end
 

data/lib/jeff/version.rb

@@ -1,3 +1,3 @@
 module Jeff
-  VERSION = '0.3.0'
+  VERSION = '0.3.1'
 end

data/spec/jeff/document_spec.rb

@@ -11,7 +11,7 @@ module Jeff
           <Author>Felix Guattari</Author>
           <Creator Role="Translator">Robert Hurley</Creator>
         </ItemAttributes>
-      }.strip.gsub />\s+</, '><'
+      }.strip
     end
 
     let(:doc) { described_class.new }
@@ -40,6 +40,10 @@ module Jeff
         creator['Role'].should eql 'Translator'
         creator['__content__'].should eql 'Robert Hurley'
       end
+
+      it 'should ignore space between tags' do
+        should_not have_key '__content__'
+      end
     end
   end
 end

data/spec/jeff_spec.rb

@@ -155,5 +155,19 @@ describe Jeff do
         client.get(mock: true).status.should be 200
       end
     end
+
+    context 'given a malformed XML' do
+      before do
+        Excon.stub({ method: :get }) do |params|
+          { body: "\n<?xml version=\"1.0\" ?><foo/>" }
+        end
+      end
+
+      after { Excon.stubs.clear }
+
+      it 'should not raise an error' do
+        client.get mock: true
+      end
+    end
   end
 end

metadata

@@ -1,96 +1,96 @@
 --- !ruby/object:Gem::Specification
 name: jeff
 version: !ruby/object:Gem::Version
-  version: 0.3.0
-  prerelease: 
+  prerelease:
+  version: 0.3.1
 platform: ruby
 authors:
 - Hakan Ensari
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
 date: 2012-07-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: excon
-  requirement: !ruby/object:Gem::Requirement
-    none: false
+  version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: 0.14.0
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
     none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: 0.14.0
+    none: false
+  prerelease: false
+  type: :runtime
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: !ruby/object:Gem::Requirement
-    none: false
+  version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: '1.5'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
     none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: '1.5'
+    none: false
+  prerelease: false
+  type: :runtime
 - !ruby/object:Gem::Dependency
   name: guard-rspec
-  requirement: !ruby/object:Gem::Requirement
-    none: false
+  version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
     none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+    none: false
+  prerelease: false
+  type: :development
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: !ruby/object:Gem::Requirement
-    none: false
+  version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
     none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+    none: false
+  prerelease: false
+  type: :development
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    none: false
+  version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
     none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+    none: false
+  prerelease: false
+  type: :development
 description: Minimum-viable Amazon Web Services (AWS) client
 email:
 - hakan.ensari@papercavalier.com
@@ -120,26 +120,26 @@ files:
 - spec/spec_helper.rb
 homepage: https://github.com/hakanensari/jeff
 licenses: []
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
+required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+  none: false
 requirements: []
-rubyforge_project: 
-rubygems_version: 1.8.23
-signing_key: 
+rubyforge_project:
+rubygems_version: 1.8.24
+signing_key:
 specification_version: 3
 summary: AWS client
 test_files:
@@ -148,4 +148,4 @@ test_files:
 - spec/jeff/streamer_spec.rb
 - spec/jeff_spec.rb
 - spec/spec_helper.rb
-has_rdoc: 
+...