test/test_header_parsing.rb in sup-1.0 vs test/test_header_parsing.rb in sup-1.1

@@ -9,13 +9,19 @@
 class TestMBoxParsing < Minitest::Test
 
   def setup
     @path = Dir.mktmpdir
     @mbox = File.join(@path, 'test_mbox')
+    @log = StringIO.new
+    Redwood::Logger.add_sink @log
+    Redwood::Logger.remove_sink $stderr
   end
 
   def teardown
+    Redwood::Logger.clear!
+    Redwood::Logger.remove_sink @log
+    Redwood::Logger.add_sink $stderr
     FileUtils.rm_r @path
   end
 
   def test_normal_headers
     h = Source.parse_raw_email_header StringIO.new(<<EOS)
@@ -67,11 +73,11 @@
     end
   end
 
   def test_blank_lines
     h = Source.parse_raw_email_header StringIO.new("")
-    assert_equal nil, h["message-id"]
+    assert_nil h["message-id"]
   end
 
   def test_empty_headers
     variants = [
       "Message-Id:       \n",
@@ -131,9 +137,11 @@
 EOS
     offset = l.next_offset 0
     assert_equal 61, offset
     offset = l.next_offset 61
     assert_nil offset
+    assert_match(/WARNING: found invalid date in potential mbox split line, not splitting/,
+                 @log.string)
   end
 
   def test_more_from_line_splitting
     l = MBox.new mbox_for_string(<<EOS)
 From [e-mail] Mon Apr 27 12:56:18 2009