mlist 0.1.18 → 0.1.19

data/CHANGELOG

@@ -1,3 +1,7 @@
+*0.1.19 [Enhancement] (2010-08-12)
+
+* Requiring subscriber instances to respond to 'rfc5322_email' instead of 'email_address' in order to allow the latter name to be used in associations. [aiwilliams]
+
 *0.1.18 [Enhancement] (2010-06-11)
 
 * Moving mail that has no rfc822 content into a settings[:failure_folder], if that setting is provided. [aiwilliams]

data/VERSION.yml

@@ -1,5 +1,5 @@
 --- 
-:patch: 18
+:patch: 19
 :major: 0
 :build: 
 :minor: 1

data/lib/mlist/email_subscriber.rb

@@ -1,6 +1,6 @@
 module MList
-  
+
   # Represents a simple subscriber instance, wrapping an email address.
   #
-  EmailSubscriber = Struct.new('EmailSubscriber', :email_address)
-end
+  EmailSubscriber = Struct.new('EmailSubscriber', :rfc5322_email)
+end

data/lib/mlist/list.rb

@@ -1,22 +1,22 @@
 module MList
-  
+
   # Represents the interface of the lists that a list manager must answer.
   # This is distinct from the MList::MailList to allow for greater flexibility
   # in processing email coming to a list - that is, whatever you include this
   # into may re-define behavior appropriately.
   #
-  # Your 'subscriber' instances MUST respond to :email_address. They may
-  # optionally respond to :display_name.
+  # Your 'subscriber' instances MUST respond to :rfc5322_email. They
+  # may optionally respond to :display_name.
   #
   module List
-    
+
     # Answers whether this list is active or not. All lists are active all the
     # time by default.
     #
     def active?
       true
     end
-    
+
     # Answers whether the email has been to this list before. The simplest
     # test is whether the email has an X-BeenThere header that matches this
     # list's address.
@@ -24,28 +24,28 @@ module MList
     def been_here?(email)
       email.been_there_addresses.include?(address.downcase)
     end
-    
+
     # Answers whether the subscriber is blocked from posting or not. This will
     # not be asked when the list is not active (answers _active?_ as false).
     #
     def blocked?(subscriber)
       false
     end
-    
+
     # Answers the footer content for this list. Default implementation is very
     # simple.
     #
     def footer_content(message)
       %Q{The "#{label}" mailing list\nPost messages: #{post_url}}
     end
-    
+
     # Answer a suitable label for the list, which will be used in various
     # parts of content that is delivered to subscribers, etc.
     #
     def label
       raise 'answer the list label'
     end
-    
+
     # Answers the headers that are to be included in the emails delivered for
     # this list. Any entries that have a nil value will not be included in the
     # delivered email.
@@ -61,20 +61,20 @@ module MList
         'list-post'        => post_url
       }
     end
-    
+
     # Answers a unique, never changing value for this list.
     #
     def list_id
       raise 'answer a unique, never changing value'
     end
-    
+
     # The web address where an archive of this list may be found, nil if there
     # is no archive.
     #
     def archive_url
       nil
     end
-    
+
     # Should the sender of an email be copied in the publication? Defaults to
     # false. If _recipients_ includes the sender email address, it will be
     # removed if this answers false.
@@ -84,47 +84,47 @@ module MList
     def copy_sender?(subscriber)
       false
     end
-    
+
     # The web address of the list help site, nil if this is not supported.
     #
     def help_url
       nil
     end
-    
+
     # The email address of the list owner, nil if this is not supported.
     #
     def owner_url
       nil
     end
-    
+
     # The email address where posts should be sent. Defaults to the address of
     # the list.
     #
     def post_url
       address
     end
-    
+
     # Should the reply-to header be set to the list's address? Defaults to
     # true. If false is returned, the reply-to will be the subscriber address.
     #
     def reply_to_list?
       true
     end
-    
+
     # The web url where subscriptions to this list may be created, nil if this
     # is not supported.
     #
     def subscribe_url
       nil
     end
-    
+
     # The web url where subscriptions to this list may be deleted, nil if this
     # is not supported.
     #
     def unsubscribe_url
       nil
     end
-    
+
     # A list is responsible for answering the recipient subscribers. The
     # answer may or may not include the subscriber; _copy_sender?_ will be
     # invoked and the subscriber will be added or removed from the Array.
@@ -135,16 +135,16 @@ module MList
     def recipients(subscriber)
       subscribers
     end
-    
+
     # A list must answer the subscriber who's email address is that of the one
     # provided. The default implementation will pick the first instance that
-    # answers subscriber.email_address == email_address. Your implementation
+    # answers subscriber.rfc5322_email == email_address. Your implementation
     # should probably select just one record.
     #
     def subscriber(email_address)
-      subscribers.detect {|s| s.email_address == email_address}
+      subscribers.detect {|s| s.rfc5322_email == email_address}
     end
-    
+
     # A list must answer whether there is a subscriber who's email address is
     # that of the one provided. This is checked before the subscriber is
     # requested in order to allow for the lightest weight check possible; that
@@ -154,12 +154,12 @@ module MList
     def subscriber?(email_address)
       !subscriber(email_address).nil?
     end
-    
+
     # Methods that will be invoked on your implementation of Mlist::List when
     # certain events occur during the processing of email sent to a list.
     #
     module Callbacks
-      
+
       # Called when an email is a post to the list by a subscriber whom the
       # list claims is blocked (answers true to _blocked?(subscriber)_). This
       # will not be called if the list is inactive (answers false to
@@ -167,10 +167,10 @@ module MList
       #
       def blocked_subscriber_post(email, subscriber)
       end
-      
+
       def bounce(email)
       end
-      
+
       # Called when an email is a post to the list while the list is inactive
       # (answers false to _active?_). This will not be called if the email is
       # from a non-subscribed sender. Instead, _non_subscriber_post_ will be
@@ -178,7 +178,7 @@ module MList
       #
       def inactive_post(email)
       end
-      
+
       # Called when an email is a post to the list from a non-subscribed
       # sender. This will be called even if the list is inactive.
       #
@@ -186,6 +186,6 @@ module MList
       end
     end
     include Callbacks
-    
+
   end
-end
+end

data/lib/mlist/mail_list.rb

@@ -209,7 +209,7 @@ module MList
         message.subject = clean_subject(message.subject)
 
         recipient_addresses = message.recipient_addresses
-        sender_address = message.subscriber.email_address
+        sender_address = message.subscriber.rfc5322_email
         if options[:copy_sender]
           recipient_addresses << sender_address unless recipient_addresses.include?(sender_address)
         else

data/lib/mlist/manager/database.rb

@@ -1,9 +1,9 @@
 module MList
   module Manager
-    
+
     class Database
       include ::MList::Manager
-      
+
       def create_list(address, attributes = {})
         attributes = {
           :address => address,
@@ -11,38 +11,38 @@ module MList
         }.merge(attributes)
         List.create!(attributes)
       end
-      
+
       def lists(email)
         lists = List.find_all_by_address(email.list_addresses)
         email.list_addresses.map { |a| lists.detect {|l| l.address == a} }.compact
       end
-      
+
       def no_lists_found(email)
         # TODO: Move to notifier
       end
-      
+
       class List < ActiveRecord::Base
         include ::MList::List
-        
+
         has_many :subscribers, :dependent => :delete_all
-        
+
         def label
           self[:label]
         end
-        
+
         def list_id
           "#{self.class.name}#{id}"
         end
-        
+
         def subscribe(address)
-          subscribers.find_or_create_by_email_address(address)
+          subscribers.find_or_create_by_rfc5322_email(address)
         end
       end
-      
+
       class Subscriber < ActiveRecord::Base
         belongs_to :list
       end
     end
-    
+
   end
-end
+end

data/lib/mlist/manager/notifier.rb

@@ -1,11 +1,11 @@
 module MList
   module Manager
-    
+
     # Constructs the notices that are sent to list subscribers. Applications
     # may subclass this to customize the content of a notice delivery.
     #
     class Notifier
-      
+
       # Answers the delivery that will be sent to a subscriber when an
       # MList::List indicates that the distribution of an email from that
       # subscriber has been blocked.
@@ -14,18 +14,18 @@ module MList
         delivery = MList::Util::TMailBuilder.new(TMail::Mail.new)
         delivery.write_header('x-mlist-loop', 'notice')
         delivery.write_header('x-mlist-notice', 'subscriber_blocked')
-        delivery.to = subscriber.email_address
+        delivery.to = subscriber.rfc5322_email
         delivery.from = "mlist-#{list.address}"
         prepare_subscriber_blocked_content(list, email, subscriber, delivery)
         delivery
       end
-      
+
       protected
         def prepare_subscriber_blocked_content(list, email, subscriber, delivery)
           delivery.set_content_type('text/plain')
           delivery.body = %{Although you are a subscriber to this list, your message cannot be posted at this time. Please contact the administrator of the list.}
         end
     end
-    
+
   end
-end
+end

data/lib/mlist/message.rb

@@ -1,22 +1,22 @@
 module MList
-  
+
   class Message < ActiveRecord::Base
     set_table_name 'mlist_messages'
-    
+
     include MList::Util::EmailHelpers
-    
+
     belongs_to :email, :class_name => 'MList::Email'
     belongs_to :parent, :class_name => 'MList::Message'
     belongs_to :mail_list, :class_name => 'MList::MailList', :counter_cache => :messages_count
     belongs_to :thread, :class_name => 'MList::Thread', :counter_cache => :messages_count
-    
+
     after_destroy :delete_unreferenced_email
-    
+
     # A temporary storage of recipient subscribers, obtained from
     # MList::Lists. This list is not available when a message is reloaded.
     #
     attr_accessor :recipients
-    
+
     # Answers an MList::TMailBuilder for assembling the TMail::Mail object
     # that will be fit for delivery. If this is not a new message, the
     # delivery will be updated to reflect the message-id, x-mailer, etc. of
@@ -33,26 +33,26 @@ module MList
         d
       end
     end
-    
+
     def email_with_capture=(email)
       self.subject = email.subject
       self.mailer = email.mailer
       self.email_without_capture = email
     end
     alias_method_chain :email=, :capture
-    
+
     # Answers the html content of the message.
     #
     def html
       email.html
     end
-    
+
     # Answers the text content of the message.
     #
     def text
       email.text
     end
-    
+
     # Answers the text content of the message as HTML. The structure of this
     # output is very simple. For examples of what it can handle, please check
     # out the spec documents for MList::Util::EmailHelpers.
@@ -60,21 +60,21 @@ module MList
     def text_html
       text_to_html(text)
     end
-    
+
     # Answers text suitable for creating a reply message.
     #
     def text_for_reply
       timestamp = email.date.to_s(:mlist_reply_timestamp)
       "On #{timestamp}, #{email.from} wrote:\n#{text_to_quoted(text)}"
     end
-    
+
     # Answers text suitable for creating a reply message, converted to the
     # same simple html of _text_html_.
     #
     def html_for_reply
       text_to_html(text_for_reply)
     end
-    
+
     def parent_with_identifier_capture=(parent)
       if parent
         self.parent_without_identifier_capture = parent
@@ -85,7 +85,7 @@ module MList
       end
     end
     alias_method_chain :parent=, :identifier_capture
-    
+
     # Answers the recipient email addresses from the MList::List recipient
     # subscribers, except those that are in the email TO or CC fields as
     # placed there by the sending MUA. It is assumed that those addresses have
@@ -93,9 +93,9 @@ module MList
     # we would cause them to receive two copies of the message.
     #
     def recipient_addresses
-      @recipients.collect(&:email_address).collect(&:downcase) - email.recipient_addresses
+      @recipients.collect(&:rfc5322_email).collect(&:downcase) - email.recipient_addresses
     end
-    
+
     # Answers the subject with 'Re:' prefixed. Note that it is the
     # responsibility of the MList::MailList to perform any processing of the
     # persisted subject (ie, cleaning up labels, etc).
@@ -109,7 +109,7 @@ module MList
     def subject_for_reply
       subject =~ REGARD_RE ? subject : "Re: #{subject}"
     end
-    
+
     # Answers the subscriber from which this message comes.
     #
     def subscriber
@@ -121,19 +121,19 @@ module MList
         end
       end
     end
-    
+
     # Assigns the subscriber from which this message comes.
     #
     def subscriber=(subscriber)
       case subscriber
       when ActiveRecord::Base
         @subscriber = subscriber
-        self.subscriber_address = subscriber.email_address
+        self.subscriber_address = subscriber.rfc5322_email
         self.subscriber_type = subscriber.class.base_class.name
         self.subscriber_id = subscriber.id
       when MList::EmailSubscriber
         @subscriber = subscriber
-        self.subscriber_address = subscriber.email_address
+        self.subscriber_address = subscriber.rfc5322_email
         self.subscriber_type = self.subscriber_id = nil
       when String
         self.subscriber = MList::EmailSubscriber.new(subscriber)
@@ -141,10 +141,10 @@ module MList
         @subscriber = self.subscriber_address = self.subscriber_type = self.subscriber_id = nil
       end
     end
-    
+
     private
       def delete_unreferenced_email
         email.destroy unless MList::Message.count(:conditions => "email_id = #{email_id} and id != #{id}") > 0
       end
   end
-end
+end

data/lib/mlist/util/email_helpers.rb

@@ -103,7 +103,7 @@ module MList
       end
 
       def subscriber_name_and_address(subscriber)
-        a = subscriber.email_address
+        a = subscriber.rfc5322_email
         a = "#{subscriber.display_name} #{bracket(a)}" if subscriber.respond_to?(:display_name)
         a
       end

data/spec/fixtures/schema.rb

@@ -1,16 +1,16 @@
 ActiveRecord::Schema.define(:version => 20081126181722) do
-  
+
   # All MList required tables are prefixed with 'mlist_' to ease integration
   # into other systems' databases.
-  
-  
+
+
   # The table in which email content is stored.
-  # 
+  #
   create_table :mlist_emails, :force => true do |t|
     t.column :source, :text
     t.column :created_at, :datetime
   end
-  
+
   # The table in which MList will store MList::Messages.
   #
   # The identifier is the 'message-id' header value of the finally delivered
@@ -18,9 +18,9 @@ ActiveRecord::Schema.define(:version => 20081126181722) do
   #
   # An MList::Message will store a reference to your application's subscriber
   # instance if it is an ActiveRecord subclass. That subclass must respond to
-  # :email_address. If your subscriber is just a string, it is assumed to be
-  # an email address. Either way, that email address will be stored with the
-  # MList::Message, providing a way for you to associate messages by
+  # :rfc5322_email. If your subscriber is just a string, it is assumed
+  # to be an email address. Either way, that email address will be stored with
+  # the MList::Message, providing a way for you to associate messages by
   # subscriber_address. This is less ideal, as you may allow subscribers to
   # change their email addresses over time.
   #
@@ -46,7 +46,7 @@ ActiveRecord::Schema.define(:version => 20081126181722) do
   add_index :mlist_messages, :subject
   add_index :mlist_messages, :subscriber_address
   add_index :mlist_messages, [:subscriber_type, :subscriber_id]
-  
+
   # Every MList::Message is associated with an MList::Thread.
   #
   create_table :mlist_threads, :force => true do |t|
@@ -55,7 +55,7 @@ ActiveRecord::Schema.define(:version => 20081126181722) do
     t.timestamps
   end
   add_index :mlist_threads, :mail_list_id
-  
+
   # The table in which MList will store MList::MailLists.
   #
   # The manager_list_identifier column stores the MList::List#list_id value.
@@ -76,8 +76,8 @@ ActiveRecord::Schema.define(:version => 20081126181722) do
   add_index :mlist_mail_lists, :manager_list_identifier
   add_index :mlist_mail_lists, [:manager_list_identifier, :manager_list_type, :manager_list_id],
     :name => :index_mlist_mail_lists_on_manager_association
-  
-  
+
+
   # Database list manager tables, used for testing purposes.
   #
   create_table :lists, :force => true do |t|
@@ -85,10 +85,10 @@ ActiveRecord::Schema.define(:version => 20081126181722) do
     t.column :label, :string
     t.column :created_at, :datetime
   end
-  
+
   create_table :subscribers, :force => true do |t|
     t.column :list_id, :integer
-    t.column :email_address, :string
+    t.column :rfc5322_email, :string
     t.column :created_at, :datetime
   end
 end

data/spec/integration/mlist_spec.rb

@@ -12,7 +12,7 @@ describe MList do
       end.should_not store_message
     end
   end
-  
+
   def store_message
     simple_matcher('store message') do |block|
       thread_count, message_count = MList::Thread.count, MList::Message.count
@@ -20,7 +20,7 @@ describe MList do
       MList::Thread.count == thread_count + 1 && MList::Message.count == message_count + 1
     end
   end
-  
+
   def start_new_thread(tmail)
     simple_matcher('start new thread') do |email_server|
       delivery_count = email_server.deliveries.size
@@ -32,34 +32,34 @@ describe MList do
         MList::Message.count == (message_count + 1)
     end
   end
-  
+
   dataset do
     @list_manager = MList::Manager::Database.new
     @list_one = @list_manager.create_list('list_one@example.com')
     @list_one.subscribe('adam@nomail.net')
     @list_one.subscribe('tom@example.com')
     @list_one.subscribe('dick@example.com')
-    
+
     @list_two = @list_manager.create_list('list_two@example.com')
     @list_two.subscribe('adam@nomail.net')
     @list_two.subscribe('jane@example.com')
-    
+
     @list_three = @list_manager.create_list('empty@example.com')
     @list_three.subscribe('adam@nomail.net')
   end
-  
+
   before do
     @email_server = MList::EmailServer::Fake.new
     @server = MList::Server.new(
       :list_manager => @list_manager,
       :email_server => @email_server
     )
-    
+
     # TODO Move this stuff to Dataset
     ActiveRecord::Base.connection.increment_open_transactions
     ActiveRecord::Base.connection.begin_db_transaction
   end
-  
+
   after do
     if ActiveRecord::Base.connection.open_transactions != 0
       ActiveRecord::Base.connection.rollback_db_transaction
@@ -67,39 +67,39 @@ describe MList do
     end
     ActiveRecord::Base.clear_active_connections!
   end
-  
+
   it 'should have threads and mail_lists updated_at set to last message receive time' do
     now = Time.now
     stub(Time).now {now}
     @email_server.receive(tmail_fixture('single_list'))
     MList::MailList.last.updated_at.to_s.should == now.to_s
     MList::Thread.last.updated_at.to_s.should == now.to_s
-    
+
     later = 5.days.from_now
     stub(Time).now {later}
     @email_server.receive(tmail_fixture('single_list_reply'))
     MList::MailList.last.updated_at.to_s.should == later.to_s
     MList::Thread.last.updated_at.to_s.should == later.to_s
   end
-  
+
   it 'should associate manager lists to mlist mail lists when they are ActiveRecord instances' do
     @email_server.receive(tmail_fixture('single_list'))
     mail_list = MList::MailList.last
     mail_list.manager_list.should == @list_one
     mail_list.manager_list_identifier.should == @list_one.list_id
   end
-  
+
   it 'should not forward mail that has been on this server before' do
     # The sample email has multiple headers to prove that's not a problem
     @email_server.should_not forward_email(tmail_fixture('x-beenthere'))
   end
-  
+
   it 'should store message/thread even when there are no recipient subscribers' do
     tmail = tmail_fixture('single_list')
     tmail.to = @list_three.address
     @email_server.should start_new_thread(tmail)
   end
-  
+
   it 'should not forward mail from non-subscriber and notify manager list' do
     tmail = tmail_fixture('single_list')
     tmail.from = 'unknown@example.com'
@@ -107,7 +107,7 @@ describe MList do
     mock(@list_one).non_subscriber_post(is_a(MList::Email))
     @email_server.should_not forward_email(tmail)
   end
-  
+
   it 'should not forward mail from non-subscriber when inactive and notify as non-subscriber' do
     tmail = tmail_fixture('single_list')
     tmail.from = 'unknown@example.com'
@@ -116,9 +116,9 @@ describe MList do
     mock(@list_one).non_subscriber_post(is_a(MList::Email))
     @email_server.should_not forward_email(tmail)
   end
-  
+
   it 'should not forward mail from blocked subscriber, notify the subscriber using list manager notifier' do
-    subscriber = MList::Manager::Database::Subscriber.find_by_email_address('adam@nomail.net')
+    subscriber = MList::Manager::Database::Subscriber.find_by_rfc5322_email('adam@nomail.net')
     tmail = tmail_fixture('single_list')
     mock(@list_one).active? {true}
     mock(@list_one).blocked?(subscriber) { true }
@@ -132,7 +132,7 @@ describe MList do
       response_tmail.header_string('x-mlist-notice').should == 'subscriber_blocked'
     end.should_not store_message
   end
-  
+
   it 'should not forward mail to inactive list and notify manager list' do
     tmail = tmail_fixture('single_list')
     mock(@list_one).active? { false }
@@ -140,26 +140,26 @@ describe MList do
     mock(@list_one).inactive_post(is_a(MList::Email))
     @email_server.should_not forward_email(tmail)
   end
-  
+
   it 'should report unrecognized email to list manager' do
     tmail = tmail_fixture('single_list')
     stub(@list_manager).lists(is_a(MList::Email)) { [] }
     mock(@list_manager).no_lists_found(is_a(MList::Email))
     @email_server.should_not forward_email(tmail)
   end
-  
+
   it 'should report bounces to the list manager' do
     stub(@list_manager).lists(is_a(MList::Email)) { [@list_one] }
     mock(@list_one).bounce(is_a(MList::Email))
     @email_server.should_not forward_email(tmail_fixture('bounces/1'))
   end
-  
+
   describe 'single list' do
     before do
       @tmail_post = tmail_fixture('single_list')
       @email_server.receive(@tmail_post)
     end
-    
+
     it 'should forward emails that are sent to a mailing list' do
       @email_server.deliveries.size.should == 1
       email = @email_server.deliveries.first
@@ -167,12 +167,12 @@ describe MList do
       # bcc fields are not in the headers of delivered emails
       email.should have_address(:'reply-to', 'list_one@example.com')
     end
-    
+
     it 'should start a new thread for a new email' do
       thread = MList::Thread.last
       thread.messages.first.email.tmail.should equal_tmail(@tmail_post)
     end
-    
+
     it 'should add to an existing thread when reply email' do
       message = MList::Message.last
       reply_tmail = tmail_fixture('single_list_reply', 'in-reply-to' => "<#{message.identifier}>")
@@ -180,7 +180,7 @@ describe MList do
       thread = MList::Thread.last
       thread.messages.size.should be(2)
     end
-    
+
     it 'should associate parent message when reply email' do
       message = MList::Message.last
       @email_server.receive(tmail_fixture('single_list_reply', 'in-reply-to' => "<#{message.identifier}>"))
@@ -188,46 +188,46 @@ describe MList do
       reply.parent_identifier.should == message.identifier
       reply.parent.should == message
     end
-    
+
     it 'should not associate a parent when not a reply' do
       tmail = tmail_fixture('single_list')
       tmail['message-id'] = 'asdfasdfj'
       tmail['subject'] = 'other thing'
       @email_server.should start_new_thread(tmail)
     end
-    
+
     it 'should store subscriber address with messages' do
       MList::Message.last.subscriber_address.should == 'adam@nomail.net'
     end
-    
+
     it 'should associate subscriber to messages when they are ActiveRecord instances' do
-      MList::Message.last.subscriber.should == MList::Manager::Database::Subscriber.find_by_email_address('adam@nomail.net')
+      MList::Message.last.subscriber.should == MList::Manager::Database::Subscriber.find_by_rfc5322_email('adam@nomail.net')
     end
   end
-  
+
   describe 'multiple lists' do
     before do
       @tmail_post = tmail_fixture('multiple_lists')
       @email_server.receive(@tmail_post)
     end
-    
+
     it 'should forward emails that are sent to a mailing list' do
       @email_server.deliveries.size.should == 2
-      
+
       email = @email_server.deliveries.first
       email.should have_address(:to, 'list_one@example.com')
       # bcc fields are not in the headers of delivered emails
       email.should have_address(:'reply-to', 'list_one@example.com')
-      
+
       email = @email_server.deliveries.last
       email.should have_address(:to, 'list_two@example.com')
       # bcc fields are not in the headers of delivered emails
       email.should have_address(:'reply-to', 'list_two@example.com')
     end
-    
+
     it 'should start a new thread for each list, both referencing the same email' do
       threads = MList::Thread.find(:all)
       threads[0].messages.first.email.should == threads[1].messages.first.email
     end
   end
-end
+end

data/spec/models/mail_list_spec.rb

@@ -122,7 +122,7 @@ describe MList::MailList do
         :copy_sender => true)
 
       tmail = @outgoing_server.deliveries.last
-      tmail.bcc.should include(@subscriber_one.email_address)
+      tmail.bcc.should include(@subscriber_one.rfc5322_email)
     end
 
     it 'should not copy the subscriber if undesired and list includes the subscriber' do
@@ -136,7 +136,7 @@ describe MList::MailList do
         :copy_sender => false)
 
       tmail = @outgoing_server.deliveries.last
-      tmail.bcc.should_not include(@subscriber_one.email_address)
+      tmail.bcc.should_not include(@subscriber_one.rfc5322_email)
     end
   end
 

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 18
-  version: 0.1.18
+  - 19
+  version: 0.1.19
 platform: ruby
 authors: 
 - Adam Williams
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-06-11 00:00:00 -04:00
+date: 2010-08-12 00:00:00 -04:00
 default_executable: 
 dependencies: []