[DRE-commits] [SCM] sup-mail.git branch, master, updated. debian/0.12.1-1-9-gf1e4d87
Per Andersson
avtobiff at gmail.com
Fri May 18 22:47:45 UTC 2012
The following commit has been merged in the master branch:
commit d95e2502d021f1cd49164e4e27ba0b570e44d0d3
Author: Per Andersson <avtobiff at gmail.com>
Date: Sat May 19 00:43:00 2012 +0200
Patch tests so they all pass.
diff --git a/debian/patches/0003-Update-tests-so-they-all-pass.patch b/debian/patches/0003-Update-tests-so-they-all-pass.patch
new file mode 100644
index 0000000..5888da2
--- /dev/null
+++ b/debian/patches/0003-Update-tests-so-they-all-pass.patch
@@ -0,0 +1,178 @@
+From: Per Andersson <avtobiff at gmail.com>
+Date: Fri, 18 May 2012 23:52:55 +0200
+Subject: Update tests so they all pass!
+
+---
+ test/dummy_source.rb | 4 ++--
+ test/test_header_parsing.rb | 29 +++++++++++++++++++++--------
+ test/test_message.rb | 27 +++++++++++++++++++--------
+ 3 files changed, 42 insertions(+), 18 deletions(-)
+
+diff --git a/test/dummy_source.rb b/test/dummy_source.rb
+index da26e44..ec4debf 100644
+--- a/test/dummy_source.rb
++++ b/test/dummy_source.rb
+@@ -11,8 +11,8 @@ class DummySource < Source
+
+ attr_accessor :messages
+
+- def initialize uri, last_date=nil, usual=true, archived=false, id=nil, labels=[]
+- super uri, last_date, usual, archived, id
++ def initialize uri, usual=true, archived=false, id=nil
++ super uri, usual, archived, id
+ @messages = nil
+ end
+
+diff --git a/test/test_header_parsing.rb b/test/test_header_parsing.rb
+index 1929e07..60eb968 100644
+--- a/test/test_header_parsing.rb
++++ b/test/test_header_parsing.rb
+@@ -106,7 +106,8 @@ EOS
+ end
+
+ def test_from_line_splitting
+- l = MBox.new StringIO.new(<<EOS)
++ tmpfile = Tempfile.new('test_from_line_splitting', '.')
++ tmpfile.puts(<<EOS)
+ From sup-talk-bounces at rubyforge.org Mon Apr 27 12:56:18 2009
+ From: Bob <bob at bob.com>
+ To: a dear friend
+@@ -125,14 +126,21 @@ From bob at bob.com
+
+ This is the end of the email.
+ EOS
+- offset, labels = l.next
+- assert_equal 0, offset
+- offset, labels = l.next
++ tmpfile.seek 0
++
++ l = MBox.new tmpfile
++ offset = l.next_offset 0
++ # I assume it is meant to actually split on the From: line
++ assert_equal 61, offset
++ offset = l.next_offset offset
+ assert_nil offset
++
++ tmpfile.close!
+ end
+
+ def test_more_from_line_splitting
+- l = MBox.new StringIO.new(<<EOS)
++ tmpfile = Tempfile.new('test_more_from_line_splitting', '.')
++ tmpfile.puts(<<EOS)
+ From sup-talk-bounces at rubyforge.org Mon Apr 27 12:56:18 2009
+ From: Bob <bob at bob.com>
+ To: a dear friend
+@@ -145,13 +153,18 @@ To: a dear friend
+
+ Hello again! Would you like to buy my products?
+ EOS
+- offset, labels = l.next
++ tmpfile.seek 0
++
++ l = MBox.new tmpfile
++ offset = l.next_offset 0
+ assert_not_nil offset
+
+- offset, labels = l.next
++ offset = l.next_offset offset
+ assert_not_nil offset
+
+- offset, labels = l.next
++ offset = l.next_offset offset
+ assert_nil offset
++
++ tmpfile.close!
+ end
+ end
+diff --git a/test/test_message.rb b/test/test_message.rb
+index 94b962a..f7c8279 100644
+--- a/test/test_message.rb
++++ b/test/test_message.rb
+@@ -29,9 +29,14 @@ module Redwood
+ class TestMessage < Test::Unit::TestCase
+
+ def setup
++ @path = Dir.mktmpdir
++ HookManager.init File.join(@path, 'hooks')
+ end
+
+ def teardown
++ FileUtils.rm_r @path if passed?
++ puts "not cleaning up #{@path}" unless passed?
++ HookManager.deinstantiate!
+ end
+
+ def test_simple_message
+@@ -72,7 +77,8 @@ EOS
+ source.messages = [ message ]
+ source_info = 0
+
+- sup_message = Message.new( {:source => source, :source_info => source_info } )
++ location = Location.new(source, source_info)
++ sup_message = Message.new( {:locations => [location] } )
+ sup_message.load_from_source!
+
+ # see how well parsing the header went
+@@ -222,7 +228,8 @@ EOS
+ source.messages = [ message ]
+ source_info = 0
+
+- sup_message = Message.new( {:source => source, :source_info => source_info } )
++ location = Location.new(source, source_info)
++ sup_message = Message.new( {:locations => [location] } )
+ sup_message.load_from_source!
+
+ # read the message body chunks
+@@ -272,7 +279,8 @@ EOS
+ source.messages = [ message ]
+ source_info = 0
+
+- sup_message = Message.new( {:source => source, :source_info => source_info } )
++ location = Location.new(source, source_info )
++ sup_message = Message.new( {:locations => [location] } )
+ sup_message.load_from_source!
+
+ to = sup_message.to
+@@ -318,7 +326,8 @@ EOS
+ source.messages = [ message ]
+ source_info = 0
+
+- sup_message = Message.new( {:source => source, :source_info => source_info } )
++ location = Location.new(source, source_info)
++ sup_message = Message.new( {:locations => [location] } )
+ sup_message.load_from_source!
+
+ # read the message body chunks: no errors should reach this level
+@@ -417,7 +426,8 @@ EOS
+ source.messages = [ message ]
+ source_info = 0
+
+- sup_message = Message.new( {:source => source, :source_info => source_info } )
++ location = Location.new(source, source_info)
++ sup_message = Message.new( {:locations => [location] } )
+ sup_message.load_from_source!
+
+ # read the message body chunks
+@@ -508,7 +518,8 @@ EOS
+ source.messages = [ message ]
+ source_info = 0
+
+- sup_message = Message.new( {:source => source, :source_info => source_info } )
++ location = Location.new(source, source_info)
++ sup_message = Message.new( {:locations => [location] } )
+ sup_message.load_from_source!
+
+ # See how well parsing the message ID went.
+@@ -517,8 +528,8 @@ EOS
+
+ # Look at another header field whose first line was blank.
+ list_unsubscribe = sup_message.list_unsubscribe
+- assert_equal("<http://mailman2.widget.com/mailman/listinfo/monitor-list>, " +
+- "<mailto:monitor-list-request at widget.com?subject=unsubscribe>",
++ assert_equal("<http://mailman2.widget.com/mailman/listinfo/monitor-list>,\n" +
++ " \t<mailto:monitor-list-request at widget.com?subject=unsubscribe>",
+ list_unsubscribe)
+
+ end
+--
diff --git a/debian/patches/series b/debian/patches/series
index 500fed9..4378afd 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
0001-Patch-out-all-requires-of-rubygems.patch
0002-Fix-support-for-gpgme-2.0.patch
+0003-Update-tests-so-they-all-pass.patch
--
sup-mail.git
More information about the Pkg-ruby-extras-commits
mailing list