Merge pull request #81 from github/zkoppert-timeout

Add timeout to prevent infinite run

Author: zkoppert

lib/email_reply_parser.rb

@@ -147,9 +147,9 @@ def scan_line(line)
       line.chomp!("\n")
       line.lstrip! unless SIG_REGEX.match(line)
 
-      # We're looking for leading `>`'s to see if this line is part of a
+      # We're looking for a leading `>` to see if this line is part of a
       # quoted Fragment.
-      is_quoted = !!(line =~ /(>+)$/)
+      is_quoted = !!(line =~ /(>)$/)
 
       # Mark the current Fragment as a signature if the current line is empty
       # and the Fragment starts with a common signature indicator.

test/email_reply_parser_test.rb

@@ -2,6 +2,7 @@
 require 'test/unit'
 require 'pathname'
 require 'pp'
+require 'timeout'
 
 dir = Pathname.new File.expand_path(File.dirname(__FILE__))
 require dir + '..' + 'lib' + 'email_reply_parser'
@@ -222,6 +223,12 @@ def test_doesnt_remove_signature_delimiter_in_mid_line
     assert_equal 1, reply.fragments.size
   end
 
+  def test_long_quote_processing_completes
+    reply = Timeout.timeout(1) { email(:email_long_quote) }
+
+    assert_equal 5, reply.fragments.size
+  end
+
   def email(name)
     body = IO.read EMAIL_FIXTURE_PATH.join("#{name}.txt").to_s
     EmailReplyParser.read body