wraptext 0.1.3 → 0.1.4

data/lib/wraptext/parser.rb

@@ -44,8 +44,10 @@ module Wraptext
     def replace_single_breaks
       nodes = @root.xpath("//p//text()")
       nodes.each do |node|
-        frag = Nokogiri::HTML::DocumentFragment.parse node.content.gsub(/(\r\n|\n)/, "<br />")
-        node.swap frag
+        if node.content.match(/[\r\n]/)
+          frag = Nokogiri::HTML::DocumentFragment.parse node.content.gsub(/(\r\n|\n)/, "<br />")
+          node.swap frag
+        end
       end
 
       @root.xpath("//p").each do |node|
@@ -82,6 +84,7 @@ module Wraptext
     # simple_format is not appropriate here, as it does not consider block-level
     # html element context when performing substitutions.
     def reparent_nodes(top, parent)
+      return if parent.nil?
       for i in (0..parent.children.length - 1) do
         node = parent.children[i]
         # Block-level tags like <div> and <blockquote> should be traversed into.

data/spec/wraptext/parser_spec.rb

@@ -148,6 +148,21 @@ EOF
 <p>This is some <em>emphasized</em> text</p>
 <p>And here is <i>another</i> line</p>
 EOF
-    Wraptext::Parser.new(doc).to_html.should == expects.strip
+    specify { Wraptext::Parser.new(doc).to_html.should == expects.strip }
+  end
+
+  context "when passed an empty document" do
+    specify { expect { Wraptext::Parser.new("") }.to_not raise_error }
+  end
+
+  context "Given an article with two concurrent non-block tags" do
+    let(:article) {<<-EOF
+      A "Get the Facts" button sends you to a <em>Washington Post</em> <a href="http://www.washingtonpost.com/wp-srv/special/politics/campaign-finance/" target="_blank">article</a> about campaign finance.
+    EOF
+    }
+
+    it "should preserve spacing between non-block tags" do
+      Wraptext::Parser.new(article).to_html.should match("</em> <a")
+    end
   end
 end

data/wraptext.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |gem|
   gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   gem.name          = "wraptext"
   gem.require_paths = ["lib"]
-  gem.version       = "0.1.3"
+  gem.version       = "0.1.4"
 
   gem.add_dependency('nokogiri')
 end

metadata

@@ -1,25 +1,24 @@
 --- !ruby/object:Gem::Specification
 name: wraptext
 version: !ruby/object:Gem::Version
-  version: 0.1.3
   prerelease: 
+  version: 0.1.4
 platform: ruby
 authors:
 - Chris Heald
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-23 00:00:00.000000000 Z
+date: 2012-11-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: nokogiri
+  type: :runtime
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
@@ -27,6 +26,7 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+  name: nokogiri
 description: Wraps bare text nodes from an HTML document in <p> tags and splits text
   nodes on double newlines. Conveniently serves to format Wordpress post content properly
   as a side effect.
@@ -98,3 +98,4 @@ test_files:
 - spec/data/wordpress_autop.php
 - spec/spec_helper.rb
 - spec/wraptext/parser_spec.rb
+has_rdoc: