summary 0.6.3 → 0.6.4

data/CHANGELOG

@@ -1,3 +1,5 @@
+v0.6.4. [Fix #2] If the strings are smaller than the terminator it should not be applied
+
 v0.6.3. [Fix #3] Exception when the word can't be summarized
 
 v0.6.2. README review

data/lib/summary.rb

@@ -38,21 +38,31 @@ module Summary
     def summary
       return pure if pure.size < @size or pure !~ /\s/
       
-  	  @text = pure[0...pure[0..(@size-backspace)].rindex(' ')]
-  	  @text.gsub(/\.$/,'') + @terminator
+      limit = pure[0..(@size-backspace)].rindex(' ')
+      
+      if limit >= purify(@terminator).size
+  	    text = pure[0...limit]
+  	    text = text.gsub(/\.$/,'') + @terminator
+      else
+        pure
+      end
     end
 
     protected
     # It cleans up the text removing the html tags, break lines and white spaces
     def pure
-      @pure ||= @text.strip.gsub(/<(.|\n)+?>|(\t|\n|\r)+/,'').gsub(/\s+/,' ')
+      @pure ||= purify @text
+    end
+    
+    def purify(string)
+      string.strip.gsub(/<(.|\n)+?>|(\t|\n|\r)+/,'').gsub(/\s+/,' ')
     end
 
     # Measures the space needed by the terminator.
     # Let's say you want a string with 50 chars, and your terminator is a '...'.
     # That means your string can only have 47 chars + 3 chars from your terminator.
     def backspace
-      (@terminator =~ /<(.|\n)+?>/ ? @terminator.gsub(/<(.|\n)+?>/,'') : @terminator).size
+      (@terminator =~ /<(.|\n)+?>/ ? purify(@terminator) : @terminator).size
     end
   end
 end

data/spec/summary_spec.rb

@@ -34,7 +34,7 @@ describe Summary, 'on plain text' do
   end
 end
 
-describe Summary, "when the string is thin" do
+describe Summary, "when the string is a word" do
   before do
     @text = "test"
   end
@@ -46,6 +46,16 @@ describe Summary, "when the string is thin" do
   end
 end
 
+describe Summary, "when the terminator is bigger" do
+  before do
+    @text = "test test"
+  end
+  
+  it "should not summarize the string" do
+    @text.summary(4, '.' * 5).should == @text
+  end
+end
+
 describe Summary, 'on html' do
   before do
     @text = 'Cras <strong>eleifend sodales sem</strong>. <img src="image.jpg" /><h1>Ut</h1> nec metus. <br />Nulla sed<p> nisl. Praesent malesuada</p> dui rhoncus quam. Aliquam ultricies. Praesent commodo. Nam nec pede. Pellentesque egestas, urna vel accumsan venenatis, mi metus sagittis augue, <strong>sed</strong> hendrerit nunc felis eget neque. Fusce vel massa. Quisque ligula enim, tempor sed, lacinia non, malesuada ac, enim. Curabitur at enim scelerisque neque egestas luctus. Mauris lacinia varius diam.'

data/summary.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{summary}
-  s.version = "0.6.3"
+  s.version = "0.6.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 1.2") if s.respond_to? :required_rubygems_version=
   s.authors = ["Bruno Azisaka Maciel"]

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: summary
 version: !ruby/object:Gem::Version 
-  hash: 1
+  hash: 15
   prerelease: false
   segments: 
   - 0
   - 6
-  - 3
-  version: 0.6.3
+  - 4
+  version: 0.6.4
 platform: ruby
 authors: 
 - Bruno Azisaka Maciel