catch mobile signatures like "Sent from my Verizon Wireless BlackBerry"

Author: Drew Batshaw

lib/email_reply_parser.rb

@@ -126,7 +126,7 @@ def scan_line(line)
       # Mark the current Fragment as a signature if the current line is empty
       # and the Fragment starts with a common signature indicator.
       if @fragment && line == EMPTY
-        if @fragment.lines.last =~ /([\-\_]\s*$)|(^\w+ #{"Sent from my".reverse}$)/
+        if @fragment.lines.last =~ /([\-\_]\s*$)|(^(\w+\s*){1,3} #{"Sent from my".reverse}$)/  #will catch up to 3 words after "sent from my "
           @fragment.signature = true
           finish_fragment
         end

test/email_reply_parser_test.rb

@@ -97,6 +97,16 @@ def test_parse_out_sent_from_BlackBerry
     assert_equal "Here is another email", EmailReplyParser.parse_reply(body)
   end
 
+  def test_parse_out_send_from_multiword_mobile_device
+    body = IO.read EMAIL_FIXTURE_PATH.join("email_multi_word_sent_from_my_mobile_device.txt").to_s
+    assert_equal "Here is another email", EmailReplyParser.parse_reply(body)
+  end
+
+  def test_do_not_parse_out_send_from_in_regular_sentence
+    body = IO.read EMAIL_FIXTURE_PATH.join("email_sent_from_my_not_signature.txt").to_s
+    assert_equal "Here is another email\n\nSent from my desk, is much easier then my mobile phone.", EmailReplyParser.parse_reply(body)
+  end
+
 
   def test_parse_reply
     body = IO.read EMAIL_FIXTURE_PATH.join("email_1_2.txt").to_s

test/emails/email_multi_word_sent_from_my_mobile_device.txt

@@ -0,0 +1,3 @@
+Here is another email
+
+Sent from my Verizon Wireless BlackBerry

test/emails/email_sent_from_my_not_signature.txt

@@ -0,0 +1,3 @@
+Here is another email
+
+Sent from my desk, is much easier then my mobile phone.