email_reply_parser 0.5.10 → 0.5.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1d5b9fdfe30fe4feec4e1b84c68b25180b5da4cbe6aa8cebae7d7b725c6918c6
-  data.tar.gz: e046ee4ce446ee8a54d2e669ae1cd10bc0ae7e5f9e55c28d439991f7c8311965
+  metadata.gz: 19456eb75469b983f8db4a0807f6691213264309223f37d02c7fcf44748e2d7c
+  data.tar.gz: 0a67068492fd8c2fe7221420e383f69f7ad0f6e7f435882830d8553e6f917bcf
 SHA512:
-  metadata.gz: 1e4680771a08970cf019ad1bbeb7a2de56b34df351cca6a6a10aaeebff681eff71e4e2fce7809667ab9e5d1505506d5835d8e180dae731825c97a1ef3ae30cf8
-  data.tar.gz: cef0b121c8c201116348fa880274191b1020ed189c11b0c49a8f13e5d00b619a40aeac8eafe1fb09e908e9ff179d3b47692212364192fe3d7adde33a2c6cfc5f
+  metadata.gz: e6b065c74ce3eec383d570824c3c22a0ff9fc1cbc17a99cbbac230a92a0eb9c90adcc96f6288891a676ff65da1a791d11b7c1a3cce9a22bacb3752c6fab6abf0
+  data.tar.gz: 72e65bff3a0850354268451e89ea352a8ab59c5bb8fe1c27ff7558df740ecd233915daaf5e35830a4c9c07617bc5ce6634a0c50aa5d3bd3464821c3328824e7e

data/email_reply_parser.gemspec

@@ -54,9 +54,11 @@ Gem::Specification.new do |s|
     test/emails/email_1_8.txt
     test/emails/email_2_1.txt
     test/emails/email_2_2.txt
+    test/emails/email_2_3.txt
     test/emails/email_BlackBerry.txt
     test/emails/email_bullets.txt
     test/emails/email_iPhone.txt
+    test/emails/email_long_quote.txt
     test/emails/email_multi_word_sent_from_my_mobile_device.txt
     test/emails/email_one_is_not_on.txt
     test/emails/email_sent_from_my_not_signature.txt

data/lib/email_reply_parser.rb

@@ -30,7 +30,7 @@ require 'strscan'
 #
 # [mail]: https://github.com/mikel/mail
 class EmailReplyParser
-  VERSION = "0.5.10"
+  VERSION = "0.5.11"
 
   # Public: Splits an email body into a list of Fragments.
   #
@@ -147,9 +147,9 @@ class EmailReplyParser
       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.

data/test/email_reply_parser_test.rb

@@ -2,6 +2,7 @@ require 'rubygems'
 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 @@ I am currently using the Java HTTP API.\n", reply.fragments[0].to_s
     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

data/test/emails/email_2_3.txt

@@ -0,0 +1,10 @@
+Outlook with a reply directly above line
+
+From: CRM Comments [crm-comment@example.com]
+Sent: Friday, 23 March 2012 5:08 p.m.
+To: John S. Greene
+Subject: [contact:106] John Greene
+
+> A new comment has been added to the Contact named 'John Greene':
+>
+> I am replying to a comment.