catch date strings above quoted regions when separated by a blank line

Author: technoweenie

lib/email_reply_parser.rb

@@ -74,7 +74,7 @@ def read(text)
       @fragment = nil
 
       # Use the StringScanner to pull out each line of the email content.
-      @scanner  = StringScanner.new(text)
+      @scanner = StringScanner.new(text)
       while line = @scanner.scan_until(/\n/)
         scan_line(line)
       end
@@ -129,7 +129,7 @@ def scan_line(line)
       # it doesn't start with `>`.
       if @fragment &&
           ((@fragment.quoted? == is_quoted) ||
-           (@fragment.quoted? && quote_header?(line)))
+           (@fragment.quoted? && (quote_header?(line) || line == EMPTY)))
         @fragment.lines << line
 
       # Otherwise, finish the fragment and start a new one.

test/email_reply_parser_test.rb

@@ -63,6 +63,15 @@ def test_reads_bottom_post
     assert_match /^_/, reply.fragments[5].to_s
   end
 
+  def test_recognizes_date_string_above_quote
+    reply = email :email_1_4
+
+    assert_match /^Awesome/, reply.fragments[0].to_s
+    assert_match /^On/,      reply.fragments[1].to_s
+    assert_match /Loader/,   reply.fragments[1].to_s
+    puts reply.fragments[1].to_s.inspect
+  end
+
   def email(name)
     body = IO.read EMAIL_FIXTURE_PATH.join("#{name}.txt").to_s
     EmailReplyParser.read body

test/emails/email_1_4.txt

@@ -0,0 +1,5 @@
+Awesome! I haven't had another problem with it.
+
+On Aug 22, 2011, at 7:37 PM, defunkt<reply@reply.github.com> wrote:
+
+> Loader seems to be working well.