mlist 0.1.16 → 0.1.17

data/CHANGELOG

@@ -1,3 +1,7 @@
+*0.1.17 [Bug Fix] (2010-05-04)
+
+* Fixed bug where rfc822 was missing. [mikehale]
+
 *0.1.16 [Bug Fix] (2010-04-13)
 
 * Fixed bug where encoding was lost when adding footers. [aiwilliams]

data/Gemfile

@@ -0,0 +1,17 @@
+source :gemcutter
+
+gem 'activerecord', '= 2.3.4'
+gem 'activesupport', '= 2.3.4'
+gem 'uuid', '>= 2.0.1'
+gem 'hpricot', '>= 0.8.2'
+gem 'tmail', '= 1.2.1'
+
+group :development do
+  gem 'jeweler'
+  gem 'dataset'
+  gem 'rake'
+  gem "sqlite3-ruby"
+  gem "rspec", :require => "spec"
+  gem "rr", "0.10.0"
+  gem "ruby-debug"
+end

data/Gemfile.lock

@@ -0,0 +1,97 @@
+--- 
+specs: 
+- rake: 
+    version: 0.8.7
+- activesupport: 
+    version: 2.3.4
+- activerecord: 
+    version: 2.3.4
+- columnize: 
+    version: 0.3.1
+- dataset: 
+    version: 1.3.2
+- json_pure: 
+    version: 1.4.2
+- gemcutter: 
+    version: 0.5.0
+- git: 
+    version: 1.2.5
+- hpricot: 
+    version: 0.8.2
+- rubyforge: 
+    version: 2.0.4
+- jeweler: 
+    version: 1.4.0
+- linecache: 
+    version: "0.43"
+- macaddr: 
+    version: 1.0.0
+- rr: 
+    version: 0.10.0
+- rspec: 
+    version: 1.3.0
+- ruby-debug-base: 
+    version: 0.10.3
+- ruby-debug: 
+    version: 0.10.3
+- sqlite3-ruby: 
+    version: 1.2.5
+- tmail: 
+    version: 1.2.1
+- uuid: 
+    version: 2.3.0
+hash: f114a3759c0acf377d0d1d01f36996ba24a98ef7
+sources: 
+- Rubygems: 
+    uri: http://gemcutter.org
+dependencies: 
+  ruby-debug: 
+    version: ">= 0"
+    group: 
+    - :development
+  sqlite3-ruby: 
+    version: ">= 0"
+    group: 
+    - :development
+  rr: 
+    version: = 0.10.0
+    group: 
+    - :development
+  activerecord: 
+    version: = 2.3.4
+    group: 
+    - :default
+  rspec: 
+    version: ">= 0"
+    require: 
+    - spec
+    group: 
+    - :development
+  rake: 
+    version: ">= 0"
+    group: 
+    - :development
+  tmail: 
+    version: = 1.2.1
+    group: 
+    - :default
+  uuid: 
+    version: ">= 2.0.1"
+    group: 
+    - :default
+  dataset: 
+    version: ">= 0"
+    group: 
+    - :development
+  jeweler: 
+    version: ">= 0"
+    group: 
+    - :development
+  hpricot: 
+    version: ">= 0.8.2"
+    group: 
+    - :default
+  activesupport: 
+    version: = 2.3.4
+    group: 
+    - :default

data/Rakefile

@@ -1,6 +1,7 @@
-$:.unshift(File.join(File.dirname(__FILE__), 'lib'))
-
 require 'rubygems'
+require "bundler"
+Bundler.setup
+
 require 'spec/rake/spectask'
 
 task :default => :spec
@@ -22,6 +23,7 @@ begin
     s.description = s.summary
     s.authors = ['Adam Williams']
   end
+  Jeweler::GemcutterTasks.new
 rescue LoadError
   puts "Jeweler not available. Install it with: sudo gem install technicalpickles-jeweler -s http://gems.github.com"
 end

data/VERSION.yml

@@ -1,5 +1,5 @@
 --- 
-:patch: 16
 :major: 0
 :build: 
 :minor: 1
+:patch: 17

data/lib/mlist/email_server/imap.rb

@@ -55,8 +55,10 @@ module MList
       end
 
       def process_message_id(id)
-        content = @imap.fetch(id, 'RFC822')[0].attr['RFC822']
-        process_message(content)
+        if rfc822 = @imap.fetch(id, 'RFC822')
+          content = rfc822[0].attr['RFC822']
+          process_message(content)
+        end
       end
 
       def process_message(content)

data/spec/integration/imap_email_server_spec.rb

@@ -55,6 +55,11 @@ describe MList::EmailServer::Imap, 'processing' do
     @imap.process_message_id(1)
   end
 
+  it 'should not blow up if an RFC822 does not exist' do
+    mock(@imap_server).fetch(1, 'RFC822') { nil }
+    lambda { @imap.process_message_id(1) }.should_not raise_error
+  end
+
   it 'should wrap up RFC822 content in a TMail::Mail object' do
     tmail = 'mock_tmail'
     mock(TMail::Mail).parse('email content') { tmail }

data/spec/models/mail_list_spec.rb

@@ -396,7 +396,7 @@ describe MList::MailList do
     it 'should append the list footer to text/plain emails' do
       @post_tmail.body = "My Email\n\n\n\n\n"
       mock(@manager_list).footer_content(is_a(MList::Message)) { 'my footer' }
-      process_post.body.should == "My Email\n\n\n\n\n#{MList::MailList::FOOTER_BLOCK_START}\nmy footer\n#{MList::MailList::FOOTER_BLOCK_END}=\n"
+      process_post.body.should == "My Email\n\n\n\n\n#{MList::MailList::FOOTER_BLOCK_START}\nmy footer\n#{MList::MailList::FOOTER_BLOCK_END}"
     end
 
     it 'should append the list footer to multipart/alternative, text/plain part of emails' do
@@ -414,7 +414,7 @@ describe MList::MailList do
     it 'should handle whitespace well when appending footer' do
       @post_tmail.body = "My Email"
       mock(@manager_list).footer_content(is_a(MList::Message)) { 'my footer' }
-      process_post.body.should == "My Email\n\n#{MList::MailList::FOOTER_BLOCK_START}\nmy footer\n#{MList::MailList::FOOTER_BLOCK_END}=\n"
+      process_post.body.should == "My Email\n\n#{MList::MailList::FOOTER_BLOCK_START}\nmy footer\n#{MList::MailList::FOOTER_BLOCK_END}"
     end
 
     it 'should strip out any existing text footers from the list' do
@@ -433,7 +433,7 @@ describe MList::MailList do
 this is without any in front
 #{MList::MailList::FOOTER_BLOCK_END}
       }
-      process_post.body.should == "My Email\n\n#{MList::MailList::FOOTER_BLOCK_START}\nmy footer\n#{MList::MailList::FOOTER_BLOCK_END}=\n"
+      process_post.body.should == "My Email\n\n#{MList::MailList::FOOTER_BLOCK_START}\nmy footer\n#{MList::MailList::FOOTER_BLOCK_END}"
     end
 
     it 'should strip out any existing html footers from the list' do

data/spec/spec_helper.rb

@@ -1,7 +1,11 @@
 require 'rubygems'
+require "bundler"
+Bundler.setup
+
 require 'spec'
 require 'rr'
 require 'ostruct'
+require 'ruby-debug'
 
 Spec::Runner.configure do |config|
   config.mock_with :rr
@@ -73,4 +77,4 @@ def visualize_email(email, recipient_address)
 end
 
 require 'mlist'
-require 'mlist/email_server/fake'
+require 'mlist/email_server/fake'

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 16
-  version: 0.1.16
+  - 17
+  version: 0.1.17
 platform: ruby
 authors: 
 - Adam Williams
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-13 00:00:00 -04:00
+date: 2010-05-04 00:00:00 -04:00
 default_executable: 
 dependencies: []
 
@@ -29,6 +29,8 @@ extra_rdoc_files:
 - TODO
 files: 
 - CHANGELOG
+- Gemfile
+- Gemfile.lock
 - README
 - Rakefile
 - TODO